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



Cy Schubert ha scritto:
In message <43EBA250.1080307@xxxxxxxxxxxx>, 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))



Ok, it compiled. New iso is at:

http://torrent.freesbie.org/FreeSBIE-unionfs-i386-20060210.iso.torrent

It seems better, I can log in and even startx. Further testing is coming :)

Thanks,
Dario

--
Dario Freni (saturnero@xxxxxxxxxxxx)
FreeSBIE developer (http://www.freesbie.org)
GPG Public key at http://www.saturnero.net/saturnero.asc

Attachment: signature.asc
Description: OpenPGP digital signature