From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755514AbcFTVIz (ORCPT ); Mon, 20 Jun 2016 17:08:55 -0400 Received: from mail-bl2on0139.outbound.protection.outlook.com ([65.55.169.139]:36742 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754834AbcFTVH6 (ORCPT ); Mon, 20 Jun 2016 17:07:58 -0400 From: "Kani, Toshimitsu" To: "snitzer@redhat.com" CC: "linux-kernel@vger.kernel.org" , "linux-nvdimm@ml01.01.org" , "agk@redhat.com" , "linux-raid@vger.kernel.org" , "viro@zeniv.linux.org.uk" , "dan.j.williams@intel.com" , "axboe@kernel.dk" , "ross.zwisler@linux.intel.com" , "dm-devel@redhat.com" Subject: Re: [PATCH 0/6] Support DAX for device-mapper dm-linear devices Thread-Topic: [PATCH 0/6] Support DAX for device-mapper dm-linear devices Thread-Index: AQHRxcNvFc0gDtWw/UKSuO/qIYbBN5/oAoaAgAqtRoCAAAWsgA== Date: Mon, 20 Jun 2016 18:31:14 +0000 Message-ID: <1466446861.3504.243.camel@hpe.com> References: <1465856497-19698-1-git-send-email-toshi.kani@hpe.com> <20160613225756.GA18417@redhat.com> <20160620180043.GA21261@redhat.com> In-Reply-To: <20160620180043.GA21261@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=toshi.kani@hpe.com; x-originating-ip: [15.219.163.9] x-ms-office365-filtering-correlation-id: ce4613de-f64a-4085-a781-08d399390fab x-microsoft-exchange-diagnostics: 1;CS1PR84MB0005;6:CvMOGEklOpfZj+VW7Er07rNOyzNjt1phmwzIUBSwJ/dNYlXF7ZQxzsXzsacgE6Ofhz3g+3CaU8pduzaOs4WlZme20obPhN6gi7G/LnVCAoLr800vhNhQz8IZyV8s6C/YaBn5M0hiURm+Cnx1JkV0QcJarTzeOJFjdl0aiPpNmfx5nrcno2IfsZUIjtsNPdoWzxZSFcltAJ/VD1IUbgAzQD+hCUeXKcuI55AN4AOA2Rdl893jFszu64LgKj+/v13V6HThXqurywA1AbfdZnOiwAnPrz2W8Jyx0yh8piwvwTY=;5:qHnjbDHPUQfZ1gOqTYAGF+vSIVO21i6EA5+AEuL8yapBTPB4RMZvhCmYaVm+JWdWmFJ8BWeXWDh1UM8izJSyPD87j6qtw9r06kDYoSGRpU2PKpsBWS4lVOqXwwNGzwQXMrunY1zn1AxK+uuK0yp4UA==;24:KpN1qkv8t6jP79EOWWANV1w0qzerwuBX+j5ZguixLwa6TrsKVTZqMJVhfLx+J/qustmiCKZfhMxUu9chMjDPMHn6Nwu6ujvT9UeNO4uJwDA=;7:q+OddhY2KzaYP+afN25sOsvY8XAfsdk65EU83zclPAW81ClEnrMzvNOSLfpTAeodwx7a/os40Tz48ec3u21YaEasKBpVYmFxmznZjKq0Dpm3u/U3DfuKfyCmjVZ8xLvAxhtOsTB7PZqZxw2/WNJn4NnHQkBbkJJbuEEtFDeJtkKfYMFtODnTsB5IZVi+mdsJOWojt6NSL3uT/7ASjeahZ/g8Qi+AXtB2fbLQuRtYiyjcAvx+HOG8jVEIUtE2bbdV x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:CS1PR84MB0005; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(227479698468861)(84791874153150); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:CS1PR84MB0005;BCL:0;PCL:0;RULEID:;SRVR:CS1PR84MB0005; x-forefront-prvs: 09796A1B83 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(7916002)(24454002)(189002)(52044002)(377424004)(199003)(97736004)(19580405001)(2900100001)(19580395003)(11100500001)(110136002)(86362001)(103116003)(10400500002)(92566002)(106356001)(189998001)(7736002)(2950100001)(15975445007)(77096005)(106116001)(66066001)(105586002)(2501003)(5002640100001)(122556002)(8936002)(6116002)(87936001)(2351001)(81166006)(81156014)(76176999)(99286002)(101416001)(50986999)(1730700003)(54356999)(4326007)(3660700001)(8676002)(3846002)(68736007)(3280700002)(7846002)(36756003)(5640700001)(102836003)(33646002)(2906002)(586003);DIR:OUT;SFP:1102;SCL:1;SRVR:CS1PR84MB0005;H:CS1PR84MB0005.NAMPRD84.PROD.OUTLOOK.COM;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="utf-8" Content-ID: MIME-Version: 1.0 X-OriginatorOrg: hpe.com X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Jun 2016 18:31:14.2123 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 105b2061-b669-4b31-92ac-24d304d195dc X-MS-Exchange-Transport-CrossTenantHeadersStamped: CS1PR84MB0005 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u5KL92VR027580 On Mon, 2016-06-20 at 14:00 -0400, Mike Snitzer wrote: > On Mon, Jun 13 2016 at  6:57pm -0400, > Mike Snitzer wrote: > > > > > On Mon, Jun 13 2016 at  6:21pm -0400, > > Toshi Kani wrote: > > > > > > > > This patch-set adds DAX support to device-mapper dm-linear devices > > > used by LVM.  It works with LVM commands as follows: > > >  - Creation of a logical volume with all DAX capable devices (such > > >    as pmem) sets the logical volume DAX capable as well. > > >  - Once a logical volume is set to DAX capable, the volume may not > > >    be extended with non-DAX capable devices. > > > > > > The direct_access interface is added to dm and dm-linear to map > > > a request to a target device. > > > > > >  - Patch 1-2 introduce GENHD_FL_DAX flag to indicate DAX capability. > > >  - Patch 3-4 add direct_access functions to dm and dm-linear. > > >  - Patch 5-6 set GENHD_FL_DAX to dm when all targets are DAX capable. > > > > > > --- > > > Toshi Kani (6): > > >  1/6 genhd: Add GENHD_FL_DAX to gendisk flags > > >  2/6 block: Check GENHD_FL_DAX for DAX capability > > >  3/6 dm: Add dm_blk_direct_access() for mapped device > > >  4/6 dm-linear: Add linear_direct_access() > > >  5/6 dm, dm-linear: Add dax_supported to dm_target > > >  6/6 dm: Enable DAX support for mapper device > > Thanks a lot for doing this.  I recently added it to my TODO so your > > patches come at a great time. > > > > I'll try to get to reviewing/testing your work by the end of this week. > > I rebased your patches on linux-dm.git's 'for-next' (which includes what > I've already staged for the 4.8 merge window).  And I folded/changed > some of the DM patches so that there are only 2 now (1 for DM core and 1 > for dm-linear).  Please see the 4 topmost commits in my 'wip' here: > > http://git.kernel.org/cgit/linux/kernel/git/snitzer/linux.git/log/?h=wip > > Feel free to pick these patches up to use as the basis for continued > work or re-posting of this set.. either that or I could post them as v2 > on your behalf. > > As for testing, I've verified that basic IO works to a pmem-based DM > linear device and that mixed table types are rejected as expected. Great! I will send additional patch, add DAX support to dm-stripe, on top of these once I finish my testing. Thanks, -Toshi