Re: [PATCH 0/8] Remove unused parameters in page_table_check

[Pasha Tatashin]

On Thu, Jul 13, 2023 at 5:25 AM Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx> wrote:
>
> Hi all, this series remove unused parameters in functions from
> page_table_check. The first 2 patches remove unused mm and addr
> parameters in static common functions page_table_check_clear and
> page_table_check_set. The last 6 patches remove unused addr parameter
> in some externed functions which only need addr for cleaned
> page_table_check_clear or page_table_check_set. There is no intended
> functional change. Thanks!

NAK

Both, mm and addr are common arguments that are used for PTE handling
in many parts of memory management even when they are not used in
every function.

Currently, they are not used in page table check, but it is possible
we may need to use them in the future when support for other arches or
different types of page tables (i.e. extended page table) is added. It
is going to be hard to again modify all arch dependent code to add
these arguments back.

Also, internally at Google we are using these arguments, as we have a
module that maps user memory in a way that is incompatible with
upstream, and these arguments are used to support this module.

Thank you,
Pasha