Page MenuHomeFreeBSD
Differential D26826

sysv_sem: semusz depends on semume.
ClosedPublic
Actions

Authored by kib on Oct 17 2020, 9:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Aug 14, 6:14 PM
Unknown Object (File)
Thu, Aug 13, 7:42 PM
Unknown Object (File)
Thu, Aug 13, 3:04 PM
Unknown Object (File)
Thu, Aug 13, 12:15 PM
Unknown Object (File)
Wed, Aug 12, 8:54 PM
Unknown Object (File)
Wed, Aug 12, 4:41 PM
Unknown Object (File)
Wed, Aug 12, 4:41 PM
Unknown Object (File)
Wed, Aug 12, 4:41 PM
View All Files
Subscribers
imp

Details

Reviewers
markj
jhb
Commits
rS366932: sysv_sem: semusz depends on semume.
Summary

Size of the per-process semaphore undo structure (semusz) depends on the number of the per-process undos. If kern.ipc.semume is adjusted, semusz must be adjusted as well, and it makes no sense to delegate adjustment to user. Make it automatic.

PR: 250361

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.Oct 17 2020, 9:07 AM
Herald added a subscriber: imp. · View Herald TranscriptOct 17 2020, 9:07 AM
kib requested review of this revision.Oct 17 2020, 9:07 AM
Harbormaster completed remote builds in B34225: Diff 78342.Oct 17 2020, 9:07 AM
kib added reviewers: markj, jhb.Oct 20 2020, 11:32 PM
markj accepted this revision.Oct 21 2020, 1:17 PM
markj added inline comments.
sys/kern/sysv_sem.c
196 ↗(On Diff #78342)

Missing parens around ix.

This revision is now accepted and ready to land.Oct 21 2020, 1:17 PM
kib updated this revision to Diff 78544.Oct 21 2020, 1:34 PM

Add parenthesis.

This revision now requires review to proceed.Oct 21 2020, 1:34 PM
Harbormaster completed remote builds in B34299: Diff 78544.Oct 21 2020, 1:35 PM
markj accepted this revision.Oct 21 2020, 1:37 PM
This revision is now accepted and ready to land.Oct 21 2020, 1:37 PM
jhb accepted this revision.Oct 21 2020, 4:08 PM
Closed by commit rS366932: sysv_sem: semusz depends on semume. (authored by kib). · Explain WhyOct 22 2020, 9:28 AM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rS366932: sysv_sem: semusz depends on semume..

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
9 lines

Diff 78580

View Options

head/sys/kern/sysv_sem.c

Loading...