On 2023/9/25 18:37, Ilias Apalodimas wrote:

Hi

On Wed, 20 Sept 2023 at 11:59, Yunsheng Lin [off-list ref] wrote:

quoted

On 2023/9/15 16:28, Jesper Dangaard Brouer wrote:

quoted

Hi Lin,

This looks reasonable, but given you are changing struct-page
(include/linux/mm_types.h) we need to MM-list [off-list ref].
Also Cc Wilcox.

I think it was Ilias and Duyck that validated the assumptions, last time
this patch was discussed. Thus I want to see their review before this is
applied.

FWIW, PAGE_SIZE aligned buffer being PAGE_SIZE aligned in DMA is
validated by Duyck:
https://lore.kernel.org/all/CAKgT0UfeUAUQpEffxnkc+gzXsjOrHkuMgxU_Aw0VXSJYKzaovQ@mail.gmail.com/ (local)

And I had done researching to find out there seems to be no combination of
the above arch with an address space >16TB:
https://lore.kernel.org/all/2b570282-24f8-f23b-1ff7-ad836794baa9@huawei.com/ (local)

Apologies for the late reply.  I just saw you sent a v9, I'll review
that instead, but I am traveling right now, will take a while

Actually there is a newer v10, see:
https://lore.kernel.org/all/20230922091138.18014-2-linyunsheng@huawei.com/ (local)

Thanks for the time reviewing.

Thanks
/Ilias