From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751836Ab0A3EI7 (ORCPT ); Fri, 29 Jan 2010 23:08:59 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751477Ab0A3EI7 (ORCPT ); Fri, 29 Jan 2010 23:08:59 -0500 Received: from mail-yx0-f193.google.com ([209.85.210.193]:62524 "EHLO mail-yx0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751373Ab0A3EI6 (ORCPT ); Fri, 29 Jan 2010 23:08:58 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:content-type; b=KlylodORl/5iZRgMhkiUSA9xrk1Yz1QCpB9hGQDEwFbKqmcc2mKUHZlD/WDQrFF7EZ +A1YONcdAz+sbA6m7025s8VUaKkDrspIA0kN49FnL+CPaJJbFJC+gx3LzSOVQC9XV3pM yf1UV/mL/Do+oEbyF5chBuYkj/Re+NTqx9EwM= MIME-Version: 1.0 Date: Fri, 29 Jan 2010 22:08:57 -0600 Message-ID: <524f69651001292008g39c796a7j6f41c04b599752c7@mail.gmail.com> Subject: Re: [patch 05/10] cifs: convert semaphore to mutex From: Steve French To: Christoph Hellwig , Jeff Layton , LKML , Ingo Molnar , Peter Zijlstra , Thomas Gleixner Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Christoph Hellwig wrote on 01/29/2010 04:29:05 PM: > Christoph Hellwig > 01/29/2010 04:29 PM > > To > > Thomas Gleixner > > cc > > LKML , Christoph Hellwig > , Peter Zijlstra , Ingo > Molnar , Jeff Layton , Steven > French/Austin/IBM@IBMUS > > Subject > > Re: [patch 05/10] cifs: convert semaphore to mutex > > On Fri, Jan 29, 2010 at 08:38:50PM -0000, Thomas Gleixner wrote: > > pSesInfo->sesSem is used as mutex. Rename it to session_mutex and > > convert it to a real mutex. > > > > Signed-off-by: Thomas Gleixner > > Cc: Jeff Layton > > Cc: Steve French > > Looks good, but I'm pretty sure I've seen an equivalent patch on the > linux-cifs-client list before. > > > Reviewed-by: Christoph Hellwig Jeff and I acked the cifs part of the patch series in July - not sure why the series didn't go in. http://marc.info/?l=linux-kernel&m=124859649902788&w=2 looks like the original post -- Thanks, Steve