[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-id: <fb13c936-aa6c-46af-7d35-72548b5baaab@samsung.com>
Date: Fri, 25 Jan 2019 19:50:50 +0530
From: Kanchan Joshi <joshi.k@...sung.com>
To: Jens Axboe <axboe@...nel.dk>
Cc: linux-fsdevel@...r.kernel.org, linux-block@...r.kernel.org,
linux-ext4@...r.kernel.org, linux-nvme@...ts.infradead.org,
jack@...e.com, david@...morbit.com, Jan Kara <jack@...e.cz>,
prakash.v@...sung.com
Subject: Re: [PATCH v2 0/4] Write-hint for FS journal
Hi Jens,
Can you please have a glance on this patch series, given than attempt is
to extend the original architecture of write-hints.
Thanks,
On Thursday 24 January 2019 01:59 PM, Jan Kara wrote:
> Hello,
>
> On Wed 09-01-19 21:00:57, Kanchan Joshi wrote:
>> Towards supporing write-hints/streams for filesystem journal.
>>
>> Here is the v1 patch for background -
>> https://marc.info/?l=linux-fsdevel&m=154444637519020&w=2
>>
>> Changes since v1:
>> - introduce four more hints for in-kernel use, as recommended by Dave chinner
>> & Jens axboe. This isolates kernel-mode hints from user-mode ones.
>> - remove mount-option to specify write-hint, as recommended by Jan kara &
>> Dave chinner. Rather, FS always sets write-hint for journal. This gets ignored
>> if device does not support stream.
>> - Removed code-redundancy for write_dirty_buffer (Jan kara's review comment)
>
> I guess the series should also go to Jens since he was the original author
> of write-hint support so he has the best idea about the architecture.
>
> Honza
>
>> Kanchan Joshi (4):
>> block: Increase count of supported write-hints
>> fs: introduce four macros for in-kernel hints
>> fs: introduce APIs to enable sending write-hint with buffer-head
>> fs/ext4,jbd2: add support for passing write-hint with journal.
>>
>> fs/buffer.c | 18 ++++++++++++++++--
>> fs/ext4/super.c | 2 ++
>> fs/jbd2/commit.c | 11 +++++++----
>> fs/jbd2/journal.c | 3 ++-
>> fs/jbd2/revoke.c | 3 ++-
>> include/linux/blkdev.h | 5 ++++-
>> include/linux/buffer_head.h | 3 +++
>> include/linux/fs.h | 5 +++++
>> include/linux/jbd2.h | 8 ++++++++
>> 9 files changed, 49 insertions(+), 9 deletions(-)
>>
>> --
>> 2.7.4
>>
>>
Powered by blists - more mailing lists