From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io0-x22b.google.com (mail-io0-x22b.google.com [IPv6:2607:f8b0:4001:c06::22b]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 759D11A009B for ; Fri, 14 Aug 2015 13:30:52 +1000 (AEST) Received: by iods203 with SMTP id s203so72904506iod.0 for ; Thu, 13 Aug 2015 20:30:50 -0700 (PDT) MIME-Version: 1.0 Sender: dan.j.williams@gmail.com In-Reply-To: <20150813143150.GA17183@lst.de> References: <1439363150-8661-1-git-send-email-hch@lst.de> <1439363150-8661-30-git-send-email-hch@lst.de> <20150813143150.GA17183@lst.de> Date: Thu, 13 Aug 2015 20:30:49 -0700 Message-ID: Subject: Re: [PATCH 29/31] parisc: handle page-less SG entries From: Dan Williams To: Christoph Hellwig Cc: Linus Torvalds , linux-mips , "linux-ia64@vger.kernel.org" , "linux-nvdimm@lists.01.org" , David Howells , sparclinux@vger.kernel.org, Hans-Christian Egtvedt , "linux-arch@vger.kernel.org" , linux-s390 , "the arch/x86 maintainers" , David Woodhouse , =?UTF-8?Q?H=C3=A5vard_Skinnemoen?= , linux-xtensa@linux-xtensa.org, grundler@parisc-linux.org, Miao Steven , Alex Williamson , linux-metag@vger.kernel.org, Jens Axboe , Michal Simek , Parisc List , Vineet Gupta , Linux Kernel Mailing List , linux-alpha@vger.kernel.org, Linux Media Mailing List , ppc-dev Content-Type: text/plain; charset=UTF-8 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Aug 13, 2015 at 7:31 AM, Christoph Hellwig wrote: > On Wed, Aug 12, 2015 at 09:01:02AM -0700, Linus Torvalds wrote: >> I'm assuming that anybody who wants to use the page-less >> scatter-gather lists always does so on memory that isn't actually >> virtually mapped at all, or only does so on sane architectures that >> are cache coherent at a physical level, but I'd like that assumption >> *documented* somewhere. > > It's temporarily mapped by kmap-like helpers. That code isn't in > this series. The most recent version of it is here: > > https://git.kernel.org/cgit/linux/kernel/git/djbw/nvdimm.git/commit/?h=pfn&id=de8237c99fdb4352be2193f3a7610e902b9bb2f0 > > note that it's not doing the cache flushing it would have to do yet, but > it's also only enabled for x86 at the moment. For virtually tagged caches I assume we would temporarily map with kmap_atomic_pfn_t(), similar to how drm_clflush_pages() implements powerpc support. However with DAX we could end up with multiple virtual aliases for a page-less pfn.