Re: [PATCH 2/2] ipc: semaphores: do not hold ipc lock more than necessary
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH 2/2] ipc: semaphores: do not hold ipc lock more than necessary
- From: Davidlohr Bueso <davidlohr.bueso@xxxxxx>
- Date: Sat, 02 Mar 2013 13:16:50 -0800
- Cc: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Rik van Riel <riel@xxxxxxxxxx>, "Vinod, Chegu" <chegu_vinod@xxxxxx>, "Low, Jason" <jason.low2@xxxxxx>, linux-tip-commits@xxxxxxxxxxxxxxx, Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, aquini@xxxxxxxxxx, Ingo Molnar <mingo@xxxxxxxxxx>, Larry Woodman <lwoodman@xxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, Steven Rostedt <rostedt@xxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>
- In-reply-to: <CANN689FuYfNZjPSPYFKq4HkN5MVe_dOOAm8yswKE=bvw8s4vUQ@mail.gmail.com>
On Sat, 2013-03-02 at 12:41 +0800, Michel Lespinasse wrote:
> On Sat, Mar 2, 2013 at 8:16 AM, Davidlohr Bueso <davidlohr.bueso@xxxxxx> wrote:
> > Instead of holding the ipc lock for permissions and security
> > checks, among others, only acquire it when necessary.
> >
> > Signed-off-by: Davidlohr Bueso <davidlohr.bueso@xxxxxx>
>
> You got some really great test results on this; I think they deserve
> to be mentioned in the commit message.
Absolutely.
>
> Code looks fine to me otherwise, but I only had a quick look.
>
> Nice work!
>
> Acked-by: Michel Lespinasse <walken@xxxxxxxxxx>
>
Thanks for reviewing, Michel.
Davidlohr
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]