Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1206224

Re: [PATCH 29/31] parisc: handle page-less SG entries

Show all headers | View raw

On Wed, Aug 12, 2015 at 12:05 AM, Christoph Hellwig <hch@lst.de> wrote:
> Make all cache invalidation conditional on sg_has_page() and use
> sg_phys to get the physical address directly.

So this worries me a bit (I'm just reacting to one random patch in the series).

The reason?

I think this wants a big honking comment somewhere saying "non-sg_page
accesses are not necessarily cache coherent").

Now, I don't think that's _wrong_, but it's an important distinction:
if you look up pages in the page tables directly, there's a very
subtle difference between then saving just the pfn and saving the
"struct page" of the result.

On sane architectures, this whole cache flushing thing doesn't matter.
Which just means that it's going to be even more subtle on the odd
broken ones..

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.

(And maybe it is, and I just didn't get to that patch yet)

                   Linus
--
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/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 29/31] parisc: handle page-less SG entries Christoph Hellwig <hch@lst.de> - 2015-08-12 09:20 +0200
  Re: [PATCH 29/31] parisc: handle page-less SG entries Linus Torvalds <torvalds@linux-foundation.org> - 2015-08-12 18:10 +0200
    Re: [PATCH 29/31] parisc: handle page-less SG entries Christoph Hellwig <hch@lst.de> - 2015-08-13 16:40 +0200
      Re: [PATCH 29/31] parisc: handle page-less SG entries Dan Williams <dan.j.williams@intel.com> - 2015-08-14 05:40 +0200
        Re: [PATCH 29/31] parisc: handle page-less SG entries James Bottomley <James.Bottomley@HansenPartnership.com> - 2015-08-14 06:00 +0200
          Re: [PATCH 29/31] parisc: handle page-less SG entries David Miller <davem@davemloft.net> - 2015-08-14 06:20 +0200
            Re: [PATCH 29/31] parisc: handle page-less SG entries Dan Williams <dan.j.williams@intel.com> - 2015-08-14 18:20 +0200

csiph-web