Page MenuHomeFreeBSD
Differential D56722

lockf: Avoid spinning when operating on an fd
ClosedPublic
Actions

Authored by des on Wed, Apr 29, 10:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 14, 2:50 PM
Unknown Object (File)
Thu, May 14, 1:29 PM
Unknown Object (File)
Wed, May 13, 9:24 PM
Unknown Object (File)
Wed, May 13, 9:24 PM
Unknown Object (File)
Wed, May 13, 1:49 PM
Unknown Object (File)
Tue, May 12, 11:37 AM
Unknown Object (File)
Tue, May 12, 11:37 AM
Unknown Object (File)
Mon, May 11, 1:48 PM
View All 22 Files
Subscribers
imp
markj

Details

Reviewers
kevans
markj
Commits
rGd0f618e7ae95: lockf: Avoid spinning when operating on an fd
rG651b82e64ef4: lockf: Avoid spinning when operating on an fd
rG4a6c4f0265ac: lockf: Avoid spinning when operating on an fd
rG2052d09b7a5f: lockf: Avoid spinning when operating on an fd
rGd90513ea8569: lockf: Avoid spinning when operating on an fd
Summary

When operating on a file descriptor, acquire_lock() would ignore the
flags argument and always operate in non-blocking mode, resulting in
unnecessary busy-looping.

PR: 294832
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Wed, Apr 29, 10:50 PM
Herald added a subscriber: imp. · View Herald TranscriptWed, Apr 29, 10:50 PM
des requested review of this revision.Wed, Apr 29, 10:50 PM
Harbormaster completed remote builds in B72645: Diff 176831.Wed, Apr 29, 10:50 PM
des added a child revision: D56723: lockf: Test that lockf does not spin on fd.Wed, Apr 29, 10:51 PM
markj accepted this revision.Sat, May 2, 5:28 PM
markj added a subscriber: markj.
markj added inline comments.
usr.bin/lockf/lockf.c
322–324

Better would be to declare lflags in this block, it's only used there.

This revision is now accepted and ready to land.Sat, May 2, 5:28 PM
des updated this revision to Diff 177050.Sat, May 2, 6:52 PM

rf

This revision now requires review to proceed.Sat, May 2, 6:52 PM
Harbormaster completed remote builds in B72727: Diff 177050.Sat, May 2, 6:52 PM
markj accepted this revision.Sun, May 3, 3:25 PM
This revision is now accepted and ready to land.Sun, May 3, 3:25 PM
Closed by commit rGd90513ea8569: lockf: Avoid spinning when operating on an fd (authored by Christian Ullrich <chris@chrullrich.net>, committed by des). · Explain WhySun, May 3, 3:35 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGd90513ea8569: lockf: Avoid spinning when operating on an fd.
des added a commit: rG2052d09b7a5f: lockf: Avoid spinning when operating on an fd.Wed, May 6, 5:38 AM
des added a commit: rG4a6c4f0265ac: lockf: Avoid spinning when operating on an fd.
des added a commit: rG651b82e64ef4: lockf: Avoid spinning when operating on an fd.
cperciva added a commit: rGd0f618e7ae95: lockf: Avoid spinning when operating on an fd.Wed, May 6, 4:31 PM

Revision Contents
Changeset List

PathSize
usr.bin/
lockf/
6 lines

Diff 177072

View Options

usr.bin/lockf/lockf.c

Loading...