From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH v2] hfsplus: avoid crash on failed block map free Date: Wed, 19 Dec 2012 15:09:03 +0100 Message-ID: <20121219140903.GA32544@lst.de> References: <1352799065.2443.3.camel@slavad-ubuntu> <20121113143444.f8d4bbc3.akpm@linux-foundation.org> <1352897453.2149.14.camel@slavad-ubuntu> <20121218144755.475462ee.akpm@linux-foundation.org> <1355898397.2528.12.camel@slavad-ubuntu> <20121218222936.1db1fa8e.akpm@linux-foundation.org> <1355899368.2528.25.camel@slavad-ubuntu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andrew Morton , Alan Cox , linux-fsdevel@vger.kernel.org, Christoph Hellwig To: Vyacheslav Dubeyko Return-path: Received: from verein.lst.de ([213.95.11.211]:46289 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751218Ab2LSOJG (ORCPT ); Wed, 19 Dec 2012 09:09:06 -0500 Content-Disposition: inline In-Reply-To: <1355899368.2528.25.camel@slavad-ubuntu> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Dec 19, 2012 at 10:42:48AM +0400, Vyacheslav Dubeyko wrote: > > Yes, they've all been in linux-next for a while. > > > > To be more specific: is Christoph OK with a mainline merge? > > I think that we need to ask his about it. :-) I haven't any private > e-mailing with him about it. > > Maybe, it makes sense to merge in mainline reworked error processing > patches previously and, then, to merge the xattr support after ACLs > support implementation. > > Christoph, could you share your opinion? I haven't actually managed to look at your new xattr patches, but at least conceptually I'm fine with them. Let's get the error handling and misc hfsplus patches in ASAP, in the meantime I'll get a review for the xattr bits in, and then we can decide to merge them directly or together with the ACL bits.