Re: [PATCH] mm/vmscan: remove prefetch_prev_lru_page

[Qian Cai]

> On Jan 14, 2020, at 9:33 PM, Alex Shi <alex.shi@xxxxxxxxxxxxxxxxx> wrote:
> 
> ï
> 
>> å 2020/1/14 äå9:46, Qian Cai åé:
>> 
>> 
>>>> On Jan 14, 2020, at 7:55 AM, Alex Shi <alex.shi@xxxxxxxxxxxxxxxxx> wrote:
>>> 
>>> This macro are never used in git history. So better to remove.
>> 
>> When removing unused thingy, it is important to figure out which commit introduced it in the first place and Cc the relevant people in that commit.
>> 
> 
> Thanks fore reminder, Qian!
> 
> This macro was introduced in 1da177e4c3f4 Linux-2.6.12-rc2, no author or commiter could be found.

Looks a bit deeper for this, and I am not sure if it is necessary to remove it especially this does not cause any complication warning noise, because the macro looks like a part of API design to have a pair of both read and write version, even though only the write version is used at the moment.

In theory,  there could be users for the read version in the future, and then it needs to be added back.