[Linux-cluster] Re: [PATCH 00/14] GFS
David Teigland
teigland at redhat.com
Mon Aug 8 06:26:36 UTC 2005
On Fri, Aug 05, 2005 at 03:14:15PM +0800, David Teigland wrote:
> On Tue, Aug 02, 2005 at 09:45:24AM +0200, Arjan van de Ven wrote:
> > * +++ b/fs/gfs2/fixed_div64.h 2005-08-01 14:13:08.009808200 +0800
> > ehhhh why?
>
> I'm not sure, actually, apart from the comments:
>
> do_div: /* For ia32 we need to pull some tricks to get past various versions
> of the compiler which do not like us using do_div in the middle
> of large functions. */
>
> do_mod: /* Side effect free 64 bit mod operation */
>
> fs/xfs/linux-2.6/xfs_linux.h (the origin of this file) has the same thing,
> perhaps this is an old problem that's now fixed?
I've looked into getting rid of these:
- The existing do_div() works fine for me with 64 bit numerators, so I'll
get rid of the "fixed" version.
- The "fixed" do_mod() seems to be the only way to do 64 bit modulus.
It would be great if I was wrong about that...
Thanks,
Dave
More information about the Linux-cluster
mailing list