On Tue, May 08, 2012 at 06:27:22PM +0200, Sam Ravnborg wrote:
> On Tue, May 08, 2012 at 06:40:19PM +0400, Vladimir Murzin wrote:
> > Move check for TASK_SIZE before MAP_FIXED in sparc32's arch_get_unmapped_area().
>
> I would be good to copy relevant parts of your cover letter into each patch.
> when the patches are applied we looses the cover letter...
>
In my opinion patches are quite trivial and self-explained. However,
if there are requests to update patches in this way I'll do it in v2.
>
> >
> > Signed-off-by: Vladimir Murzin <murzin.v@xxxxxxxxx>
> > ---
> > arch/sparc/kernel/sys_sparc_32.c | 9 +++++----
> > 1 files changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/sparc/kernel/sys_sparc_32.c b/arch/sparc/kernel/sys_sparc_32.c
> > index 42b282f..7bf30de 100644
> > --- a/arch/sparc/kernel/sys_sparc_32.c
> > +++ b/arch/sparc/kernel/sys_sparc_32.c
> > @@ -40,6 +40,11 @@ unsigned long arch_get_unmapped_area(struct file *filp, unsigned long addr, unsi
> > {
> > struct vm_area_struct * vmm;
> >
> > + if (len > TASK_SIZE - PAGE_SIZE)
> > + return -ENOMEM;
> This is not the same as the check it replaces - what is going on here?
>
It shouldn't be the same. The a_g_u_a serves to help arches customize
get_unmapped_are depends on their specific like cache aliasing,
etc. Checking for length (in this case) could be also customized.
> Sam
Best wishes
Vladimir Murzin
--
To unsubscribe from this list: send the line "unsubscribe linux-arch" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
