Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751816Ab3DUGJU (ORCPT ); Sun, 21 Apr 2013 02:09:20 -0400 Received: from mail-wg0-f45.google.com ([74.125.82.45]:44265 "EHLO mail-wg0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751706Ab3DUGJS (ORCPT ); Sun, 21 Apr 2013 02:09:18 -0400 Message-ID: <5173828A.2030809@suse.cz> Date: Sun, 21 Apr 2013 08:09:14 +0200 From: Jiri Slaby User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:19.0) Gecko/20130124 Thunderbird/19.0 MIME-Version: 1.0 To: "Theodore Ts'o" , Ext4 Developers List CC: linux-mm@kvack.org, Linux Kernel Developers List , mgorman@suse.de Subject: Re: [PATCH 3/3] ext4: mark metadata blocks using bh flags References: <20130421000522.GA5054@thunk.org> <1366502828-7793-1-git-send-email-tytso@mit.edu> <1366502828-7793-3-git-send-email-tytso@mit.edu> In-Reply-To: <1366502828-7793-3-git-send-email-tytso@mit.edu> X-Enigmail-Version: 1.6a1pre Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2033 Lines: 62 On 04/21/2013 02:07 AM, Theodore Ts'o wrote: > This allows metadata writebacks which are issued via block device > writeback to be sent with the current write request flags. Hi, where do these come from? fs/ext4/ext4_jbd2.c: In function ‘__ext4_handle_dirty_metadata’: fs/ext4/ext4_jbd2.c:218:2: error: implicit declaration of function ‘mark_buffer_meta’ [-Werror=implicit-function-declaration] fs/ext4/ext4_jbd2.c:219:2: error: implicit declaration of function ‘mark_buffer_prio’ [-Werror=implicit-function-declaration] cc1: some warnings being treated as errors > Signed-off-by: "Theodore Ts'o" > --- > fs/ext4/ext4_jbd2.c | 2 ++ > fs/ext4/inode.c | 6 +++++- > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/fs/ext4/ext4_jbd2.c b/fs/ext4/ext4_jbd2.c > index 0e1dc9e..fd97b81 100644 > --- a/fs/ext4/ext4_jbd2.c > +++ b/fs/ext4/ext4_jbd2.c > @@ -215,6 +215,8 @@ int __ext4_handle_dirty_metadata(const char *where, unsigned int line, > > might_sleep(); > > + mark_buffer_meta(bh); > + mark_buffer_prio(bh); > if (ext4_handle_valid(handle)) { > err = jbd2_journal_dirty_metadata(handle, bh); > if (err) { > diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c > index 62492e9..d7518e2 100644 > --- a/fs/ext4/inode.c > +++ b/fs/ext4/inode.c > @@ -1080,10 +1080,14 @@ retry_journal: > /* For write_end() in data=journal mode */ > static int write_end_fn(handle_t *handle, struct buffer_head *bh) > { > + int ret; > if (!buffer_mapped(bh) || buffer_freed(bh)) > return 0; > set_buffer_uptodate(bh); > - return ext4_handle_dirty_metadata(handle, NULL, bh); > + ret = ext4_handle_dirty_metadata(handle, NULL, bh); > + clear_buffer_meta(bh); > + clear_buffer_prio(bh); > + return ret; > } > > /* > -- js suse labs -- 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/