Kurt Jaeger wrote:It looks like interesting. But your patch has a issues. + /* fprintf(stderr,"found upper vnode\n"); */ + res = ufs_filestat(&upper, fsp); and + /* fprintf(stderr,"found lower vnode\n"); */ + res = ufs_filestat(&lower, fsp); depend on UFS. It must treat both UFS and no UFS fs. And I am not a maintainer of fstat(1). Please contact to maintainer. Perhaps Ed Maste(emaste) is maintainer I suppose from commit log. The isunionfs.c looks like interesting, too :) But your program is not complete. 'below' option gives it non-correct work. Addition it does not consider unionfs and nullfs combibation or something like that. To check upper/lower completely, you need the same way of your fstat(1) patch. But your idea looks interesting :) Keep your concern of unionfs. To get keep concern is very good for us! -- Daichi GOTO, http://people.freebsd.org/~daichi _______________________________________________ freebsd-current@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-current To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
| Tarkan Erimer | Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3 |
| Greg Kroah-Hartman | [PATCH 004/196] Chinese: add translation of SubmittingPatches |
| Artem Bityutskiy | [PATCH 18/44 take 2] [UBI] build unit implementation |
| James Morris | Re: LSM conversion to static interface |
git: | |
| Paul Jackson | [PATCH] cpuset sched_load_balance kmalloc fix |
| Gerrit Renker | [PATCH 15/37] dccp: Set per-connection CCIDs via socket options |
| Jarek Poplawski | [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Linus Torvalds | Re: [GIT]: Networking |
