[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b040c32a0706111353x7ec86054x8656bbdde02adc47@mail.gmail.com>
Date: Mon, 11 Jun 2007 13:53:44 -0700
From: "Ken Chen" <kenchen@...gle.com>
To: "Adam Litke" <aglitke@...il.com>
Cc: "Eric W. Biederman" <ebiederm@...ssion.com>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Serge E. Hallyn" <serge@...lyn.com>,
"Albert Cahalan" <acahalan@...il.com>,
linux-kernel@...r.kernel.org, linux-mm@...ck.org,
torvalds@...ux-foundation.org,
"Badari Pulavarty" <pbadari@...ibm.com>
Subject: Re: [PATCH] shm: Fix the filename of hugetlb sysv shared memory
On 6/11/07, Adam Litke <aglitke@...il.com> wrote:
> On 6/8/07, Eric W. Biederman <ebiederm@...ssion.com> wrote:
> > -struct file *hugetlb_zero_setup(size_t size)
> > +struct file *hugetlb_file_setup(const char *name, size_t size)
>
> The bulk of this patch seems to handle renaming this function. Is
> that really necessary?
It looks OK to me though, because the argument list to that function
is changed. Avoid change the function name isn't going to reduce the
patch size either. So we might just change the name as well to match
the function name shmem_file_setup().
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists