Re: kern/39302: can't compile non-WAPBL kernel

[Juergen Hannken-Illjes <hannken%eis.cs.tu-bs.de@localhost>]

The following reply was made to PR kern/39302; it has been noted by GNATS.

From: Juergen Hannken-Illjes <hannken%eis.cs.tu-bs.de@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: kern/39302: can't compile non-WAPBL kernel
Date: Wed, 6 Aug 2008 19:25:28 +0200

 On Wed, Aug 06, 2008 at 05:00:01PM +0000, martin%duskware.de@localhost wrote:
 > >Number:         39302
 > >Category:       kern
 > >Synopsis:       can't compile non-WAPBL kernel
 [snip]
 > Trying to build a kernel for a diskless configuration fails:
 > 
 > cc1: warnings being treated as errors
 > ../../../../ufs/ffs/ffs_alloc.c: In function 'ffs_freefile':
 > ../../../../ufs/ffs/ffs_alloc.c:2158: warning: empty body in an if-statement
 
 Does this diff fix the problem:
 
 --- sys/ufs/ffs/ffs_alloc.c    2008-08-06 14:54:26.000000000 +0200
 +++ sys/ufs/ffs/ffs_alloc.c.new        2008-08-06 19:23:07.000000000 +0200
 @@ -2155,7 +2155,8 @@ ffs_freefile(struct fs *fs, struct vnode
        }
        clrbit(inosused, ino);
 -      if (!devvp_is_snapshot)
 +      if (!devvp_is_snapshot) {
                UFS_WAPBL_UNREGISTER_INODE(devvp->v_specmountpoint,
                    ino + cg * fs->fs_ipg, mode);
 +      }
        if (ino < ufs_rw32(cgp->cg_irotor, needswap))
                cgp->cg_irotor = ufs_rw32(ino, needswap);
 
 -- 
 Juergen Hannken-Illjes - hannken%eis.cs.tu-bs.de@localhost - TU Braunschweig 
(Germany)