public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* vmalloc fix buggy again?
@ 2004-03-18 18:07 Sergey Vlasov
  2004-03-18 21:11 ` Mark Lord
  0 siblings, 1 reply; 2+ messages in thread
From: Sergey Vlasov @ 2004-03-18 18:07 UTC (permalink / raw)
  To: mlord; +Cc: linux-kernel

[-- Attachment #1: Type: text/plain, Size: 588 bytes --]

Hello!

> # ChangeSet
> #   2004/03/14 13:16:58-03:00 mlord...
> #   [PATCH] Yet another vmalloc() fixup
> # 
> diff -Nru a/mm/vmalloc.c b/mm/vmalloc.c
> --- a/mm/vmalloc.c	Thu Mar 18 09:44:53 2004
> +++ b/mm/vmalloc.c	Thu Mar 18 09:44:53 2004
> @@ -184,7 +184,7 @@
>  	spin_unlock(&init_mm.page_table_lock);
>  	flush_cache_all();
>  	if (address > start)
> -		vmfree_area_pages((address - start), address - start);
> +		vmfree_area_pages(address, address - start);
>  	return -ENOMEM;
>  }
>  

Looks like this should be

		vmfree_area_pages(start, address - start);

-- 
Sergey Vlasov

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: vmalloc fix buggy again?
  2004-03-18 18:07 vmalloc fix buggy again? Sergey Vlasov
@ 2004-03-18 21:11 ` Mark Lord
  0 siblings, 0 replies; 2+ messages in thread
From: Mark Lord @ 2004-03-18 21:11 UTC (permalink / raw)
  To: Sergey Vlasov; +Cc: linux-kernel

Yup.

Thanks, Sergey.  I'm suffering severe brain-impairment on this one.
Please pass this on to Marcelo.

Cheers
-- 
Mark Lord
Real-Time Remedies Inc.
mlord@pobox.com

Sergey Vlasov wrote:
> Hello!
> 
> 
>># ChangeSet
>>#   2004/03/14 13:16:58-03:00 mlord...
>>#   [PATCH] Yet another vmalloc() fixup
>># 
>>diff -Nru a/mm/vmalloc.c b/mm/vmalloc.c
>>--- a/mm/vmalloc.c	Thu Mar 18 09:44:53 2004
>>+++ b/mm/vmalloc.c	Thu Mar 18 09:44:53 2004
>>@@ -184,7 +184,7 @@
>> 	spin_unlock(&init_mm.page_table_lock);
>> 	flush_cache_all();
>> 	if (address > start)
>>-		vmfree_area_pages((address - start), address - start);
>>+		vmfree_area_pages(address, address - start);
>> 	return -ENOMEM;
>> }
>> 
> 
> 
> Looks like this should be
> 
> 		vmfree_area_pages(start, address - start);
> 



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-03-18 21:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-03-18 18:07 vmalloc fix buggy again? Sergey Vlasov
2004-03-18 21:11 ` Mark Lord

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox