update proc->p_ptevents annotation
ClosedPublic
Actions

Authored by yanko.yankulov_gmail.com on Jun 21 2018, 4:55 PM.

Tags

None

Referenced Files

	F82797057: D15954.diff
	Thu, May 2, 4:39 PM

	Unknown Object (File)
	Thu, Apr 25, 9:41 PM

	Unknown Object (File)
	Thu, Apr 25, 9:41 PM

	Unknown Object (File)
	Thu, Apr 25, 9:41 PM

	Unknown Object (File)
	Thu, Apr 25, 9:39 PM

	Unknown Object (File)
	Thu, Apr 25, 9:31 PM

	Unknown Object (File)
	Mar 20 2024, 7:05 PM

	Unknown Object (File)
	Dec 20 2023, 4:17 AM

View All 14 Files

Subscribers

imp

Details

Reviewers

kib
jhb

Commits

rS335503: Update proc->p_ptevents annotation to reflect the actual locking.

Summary

So it reflects the actual use

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

yanko.yankulov_gmail.com created this revision.Jun 21 2018, 4:55 PM

yanko.yankulov_gmail.com added a child revision: D15857: fork: avoid endless wait with PTRACE_FORK and RFSTOPPED.

kib accepted this revision.Jun 21 2018, 5:25 PM

kib added a reviewer: jhb.

kib added inline comments.

sys/kern/sys_process.c
690	The blank line is intended. There must be the blank line after the locals, even if the locals block is empty.

This revision is now accepted and ready to land.Jun 21 2018, 5:25 PM

With @kib's requested style fix.

Restore the wrongfully deleted empty line.

This revision now requires review to proceed.Jun 21 2018, 6:52 PM

jhb accepted this revision.Jun 21 2018, 7:00 PM

This revision is now accepted and ready to land.Jun 21 2018, 7:00 PM

Closed by commit rS335503: Update proc->p_ptevents annotation to reflect the actual locking. (authored by kib). · Explain WhyJun 21 2018, 9:07 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptJun 21 2018, 9:07 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

sys_process.c

1 line

sys/

proc.h

2 lines

Diff 44252

View Options

sys/kern/sys_process.c