From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([198.137.202.9]:49688 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750792AbaKYRBC (ORCPT ); Tue, 25 Nov 2014 12:01:02 -0500 Date: Tue, 25 Nov 2014 09:01:00 -0800 From: Christoph Hellwig To: dsterba@suse.cz, Christoph Hellwig , "Theodore Ts'o" , linux-fsdevel@vger.kernel.org, Ext4 Developers List , xfs@oss.sgi.com, linux-btrfs@vger.kernel.org Subject: Re: [PATCH 1/4] fs: split update_time() into update_time() and write_time() Message-ID: <20141125170100.GA1553@infradead.org> References: <1416599964-21892-1-git-send-email-tytso@mit.edu> <1416599964-21892-2-git-send-email-tytso@mit.edu> <20141124152101.GA12575@infradead.org> <20141124173430.GB26471@twin.jikos.cz> <20141125155141.GF26471@twin.jikos.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20141125155141.GF26471@twin.jikos.cz> Sender: linux-btrfs-owner@vger.kernel.org List-ID: On Tue, Nov 25, 2014 at 04:51:41PM +0100, David Sterba wrote: > Does not work because the security.* and system.* namespaces do not call > the permission() hook, so no patch. However, if the proposed > inode_is_readonly callback is merged, we can replace the btrfs-specific > checks with is_readonly check in xattr_permission(). I think that patch should go first in the series. But I really need to find some time to review the whole thing before commenting with profound half-knowledge..