2007-05-09 03:28:25

by David Gibson

Subject: Remove an unused variable from get_unmapped_area()

Some recent changes in get_unmapped_area() have left the 'ret'
local unused. This patch removes it.

Signed-off-by: David Gibson <[email protected]>

Index: working-2.6/mm/mmap.c
===================================================================
--- working-2.6.orig/mm/mmap.c 2007-05-09 13:26:33.000000000 +1000
+++ working-2.6/mm/mmap.c 2007-05-09 13:26:47.000000000 +1000
@@ -1366,7 +1366,6 @@ unsigned long
get_unmapped_area(struct file *file, unsigned long addr, unsigned long len,
unsigned long pgoff, unsigned long flags)
{
- unsigned long ret;
unsigned long (*get_area)(struct file *, unsigned long,
unsigned long, unsigned long, unsigned long);


--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson

2007-05-09 03:52:30

by Benjamin Herrenschmidt

Subject: Re: Remove an unused variable from get_unmapped_area()

On Wed, 2007-05-09 at 13:28 +1000, David Gibson wrote:
> Some recent changes in get_unmapped_area() have left the 'ret'
> local unused. This patch removes it.
>
> Signed-off-by: David Gibson <[email protected]>

Yeah, my fault, though somebody already posted a patch for this

Ben.