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
| ||
|
Date: Tue, 1 Aug 2017 03:54:27 -0700 From: Christoph Hellwig <hch@...radead.org> To: Jan Kara <jack@...e.cz> Cc: linux-fsdevel@...r.kernel.org, linux-ext4@...r.kernel.org, Ross Zwisler <ross.zwisler@...ux.intel.com>, Dan Williams <dan.j.williams@...el.com>, Andy Lutomirski <luto@...nel.org>, linux-nvdimm@...ts.01.org, linux-xfs@...r.kernel.org, Christoph Hellwig <hch@...radead.org>, Dave Chinner <david@...morbit.com> Subject: Re: [PATCH 3/7] dax: Simplify arguments of dax_insert_mapping() On Thu, Jul 27, 2017 at 03:12:41PM +0200, Jan Kara wrote: > dax_insert_mapping() has lots of arguments and a lot of them is actuall > duplicated by passing vm_fault structure as well. Change the function to > take the same arguments as dax_pmd_insert_mapping(). Some of this is probably for historic reasons, when I did the initial pass at iomap based DAX dax_insert_mapping still had to be usable for the non-DAX case. Either way this looks like a great cleanup: Reviewed-by: Christoph Hellwig <hch@....de>
Powered by blists - more mailing lists