ast_sig(): add missed TDAI()
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F137343352: D36033.id108853.diff
	Sat, Nov 22, 3:48 PM

	Unknown Object (File)
	Wed, Nov 19, 9:53 AM

	Unknown Object (File)
	Mon, Nov 10, 10:05 AM

	Unknown Object (File)
	Mon, Nov 10, 8:44 AM

	Unknown Object (File)
	Mon, Nov 10, 3:56 AM

	Unknown Object (File)
	Tue, Nov 4, 9:50 AM

	Unknown Object (File)
	Tue, Nov 4, 9:47 AM

	Unknown Object (File)
	Thu, Oct 30, 9:45 PM

View All 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 Not Applicable

Unit

Tests Not Applicable

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

2 lines

Diff 108853

View Options

ast_sig(): add missed TDAI()ClosedPublicActions