linux-kernel - Re: [RFC DRAFT PATCH] per-buffered-write stream IDs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150422135734.GA28300@quack.suse.cz>
Date:	Wed, 22 Apr 2015 15:57:34 +0200
From:	Jan Kara <jack@...e.cz>
To:	Jens Axboe <axboe@...com>
Cc:	Ming Lin <minggr@...il.com>, Ming Lin <ming.l@...sung.com>,
	lkml <linux-kernel@...r.kernel.org>,
	linux-fsdevel@...r.kernel.org, Dave Chinner <david@...morbit.com>,
	Changho Choi-SSI <changho.c@....samsung.com>,
	"Kwan (Hingkwan) Huen-SSI" <kwan.huen@....samsung.com>
Subject: Re: [RFC DRAFT PATCH] per-buffered-write stream IDs

On Tue 21-04-15 13:05:30, Jens Axboe wrote:
> On 04/21/2015 12:34 PM, Ming Lin wrote:
> >On Tue, Apr 21, 2015 at 10:12 AM, Jens Axboe <axboe@...com> wrote:
> >>On 04/21/2015 11:09 AM, Ming Lin wrote:
> >>>
> >>>Hi Jens,
> >>>
> >>>This RFC DRAFT patch is on top of your "[PATCH v2] Support for write
> >>>stream IDs"
> >>>I throw it out early to get comments if it's the way to go.
> >>>
> >>>Quote LWN(http://lwn.net/Articles/638722):
> >>>
> >>>"There would be clear value in a closer association between stream IDs
> >>>and specific buffered-write operations. Getting there would require
> >>>storing
> >>>the stream ID with each dirtied page, though; that, in turn, almost
> >>>certainly
> >>>implies shoehorning the stream ID into the associated page structure.
> >>>That would not be an easy task; it is not surprising that it is not a part
> >>>of
> >>>this patch set. Should the lack of per-buffered-write stream IDs prove to
> >>>be
> >>>a serious constraint in the future, somebody will certainly be motivated
> >>>to
> >>>try to find a place to store another eight bits in struct page."
> >>>
> >>>This draft patch stores stream_id in buffer head instead of page.
> >>
> >>
> >>This is pointless. You need to store it in the page, if the whole point is
> >>that you want this to be trackable. And adding it to struct page would be a
> >>no-go, we can't increase the size of that. See various other discussions
> >>around, for instance, IO priorities for buffered writeback and tracking that
> >>state on the side.
> >
> >I googled, but didn't find related discussions.
> >Could you please point me a link?
> 
> This is the most recent effort:
> 
> https://lwn.net/Articles/628631/
> 
> My point is that adding it to the buffer_head accomplishes nothing.
> You need to track from when the page was dirtied.
  Technically, you could have a radix tree in an inode that would store
stream id (or other info you need to track) for each dirty page. I'm not
convinced it's worth the complexity and the overhead but it would be doable
and it won't bloat struct page...

								Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/