From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io0-f200.google.com (mail-io0-f200.google.com [209.85.223.200]) by kanga.kvack.org (Postfix) with ESMTP id B2A3E6B02F4 for ; Tue, 20 Jun 2017 08:34:42 -0400 (EDT) Received: by mail-io0-f200.google.com with SMTP id i93so93546949iod.4 for ; Tue, 20 Jun 2017 05:34:42 -0700 (PDT) Received: from bombadil.infradead.org (bombadil.infradead.org. [65.50.211.133]) by mx.google.com with ESMTPS id m4si12711236ioe.145.2017.06.20.05.34.41 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 20 Jun 2017 05:34:41 -0700 (PDT) Date: Tue, 20 Jun 2017 05:34:33 -0700 From: Christoph Hellwig Subject: Re: [PATCH v7 11/22] fs: new infrastructure for writeback error handling and reporting Message-ID: <20170620123433.GB19781@infradead.org> References: <20170616193427.13955-1-jlayton@redhat.com> <20170616193427.13955-12-jlayton@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170616193427.13955-12-jlayton@redhat.com> Sender: owner-linux-mm@kvack.org List-ID: To: Jeff Layton Cc: Andrew Morton , Al Viro , Jan Kara , tytso@mit.edu, axboe@kernel.dk, mawilcox@microsoft.com, ross.zwisler@linux.intel.com, corbet@lwn.net, Chris Mason , Josef Bacik , David Sterba , "Darrick J . Wong" , Carlos Maiolino , Eryu Guan , David Howells , Christoph Hellwig , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-ext4@vger.kernel.org, linux-xfs@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-block@vger.kernel.org > @@ -393,6 +394,7 @@ struct address_space { > gfp_t gfp_mask; /* implicit gfp mask for allocations */ > struct list_head private_list; /* ditto */ > void *private_data; /* ditto */ > + errseq_t wb_err; > } __attribute__((aligned(sizeof(long)))); > /* > * On most architectures that alignment is already the case; but > @@ -847,6 +849,7 @@ struct file { > * Must not be taken from IRQ context. > */ > spinlock_t f_lock; > + errseq_t f_wb_err; > atomic_long_t f_count; > unsigned int f_flags; > fmode_t f_mode; Did you check the sizes of the structure before and after? These places don't look like holes in the packing, but there probably are some available. > +static inline int filemap_check_wb_err(struct address_space *mapping, errseq_t since) Overly long line here (the patch has a few more) -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org