Re: patchset-8 release (Re: [unionfs][patch] improvements of the unionfs - Problem Report, kern/91010)

In message <43EBA250.1080307_at_freesbie.org>, Dario Freni writes:
> Daichi GOTO ha scritto:
> > I have updated the patchset-8 of unionfs.
> > 
> > Patchset-8:
> >    For 7-current
> >      http://people.freebsd.org/~daichi/unionfs/unionfs-p8.diff
> > 
> >    For 6.x
> >      http://people.freebsd.org/~daichi/unionfs/unionfs6-p8.diff
> > 
> >    Changes in unionfs-p8.diff
> >      - Fixed the issue that user whom has access permission
> >        cannot change the directory because he cannot create
> >        its shadow directory. As a result of this fixed, now
> >        unionfs uses root permission creating shadow directory
> >        temporarily.
> > 
> > The document of those unionfs patches is pretty improved by Hiroo
> > ONO-san.
> > 
> >   http://people.freebsd.org/~daichi/unionfs/  (English)
> >   http://people.freebsd.org/~daichi/unionfs/index-ja.html  (Japanese)
> > 
> > Please try -p8 Dario. We are thinking that you cat get it with -p8 :)
> 
> It doesn't compile on 6.x :/ (using unionfs6-p8.diff on a fresh RELENG_6)
> 
> cc -c -O2 -pipe -fno-strict-aliasing  -Wall -Wredundant-decls 
> -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes 
> -Wpointer-arith -Winline -Wcast-qual  -fformat-extensions -std=c99 
> -nostdinc -I-  -I. -I/usr/src/sys -I/usr/src/sys/contrib/altq 
> -I/usr/src/sys/contrib/ipfilter -I/usr/src/sys/contrib/pf 
> -I/usr/src/sys/contrib/dev/ath -I/usr/src/sys/contrib/dev/ath/freebsd 
> -I/usr/src/sys/contrib/ngatm -I/usr/src/sys/dev/twa -D_KERNEL 
> -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common 
> -finline-limit=8000 --param inline-unit-growth=100 --param 
> large-function-growth=1000  -mno-align-long-strings 
> -mpreferred-stack-boundary=2  -mno-mmx -mno-3dnow -mno-sse -mno-sse2 
> -ffreestanding -Werror  /usr/src/sys/ufs/ufs/ufs_lookup.c
> /usr/src/sys/ufs/ufs/ufs_lookup.c: In function `ufs_direnter':
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: `vdp' undeclared (first 
> use in this function)
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: (Each undeclared 
> identifier is reported only once
> /usr/src/sys/ufs/ufs/ufs_lookup.c:880: error: for each function it 
> appears in.)
> *** Error code 1
> 
> Stop in /usr/obj.unionfs-i386/usr/src/sys/FREESBIE.
> *** Error code 1
> 
> Stop in /usr/src.
> *** Error code 1
> 
> Stop in /usr/src.

Replace the line in the patch that says:

+               if (OFSFMT(vdp))

with:

+               if (OFSFMT(dvp))



Cheers,
Cy Schubert <Cy.Schubert_at_komquats.com>
Web:  http://www.komquats.com and http://www.bcbodybuilder.com
FreeBSD UNIX:  <cy_at_FreeBSD.org>   Web:  http://www.FreeBSD.org
BC Government:  <Cy.Schubert_at_gov.bc.ca>

    "Lift long enough and I believe arrogance is replaced by
    humility and fear by courage and selfishness by generosity
    and rudeness by compassion and caring."
        -- Dave Draper