Re: [PATCH] vmscan.c fixes

[Marcelo Tosatti]

On Mon, 14 May 2001, Rik van Riel wrote:

> Hi Linus,
> 
> the following patch does:

<snip>

>  	pg_data_t *pgdat = pgdat_list;
>  	int sum = 0;
>  	int freeable = nr_free_pages() + nr_inactive_clean_pages();
> +	/* XXX: dynamic free target is complicated and may be wrong... */
>  	int freetarget = freepages.high + inactive_target / 3;

I think its better if we just remove " + inactive_target / 3" here ---
callers already account for the inactive_target when trying to
calculate the free target anyway.

Example: 

static int refill_inactive(unsigned int gfp_mask, int user)
{
        int count, start_count, maxtry;

        count = inactive_shortage() + free_shortage();

...