From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io0-x242.google.com (mail-io0-x242.google.com [IPv6:2607:f8b0:4001:c06::242]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E60412034A88F for ; Fri, 27 Oct 2017 04:38:30 -0700 (PDT) Received: by mail-io0-x242.google.com with SMTP id 97so12069301iok.7 for ; Fri, 27 Oct 2017 04:42:17 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20171027064854.GE3666@dastard> References: <150846713528.24336.4459262264611579791.stgit@dwillia2-desk3.amr.corp.intel.com> <20171020074750.GA13568@lst.de> <20171020093148.GA20304@lst.de> <20171026105850.GA31161@quack2.suse.cz> <1509061831.25213.2.camel@intel.com> <20171027064854.GE3666@dastard> From: Dan Williams Date: Fri, 27 Oct 2017 13:42:16 +0200 Message-ID: Subject: Re: [PATCH v3 00/13] dax: fix dma vs truncate and remove 'page-less' support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: Dave Chinner Cc: "mhocko@suse.com" , "jack@suse.cz" , "benh@kernel.crashing.org" , "dave.hansen@linux.intel.com" , "heiko.carstens@de.ibm.com" , "bfields@fieldses.org" , "linux-mm@kvack.org" , "paulus@samba.org" , "Hefty, Sean" , "hch@lst.de" , "mawilcox@microsoft.com" , "linux-rdma@vger.kernel.org" , "mpe@ellerman.id.au" , "jgunthorpe@obsidianresearch.com" , "dledford@redhat.com" , "hal.rosenstock@gmail.com" , "linux-nvdimm@lists.01.org" , "viro@zeniv.linux.org.uk" , "jlayton@poochiereds.net" , "gerald.schaefer@de.ibm.com" , "darrick.wong@oracle.com" , "linux-kernel@vger.kernel.org" , "linux-xfs@vger.kernel.org" , "schwidefsky@de.ibm.com" , "linux-fsdevel@vger.kernel.org" , "akpm@linux-foundation.org" , "kirill.shutemov@linux.intel.com" List-ID: [replying from my phone, please forgive formatting] On Friday, October 27, 2017, Dave Chinner wrote: > > Here are the two primary patches in > > the series, do you think the extent-busy approach would be cleaner? > > The XFS_DAXDMA.... > > $DEITY that patch is so ugly I can't even bring myself to type it. Right, and so is the problem it's trying to solve. So where do you want to go from here? I could go back to the FL_ALLOCATED approach, but use page idle callbacks instead of polling for the lease end notification. Or do we want to try busy extents? My concern with busy extents is that it requires more per-fs code. _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Williams Subject: Re: [PATCH v3 00/13] dax: fix dma vs truncate and remove 'page-less' support Date: Fri, 27 Oct 2017 13:42:16 +0200 Message-ID: References: <150846713528.24336.4459262264611579791.stgit@dwillia2-desk3.amr.corp.intel.com> <20171020074750.GA13568@lst.de> <20171020093148.GA20304@lst.de> <20171026105850.GA31161@quack2.suse.cz> <1509061831.25213.2.camel@intel.com> <20171027064854.GE3666@dastard> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="001a1144a50cdc7c42055c85c780" Return-path: In-Reply-To: <20171027064854.GE3666@dastard> Sender: owner-linux-mm@kvack.org To: Dave Chinner Cc: "mhocko@suse.com" , "jack@suse.cz" , "benh@kernel.crashing.org" , "dave.hansen@linux.intel.com" , "heiko.carstens@de.ibm.com" , "bfields@fieldses.org" , "linux-mm@kvack.org" , "paulus@samba.org" , "Hefty, Sean" , "jlayton@poochiereds.net" , "mawilcox@microsoft.com" , "linux-rdma@vger.kernel.org" , "mpe@ellerman.id.au" , "dledford@redhat.com" , "hch@lst.de" , "jgunthorpe@obsidianresearch.com" , "hal.rosenstock@gmail.com" , "schwidefsky@de.ibm.com" , viro@zeniv List-Id: linux-rdma@vger.kernel.org --001a1144a50cdc7c42055c85c780 Content-Type: text/plain; charset="UTF-8" [replying from my phone, please forgive formatting] On Friday, October 27, 2017, Dave Chinner wrote: > > Here are the two primary patches in > > the series, do you think the extent-busy approach would be cleaner? > > The XFS_DAXDMA.... > > $DEITY that patch is so ugly I can't even bring myself to type it. Right, and so is the problem it's trying to solve. So where do you want to go from here? I could go back to the FL_ALLOCATED approach, but use page idle callbacks instead of polling for the lease end notification. Or do we want to try busy extents? My concern with busy extents is that it requires more per-fs code. --001a1144a50cdc7c42055c85c780 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

[replying from my phone, please forgive formatting]

O= n Friday, October 27, 2017, Dave Chinner <david@fromorbit.com> wrote:
=C2=A0
> Here are the two primary patches in
> the series, do you think the extent-busy approach would be cleaner?
The XFS_DAXDMA....

$DEITY that patch is so ugly I can't even bring myself to type it.

Right, and so is the problem it's trying to= solve. So where do you want to go from here?

I co= uld go back to the FL_ALLOCATED approach, but use page idle callbacks inste= ad of polling for the lease end notification. Or do we want to try busy ext= ents? My concern with busy extents is that it requires more per-fs code. --001a1144a50cdc7c42055c85c780-- -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-Version: 1.0 In-Reply-To: <20171027064854.GE3666@dastard> References: <150846713528.24336.4459262264611579791.stgit@dwillia2-desk3.amr.corp.intel.com> <20171020074750.GA13568@lst.de> <20171020093148.GA20304@lst.de> <20171026105850.GA31161@quack2.suse.cz> <1509061831.25213.2.camel@intel.com> <20171027064854.GE3666@dastard> From: Dan Williams Date: Fri, 27 Oct 2017 13:42:16 +0200 Message-ID: Subject: Re: [PATCH v3 00/13] dax: fix dma vs truncate and remove 'page-less' support To: Dave Chinner Cc: "mhocko@suse.com" , "jack@suse.cz" , "benh@kernel.crashing.org" , "dave.hansen@linux.intel.com" , "heiko.carstens@de.ibm.com" , "bfields@fieldses.org" , "linux-mm@kvack.org" , "paulus@samba.org" , "Hefty, Sean" , "jlayton@poochiereds.net" , "mawilcox@microsoft.com" , "linux-rdma@vger.kernel.org" , "mpe@ellerman.id.au" , "dledford@redhat.com" , "hch@lst.de" , "jgunthorpe@obsidianresearch.com" , "hal.rosenstock@gmail.com" , "schwidefsky@de.ibm.com" , "viro@zeniv.linux.org.uk" , "gerald.schaefer@de.ibm.com" , "linux-nvdimm@lists.01.org" , "darrick.wong@oracle.com" , "linux-kernel@vger.kernel.org" , "linux-xfs@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" , "akpm@linux-foundation.org" , "kirill.shutemov@linux.intel.com" Content-Type: multipart/alternative; boundary="001a1144a50cdc7c42055c85c780" Sender: owner-linux-mm@kvack.org List-ID: --001a1144a50cdc7c42055c85c780 Content-Type: text/plain; charset="UTF-8" [replying from my phone, please forgive formatting] On Friday, October 27, 2017, Dave Chinner wrote: > > Here are the two primary patches in > > the series, do you think the extent-busy approach would be cleaner? > > The XFS_DAXDMA.... > > $DEITY that patch is so ugly I can't even bring myself to type it. Right, and so is the problem it's trying to solve. So where do you want to go from here? I could go back to the FL_ALLOCATED approach, but use page idle callbacks instead of polling for the lease end notification. Or do we want to try busy extents? My concern with busy extents is that it requires more per-fs code. --001a1144a50cdc7c42055c85c780 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

[replying from my phone, please forgive formatting]

O= n Friday, October 27, 2017, Dave Chinner <david@fromorbit.com> wrote:
=C2=A0
> Here are the two primary patches in
> the series, do you think the extent-busy approach would be cleaner?
The XFS_DAXDMA....

$DEITY that patch is so ugly I can't even bring myself to type it.

Right, and so is the problem it's trying to= solve. So where do you want to go from here?

I co= uld go back to the FL_ALLOCATED approach, but use page idle callbacks inste= ad of polling for the lease end notification. Or do we want to try busy ext= ents? My concern with busy extents is that it requires more per-fs code. --001a1144a50cdc7c42055c85c780-- -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org