From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from quartz.orcorp.ca (quartz.orcorp.ca [184.70.90.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 5BC5F20347141 for ; Wed, 18 Oct 2017 22:59:41 -0700 (PDT) Date: Thu, 19 Oct 2017 00:02:49 -0600 From: Jason Gunthorpe Subject: Re: [PATCH v9 0/6] MAP_DIRECT for DAX userspace flush Message-ID: <20171019060249.GA6555@obsidianresearch.com> References: <20171012142319.GA11254@lst.de> <20171013065716.GB26461@lst.de> <20171013163822.GA17411@obsidianresearch.com> <20171013173145.GA18702@obsidianresearch.com> <20171014015752.GA25172@obsidianresearch.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: 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: Sagi Grimberg Cc: linux-xfs@vger.kernel.org, Jan Kara , Arnd Bergmann , "Darrick J. Wong" , Linux API , "linux-nvdimm@lists.01.org" , Dave Chinner , Christoph Hellwig , "J. Bruce Fields" , Linux MM , Jeff Layton , Al Viro , Andy Lutomirski , linux-fsdevel , Linus Torvalds , Andrew Morton List-ID: On Mon, Oct 16, 2017 at 03:02:52PM +0300, Sagi Grimberg wrote: > But why should the kernel ever need to mangle the CQ? if a lease break > would deregister the MR the device is expected to generate remote > protection errors on its own. The point is to avoid protection errors - hittles change over when the DAX mapping changes like ODP does. Theonly way to get there is to notify the app before the mappings change.. Dan suggested having ibv_pollcq return this indication.. Jason _______________________________________________ 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 Return-Path: Date: Thu, 19 Oct 2017 00:02:49 -0600 From: Jason Gunthorpe To: Sagi Grimberg Cc: Dan Williams , "J. Bruce Fields" , Jan Kara , Andrew Morton , Arnd Bergmann , "Darrick J. Wong" , Linux API , "linux-nvdimm@lists.01.org" , Dave Chinner , linux-xfs@vger.kernel.org, Linux MM , Al Viro , Andy Lutomirski , Jeff Layton , linux-fsdevel , Linus Torvalds , Christoph Hellwig Subject: Re: [PATCH v9 0/6] MAP_DIRECT for DAX userspace flush Message-ID: <20171019060249.GA6555@obsidianresearch.com> References: <20171012142319.GA11254@lst.de> <20171013065716.GB26461@lst.de> <20171013163822.GA17411@obsidianresearch.com> <20171013173145.GA18702@obsidianresearch.com> <20171014015752.GA25172@obsidianresearch.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: On Mon, Oct 16, 2017 at 03:02:52PM +0300, Sagi Grimberg wrote: > But why should the kernel ever need to mangle the CQ? if a lease break > would deregister the MR the device is expected to generate remote > protection errors on its own. The point is to avoid protection errors - hittles change over when the DAX mapping changes like ODP does. Theonly way to get there is to notify the app before the mappings change.. Dan suggested having ibv_pollcq return this indication.. Jason -- 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: Received: from quartz.orcorp.ca ([184.70.90.242]:50118 "EHLO quartz.orcorp.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751468AbdJSGD1 (ORCPT ); Thu, 19 Oct 2017 02:03:27 -0400 Date: Thu, 19 Oct 2017 00:02:49 -0600 From: Jason Gunthorpe Subject: Re: [PATCH v9 0/6] MAP_DIRECT for DAX userspace flush Message-ID: <20171019060249.GA6555@obsidianresearch.com> References: <20171012142319.GA11254@lst.de> <20171013065716.GB26461@lst.de> <20171013163822.GA17411@obsidianresearch.com> <20171013173145.GA18702@obsidianresearch.com> <20171014015752.GA25172@obsidianresearch.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Sagi Grimberg Cc: Dan Williams , "J. Bruce Fields" , Jan Kara , Andrew Morton , Arnd Bergmann , "Darrick J. Wong" , Linux API , "linux-nvdimm@lists.01.org" , Dave Chinner , linux-xfs@vger.kernel.org, Linux MM , Al Viro , Andy Lutomirski , Jeff Layton , linux-fsdevel , Linus Torvalds , Christoph Hellwig On Mon, Oct 16, 2017 at 03:02:52PM +0300, Sagi Grimberg wrote: > But why should the kernel ever need to mangle the CQ? if a lease break > would deregister the MR the device is expected to generate remote > protection errors on its own. The point is to avoid protection errors - hittles change over when the DAX mapping changes like ODP does. Theonly way to get there is to notify the app before the mappings change.. Dan suggested having ibv_pollcq return this indication.. Jason From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH v9 0/6] MAP_DIRECT for DAX userspace flush Date: Thu, 19 Oct 2017 00:02:49 -0600 Message-ID: <20171019060249.GA6555@obsidianresearch.com> References: <20171012142319.GA11254@lst.de> <20171013065716.GB26461@lst.de> <20171013163822.GA17411@obsidianresearch.com> <20171013173145.GA18702@obsidianresearch.com> <20171014015752.GA25172@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sagi Grimberg Cc: Dan Williams , "J. Bruce Fields" , Jan Kara , Andrew Morton , Arnd Bergmann , "Darrick J. Wong" , Linux API , "linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org" , Dave Chinner , linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Linux MM , Al Viro , Andy Lutomirski , Jeff Layton , linux-fsdevel , Linus Torvalds , Christoph Hellwig List-Id: linux-api@vger.kernel.org On Mon, Oct 16, 2017 at 03:02:52PM +0300, Sagi Grimberg wrote: > But why should the kernel ever need to mangle the CQ? if a lease break > would deregister the MR the device is expected to generate remote > protection errors on its own. The point is to avoid protection errors - hittles change over when the DAX mapping changes like ODP does. Theonly way to get there is to notify the app before the mappings change.. Dan suggested having ibv_pollcq return this indication.. Jason