> I have got 3 more comments:
>
> 1)
> http://cr.opensolaris.org/~pavelf/6317111/usr/src/uts/i86pc/os/fakebop.c.frames.html
>
> 1104 * then we need to set "fstype" to "nfsdyn" so we mount
>
> the comment should change "nfsdyn" to "nfs"
Yep.
> 2) again, "nfsdyn" should be replaced via "nfs" in
> /usr/sadm/lib/wbem/config_tftp
>
> 59 echo "setprop fstype 'nfsdyn'" >> ${BOOTENVRC}
>
> Since /usr/sadm/lib/wbem/config_tftp is part of closed consolidation
> (not in onnv), I will handle this separately.
Hmm, is that part of the diskless client setup code?
Are there more closed source files that use "nfsdyn" ?
> 3) Fix for 6350006 BAD TRAP panic, when using unknown nfs protocol
> version in fstype property
>
> should address also sparc,
> uts/common/os/swapgeneric.c, function rootconf()
>
> RLOCK_VFSSW();
> vsw = vfs_getvfsswbyname(rootfs.bo_fstype);
> RUNLOCK_VFSSW();
>
> should be changed to
>
> RLOCK_VFSSW();
> vsw = vfs_getvfsswbyname(rootfs.bo_fstype);
> RUNLOCK_VFSSW();
> if (vsw == NULL) {
> cmn_err(CE_CONT, "Cannot find %s filesystem\n", fstyp);
> return (ENXIO);
Minor nit: fstyp isn't defined, we must to use
cmn_err(CE_CONT, "Cannot find %s filesystem\n", rootfs.bo_fstype);
This message posted from opensolaris.org
