Re: [PATCH v3 15/17] mm: pgtable: remove tlb_remove_page_ptdesc()

Hi Mike,

On 2024/12/28 17:26, Mike Rapoport wrote:

On Mon, Dec 23, 2024 at 05:41:01PM +0800, Qi Zheng wrote:

quoted

Here we are explicitly dealing with struct page, and the following logic
semms strange:

tlb_remove_page_ptdesc((tlb), (page_ptdesc(pte)));

tlb_remove_page_ptdesc
--> tlb_remove_page(tlb, ptdesc_page(pt));

So remove tlb_remove_page_ptdesc() and make callers call tlb_remove_page()
directly.

Please don't. The ptdesc wrappers are there as a part of reducing the size
of struct page project [1].

For now struct ptdesc overlaps struct page, but the goal is to have them
separate and always operate on struct ptdesc when working with page tables.

OK, so tlb_remove_page_ptdesc() and tlb_remove_ptdesc() are somewhat
intermediate products of the project.

Hi Andrew, can you help remove [PATCH v3 15/17], [PATCH v3 16/17] and
[PATCH v3 17/17] from the mm-unstable branch?

For [PATCH v3 17/17], I can send it separately later, or Kevin Brodsky
can help do this in his patch series. ;)

Thanks,
Qi

[1] https://kernelnewbies.org/MatthewWilcox/Memdescs