Hello,

                union {
                        u_int16_t cpg;  /* UFS: FS cylinders per group */
                        u_int16_t sgs;  /* LFS: FS segment shift */
                } __partition_u1;
#define p_cpg   __partition_u1.cpg
#define p_sgs   __partition_u1.sgs
        } d_partitions[MAXPARTITIONS];  /* actually may be more */

I notice that neither LFS itself nor mount_lfs or newfs_lfs use sgs.
(its in the superblock, if I looked right, and newfs_lfs derives it from
other parameters).
Shouldn't we garbage collect this? Or should newfs_lfs actually use it to
set the segment size if not set on the command line?

Regards,
	Ignatios