Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752372AbbD3Owd (ORCPT ); Thu, 30 Apr 2015 10:52:33 -0400 Received: from bedivere.hansenpartnership.com ([66.63.167.143]:40557 "EHLO bedivere.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751172AbbD3Ow3 (ORCPT ); Thu, 30 Apr 2015 10:52:29 -0400 Message-ID: <1430405547.2167.12.camel@HansenPartnership.com> Subject: Re: [RFC 1/3] sysfs,kernfs: add flush operation From: James Bottomley To: Greg Kroah-Hartman Cc: linux-efi@vger.kernel.org, "Kweh, Hock Leong" , LKML , Andy Lutomirski , Peter Jones Date: Thu, 30 Apr 2015 07:52:27 -0700 In-Reply-To: <20150430131159.GC17936@kroah.com> References: <1430348859.2189.37.camel@HansenPartnership.com> <1430348985.2189.39.camel@HansenPartnership.com> <20150430131159.GC17936@kroah.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.11 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1497 Lines: 43 On Thu, 2015-04-30 at 15:11 +0200, Greg Kroah-Hartman wrote: > On Wed, Apr 29, 2015 at 04:09:45PM -0700, James Bottomley wrote: > > From: James Bottomley > > > > This is necessary to allow sysfs operations to return error in close (we are > > using close to initiate and return a possible error from a transaction). > > > > Signed-off-by: James Bottomley > > Are there any side-affects now that we have a flush() call for binary > files that the vfs will act differently from not defining one at all? See answer to Andy, but basically, no, VFS flush is only used to tidy up before close. To be honest, it might be nice if fsync *would* issue a transaction because I think having a multi-transaction API with write/fsync/write is not such a bad thing. It turns out that fsync and fdatasync do have a VFS op, but it's ->fsync() not ->flush(), so we have the option to wire this up if anyone finds a use case. > if not, no objection to me for this change. Thanks, James > thanks, > > greg k-h > -- > To unsubscribe from this list: send the line "unsubscribe linux-efi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/