Date: Tue, 26 Nov 1996 19:12:32 -0700 (MST) From: Terry Lambert <terry@lambert.org> To: joerg_wunsch@uriah.heep.sax.de Cc: freebsd-current@FreeBSD.org Subject: Re: Dosboot and bcc Message-ID: <199611270212.TAA26348@phaeton.artisoft.com> In-Reply-To: <199611270139.CAA23113@uriah.heep.sax.de> from "J Wunsch" at Nov 27, 96 02:39:51 am
next in thread | previous in thread | raw e-mail | index | archive | help
> > This is so frigging bogus. > ... > > The ufs code should use "uint64", not "long long". > ...and Terry should use grep. > > j@uriah 606% grep 'long *long' /sys/ufs/*/* > j@uriah 607% > > Christian has been referring to system _header_ files here since he > wants to re-use the system's native files for the dosboot stuff, as > opposed to whack his own set. He will need his own definition of uint64. Obviously one that does not use "long long". However, ig you grep for the 64 bit fields in the ufs/ffs headers, and then you grep for their references, you will see that they directly reference the values as if they were atomic types. This is the bogus part of the FS code, since it should not be dependent on the definition of a 64bit type to be atomic. So rather than me using grep once, you should have used grep twice. FWIW: The same thing applies to system time references, which should potentially be function call wrappers, since not all systems have a global variable containing the current time. Regards, Terry Lambert terry@lambert.org --- Any opinions in this posting are my own and not those of my present or previous employers.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199611270212.TAA26348>