Re: [PATCH] spufs: remove redundant test on unsigned

2009-06-23 Thread Jeremy Kerr
Roel, > Unsigned `len' cannot be less than 0. > > Signed-off-by: Roel Kluin Thanks for the patch. However, we have a patch from Christoph Hellwig pending that will rework the sputrace code: http://patchwork.ozlabs.org/patch/28641/ It's currently queued in benh's tree, should be up soon. Che

[PATCH] spufs: remove redundant test on unsigned

2009-06-22 Thread Roel Kluin
Unsigned `len' cannot be less than 0. Signed-off-by: Roel Kluin --- Or should it be `if (!buf || len > MAX)' and what should MAX be then? diff --git a/arch/powerpc/platforms/cell/spufs/sputrace.c b/arch/powerpc/platforms/cell/spufs/sputrace.c index d0b1f3f..8f799ee 100644 --- a/arch/powerpc/pla