Messages in this thread | | | Date | Mon, 12 May 2014 19:50:39 +0200 | From | Manfred Spraul <> | Subject | Re: [PATCH 1/6] ipc/sem.c: further whitespace cleanups |
| |
Hi Davidlohr,
On 05/12/2014 01:34 AM, Davidlohr Bueso wrote: > On Sat, 2014-05-10 at 12:03 +0200, Manfred Spraul wrote: >> Somehow <TAB>$ was overlooked in the last round of whitespace >> cleanups. >> Do that now, before making further changes. > No big deal, but this patch could easily be included in the the second > patch instead on its own. Thanks. The patch series you see is the third approach to fix the issue: - do not store anything, call perform_atomic_semop() in count_semncnt() Really cool, no impact at all for semop() - but nasty. A sleeping operation could fail suddenly due to -ERANGE.
- store semncnt and semzcnt in struct sem: Increase in perform_atomic_semop(), decrease in unlink_queue().
- Now: store the pointer to the blocking op.
That's why the series got so fine-grained...
-- Manfred
| |