RE: [PATCH] 2.4.9-ac16 swapoff 2*vfree

[Leif Sawyer]

I've applied this patch, and together with 2.4.9-ac16 on
my UltraSparc (1x300mHz, 1024Mb)  seems to be working just fine
so far.

I've got a pretty popular Unreal Tournament Map Mirror site,
so my webserver gets a lot of traffic which should exercise the VM
pretty well, especially as I keep quite a few xterms open.  Plus
a mozilla build every now and then.

But up to now it looks good!  Nice job, all.

> -----Original Message-----
> From: Hugh Dickins [mailto:[e-mail]]
> 
> 2.4.9-ac16 swapoff warns "Trying to vfree() nonexistent vm area":
> the new (outside locks) vfree added, the old (inside) not removed.
> 
> Hugh
> 
> --- 2.4.9-ac16/mm/swapfile.c	Thu Sep 27 19:10:00 2001
> +++ linux/mm/swapfile.c	Thu Sep 27 19:43:12 2001
> @@ -636,7 +636,6 @@
>  	p->swap_device = 0;
>  	p->max = 0;
>  	swap_map = p->swap_map;
> -	vfree(p->swap_map);
>  	p->swap_map = NULL;
>  	p->flags = 0;
>  	swap_device_unlock(p);
>