linux-kernel - Re: [PATCH v3 1/4] fs: kiocb: introduce IOCB_DONT_DIRTY

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]

Date:	Mon, 11 May 2015 21:02:54 +0800
From:	Ming Lei <ming.lei@...onical.com>
To:	Christoph Hellwig <hch@...radead.org>
Cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Dave Kleikamp <dave.kleikamp@...cle.com>,
	Jens Axboe <axboe@...nel.dk>, Zach Brown <zab@...bo.net>,
	Maxim Patlasov <mpatlasov@...allels.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Alexander Viro <viro@...iv.linux.org.uk>,
	Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH v3 1/4] fs: kiocb: introduce IOCB_DONT_DIRTY_PAGE flag for
 direct IO

On Mon, May 11, 2015 at 3:19 PM, Christoph Hellwig <hch@...radead.org> wrote:
> On Thu, May 07, 2015 at 08:12:31PM +0800, Ming Lei wrote:
>> In case of loop, it is quite specific about dirtying READ pages in
>> direct IO because fs over loop has considered dirtying these pages
>> already.
>>
>> For other cases of ITER_BVEC or ITER_KVEC, if the page is
>> anonymous or mapped, dirtying is still needed, otherwise
>> the new written data may be dropped like direct-io from user space.
>
> But those would be user pages.  Anyone who passes a kernel page doesn't

Inside kernel it is hard to say one page in page cache is kernel or
user page, :-)

> expect it do be dirtied for normal kernel interfaces.

Currently the coming direct IO introduced for loop should be the 1st direct
read inside kernel, so there isn't the case of passing kernel pages, and
obviously ITER_BVEC/ITER_KVEC doesn't mean the page is 'kernel' page.

Another direct IO inside kernel is __swap_writepage(), and it is still
ITER_BVEC and user page. Maybe in the future swap_readpage()
can use direct IO with BVEC too.

It depends if all kernel direct read I/O to 'user' page(anonymous, mapped,
page cache) via BVEC/KVEC do not need to dirty the page. At least for loop,
I understand the reason is specific, that is why I suggest to use the flag now,
at least it has document benefit because the reason isn't obvious.

Thanks,
Ming Lei
--
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/