Page MenuHomeFreeBSD
Differential D13768

Always use atomic_fetchadd() when updating per-user accounting values.
ClosedPublic
Actions

Authored by jhb on Jan 4 2018, 7:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 13, 2:58 PM
Unknown Object (File)
Wed, May 13, 2:35 PM
Unknown Object (File)
Mon, May 11, 6:28 PM
Unknown Object (File)
Mon, May 11, 7:14 AM
Unknown Object (File)
Sun, May 10, 12:42 PM
Unknown Object (File)
Sun, May 10, 12:41 PM
Unknown Object (File)
Fri, May 1, 12:12 PM
Unknown Object (File)
Tue, Apr 28, 1:35 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
Commits
rS327562: Always use atomic_fetchadd() when updating per-user accounting values.
Summary

This avoids re-reading a variable after it has been updated via an
atomic op. It is just a cosmetic cleanup as the read value was only
used to control a diagnostic printf that should rarely occur (if ever).

Test Plan
  • have had a variant of this patch around for several years and have booted amd64 and i386 VMs with it. OTOH, I have never seen this printf fire. It's been a low priority to ever get this reviewed but I noticed it today and figured might as well either get it in or discard it.

Diff Detail

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

Event Timeline

jhb created this revision.Jan 4 2018, 7:30 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 4 2018, 7:30 PM
Harbormaster completed remote builds in B14118: Diff 37527.Jan 4 2018, 7:30 PM
kib accepted this revision.Jan 4 2018, 8:11 PM
kib added inline comments.
sys/kern/kern_resource.c
1397 ↗(On Diff #37527)

You can write '} else if (new < 0) {' to reduce indent.

This revision is now accepted and ready to land.Jan 4 2018, 8:11 PM
Closed by commit rS327562: Always use atomic_fetchadd() when updating per-user accounting values. (authored by jhb). · Explain WhyJan 4 2018, 10:08 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
11 lines

Diff 37534

View Options

head/sys/kern/kern_resource.c

Loading...