[PATCH] include/linux/sunrpc/svc.h compilation error

[Grzegorz Jaskiewicz <gj@pointblue.com.pl>]

On Tue, 2003-05-06 at 11:08, Paweł Gołaszewski wrote:
> That kernel fails for me when building...
> [cut]
> 
> gcc -Wp,-MD,fs/lockd/.clntproc.o.d -D__KERNEL__ -Iinclude -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common -pipe -mpreferred-stack-boundary=2 -march=i686 -malign-functions=4 -Iinclude/asm-i386/mach-default -nostdinc -iwithprefix include -DMODULE -DKBUILD_BASENAME=clntproc -DKBUILD_MODNAME=lockd -c -o fs/lockd/.tmp_clntproc.o fs/lockd/clntproc.c
> In file included from fs/lockd/clntproc.c:17:
> include/linux/sunrpc/svc.h: In function `svc_take_page': 
> include/linux/sunrpc/svc.h:180: invalid lvalue in assignment
> make[3]: *** [fs/lockd/clntproc.o] Error 1
> make[2]: *** [fs/lockd] Error 2
> make[1]: *** [fs] Error 2
> make: *** [modules] Error 2

Looks like gcc fault, can You Pawel give as gcc version  ?

this patch as reported by Pawel helps:


--- ./include/linux/sunrpc/svc.h.buildfix       Mon May  6 12:45:11 2003
+++ ./include/linux/sunrpc/svc.h        Tue May  6 12:42:13 2003
@@ -176,8 +176,14 @@
 {
        if (rqstp->rq_arghi <= rqstp->rq_argused)
                return -ENOMEM;
+
+       rqstp->rq_arghi--;
+
        rqstp->rq_respages[rqstp->rq_resused++] =
                rqstp->rq_argpages[--rqstp->rq_arghi];
+
+       rqstp->rq_resused++;
+
        return 0;
 }


but still, it looks strange - i am sure it is just an gcc issue

-- 
Grzegorz Jaskiewicz <gj@pointblue.com.pl>
K4 labs