Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751877AbbEGTsl (ORCPT ); Thu, 7 May 2015 15:48:41 -0400 Received: from mail-wi0-f170.google.com ([209.85.212.170]:36659 "EHLO mail-wi0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751072AbbEGTsi (ORCPT ); Thu, 7 May 2015 15:48:38 -0400 Date: Thu, 7 May 2015 21:48:32 +0200 From: Ingo Molnar To: Jerome Glisse Cc: Dave Hansen , Dan Williams , Linus Torvalds , Linux Kernel Mailing List , Boaz Harrosh , Jan Kara , Mike Snitzer , Neil Brown , Benjamin Herrenschmidt , Heiko Carstens , Chris Mason , Paul Mackerras , "H. Peter Anvin" , Christoph Hellwig , Alasdair Kergon , "linux-nvdimm@lists.01.org" , Mel Gorman , Matthew Wilcox , Ross Zwisler , Rik van Riel , Martin Schwidefsky , Jens Axboe , "Theodore Ts'o" , "Martin K. Petersen" , Julia Lawall , Tejun Heo , linux-fsdevel , Andrew Morton , paulmck@linux.vnet.ibm.com Subject: Re: [PATCH v2 00/10] evacuate struct page from the block layer, introduce __pfn_t Message-ID: <20150507194832.GB23511@gmail.com> References: <20150507173641.GA21781@gmail.com> <554BA748.9030804@linux.intel.com> <20150507191107.GB22952@gmail.com> <20150507193635.GC5966@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150507193635.GC5966@gmail.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2243 Lines: 53 * Jerome Glisse wrote: > > So I think the main value of struct page is if everyone on the > > system sees the same struct page for the same pfn - not just the > > temporary IO instance. > > > > The idea of having very temporary struct page arrays misses the > > point I think: if struct page is used as essentially an IO sglist > > then most of the synchronization properties are lost: then we > > might as well use the real deal in that case and skip the dynamic > > allocation and use pfns directly and avoid the dynamic allocation > > overhead. > > > > Stable, global page-struct descriptors are a given for real RAM, > > where we allocate a struct page for every page in nice, large, > > mostly linear arrays. > > > > We'd really need that for pmem too, to get the full power of > > struct page: and that means allocating them in nice, large, > > predictable places - such as on the device itself ... > > Is handling kernel pagefault on the vmemmap completely out of the > picture ? So we would carveout a chunck of kernel address space for > those pfn and use it for vmemmap and handle pagefault on it. That's pretty clever. The page fault doesn't even have to do remote TLB shootdown, because it only establishes mappings - so it's pretty atomic, a bit like the minor vmalloc() area faults we are doing. Some sort of LRA (least recently allocated) scheme could unmap the area in chunks if it's beyond a certain size, to keep a limit on size. Done from the same context and would use remote TLB shootdown. The only limitation I can see is that such faults would have to be able to sleep, to do the allocation. So pfn_to_page() could not be used in arbitrary contexts. > Again here i think that GPU folks would like a solution where they > can have a page struct but it would not be PMEM just device memory. > So if we can come up with something generic enough to server both > purpose that would be better in my view. Yes. Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/