Date: Mon, 4 Feb 2013 16:04:49 +1100 (EST) From: Bruce Evans <brde@optusnet.com.au> To: Pedro Giffuni <pfg@FreeBSD.org> Cc: "freebsd-fs@freebsd.org" <freebsd-fs@FreeBSD.org>, Andriy Gapon <avg@FreeBSD.org>, Kirk McKusick <mckusick@FreeBSD.org> Subject: Re: svn commit: r246289 - head/sys/ufs/ffs Message-ID: <20130204155554.I932@besplex.bde.org> In-Reply-To: <1359945154.62069.YahooMailNeo@web162105.mail.bf1.yahoo.com> References: <201302031716.r13HGXNP060303@svn.freebsd.org> <510E9D47.2030403@FreeBSD.org> <20130204062149.U2673@besplex.bde.org> <1359945154.62069.YahooMailNeo@web162105.mail.bf1.yahoo.com>
next in thread | previous in thread | raw e-mail | index | archive | help
This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --0-874308365-1359954289=:932 Content-Type: TEXT/PLAIN; charset=X-UNKNOWN; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE On Sun, 3 Feb 2013, Pedro Giffuni wrote: > (Moving the discussion to freebsd-fs) >> Da: Bruce Evans=A0 >> > ... >>> Just a note that clang actually warned about this one. >>> It has a few more similar warnings for ufs/ffs code. >> >> I wondered how the DIP macro hid the warning. >> =A0 > > The comparison is perfectly legal for UFS1 so perhaps > gcc gives the "benefit of the doubt" to avoid false positives. >=20 >> Perhaps the larger bugs pointed to this warning were lost in translation= : >> - di_blocks overflows for ffs1.=A0 This is now physically possible. >> ... > Surely not anywhere near a complete solution but perhaps it wouldn't > be incompatible=A0to change i_blocks and friends to be unsigned in UFS1. > That is=A0something that remains to be completed in ext2fs, but according > to fsx there=A0are bigger=A0problems there at this time. That only gives 1 more bit, but 7 more are required (for the expansion factor MAXBSIZE / DEV_BSIZE =3D 2**7). More if someone increases MAXBSIZE. Bruce --0-874308365-1359954289=:932--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130204155554.I932>