ast_sig(): add missed TDAI()
ClosedPublic
Actions

Authored by kib on Aug 3 2022, 1:47 PM.

Tags

None

Referenced Files

	F108565902: D36033.diff
	Sun, Jan 26, 10:08 AM

	Unknown Object (File)
	Thu, Jan 23, 6:35 PM

	Unknown Object (File)
	Sun, Jan 12, 2:47 PM

	Unknown Object (File)
	Mon, Jan 6, 9:11 PM

	Unknown Object (File)
	Dec 12 2024, 3:03 AM

	Unknown Object (File)
	Dec 10 2024, 11:01 PM

	Unknown Object (File)
	Dec 5 2024, 7:18 PM

	Unknown Object (File)
	Oct 29 2024, 11:49 PM

View All 51 Files

Subscribers

imp

sigsys_gmail.com

Details

Reviewers

markj

Commits

rGc53fec76033a: sig_suspend_threads(): remove 'sending' arg
rGf2fd7d8bfc69: ast_sig(): add missed TDAI()

Summary

Mask checked was completely wrong

Also the cleanup:

sig_suspend_threads(): remove 'sending' arg

The TDA_AST flag is set on td2 unconditionally (as it was TDF_ASTPENDING
before AST rework), so it is not used practically for some time.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

kib created this revision.Aug 3 2022, 1:47 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 3 2022, 1:47 PM

kib requested review of this revision.Aug 3 2022, 1:47 PM

markj accepted this revision.Aug 3 2022, 1:54 PM

This revision is now accepted and ready to land.Aug 3 2022, 1:54 PM

kib added a subscriber: sigsys_gmail.com.Aug 3 2022, 1:54 PM

Closed by commit rGf2fd7d8bfc69: ast_sig(): add missed TDAI() (authored by kib). · Explain WhyAug 3 2022, 1:56 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rGf2fd7d8bfc69: ast_sig(): add missed TDAI().

kib added a commit: rGc53fec76033a: sig_suspend_threads(): remove 'sending' arg.

I don't understand much of it, but I can confirm that the last commits fixed the problem I noticed with signals sent in syscallret().

Revision Contents
Changeset List

Path

Size

sys/

kern/

kern_sig.c

15 lines

Diff 108852

View Options

ast_sig(): add missed TDAI()ClosedPublicActions