From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754009AbaKMPgR (ORCPT ); Thu, 13 Nov 2014 10:36:17 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:33054 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752261AbaKMPgP (ORCPT ); Thu, 13 Nov 2014 10:36:15 -0500 Date: Thu, 13 Nov 2014 07:34:58 -0800 From: Greg Kroah-Hartman To: Martin Kepplinger Cc: Miklos Szeredi , fuse-devel , thierry.reding@gmail.com, giedriuswork@gmail.com, Kernel Mailing List Subject: Re: [PATCH] fuse: don't check for file->private_data on open(). Message-ID: <20141113153458.GB26905@kroah.com> References: <20141112112643.GA30821@ulmo.nvidia.com> <1415809894-24084-1-git-send-email-martink@posteo.de> <0198135B-B7F1-4E40-A9E2-C9D56C421B6F@posteo.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <0198135B-B7F1-4E40-A9E2-C9D56C421B6F@posteo.de> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 13, 2014 at 12:13:32PM +0100, Martin Kepplinger wrote: > Am 13. November 2014 11:53:29 MEZ, schrieb Miklos Szeredi : > >On Thu, Nov 13, 2014 at 11:05 AM, Martin Kepplinger > >wrote: > > > >> In this week's -next this should have changed. My SSD broke down so i > >have to delay further work for a few days, i'm sorry. > > > >Please be more careful with such patches. Have you audited all of the > >(ca. 200) misc drivers? If not, this might be a good time to do so. > >If it turns out to be too much, then consider not doing the change. > >The gain might not be worth the cost. > > > >NACK in this form. > > > >Thanks, > >Miklos > > Definitely should have been more careful as I checked fs too late now. I totally unnecessarily broke -next. > > But If fuse-devel is ok with a fix like the one i sent (still has to get tested too) i believe we're covered. I still call for independent review though. I don't think it's worth it anymore, I'm going to revert the original patch that caused these problems, as it has the potential to break other drivers in ways that are not obvious (i.e. not at build time.) Sorry, greg k-h