Andi Kleen wrote:
> On Thu, Jun 01, 2000 at 04:11:41PM -0500, Russell Cattelan wrote:
> > >
> > > to do that for the ``TAKE'' messages on the list? Then you could just
> > > click on a change to look at it.
> > >
> > > -Andi
> >
> > BTW did switching compilers help?
>
> Yes, it seems switching compilers did help.
>
> Do you know which function is miscompiled ? [it may be useful to write a gcc
> bugreport]
>
> -Andi
Look at about line 3826 in the function
__pb_set_blocks_mp
bh->b_blocknr = (long)mp->pbm_bn +
(mp->pbm_delta >> inode->i_sb->s_blocksize_bits);
This line --> bh->b_blocknr >>= block_bits;
Putting printk's in will show that b_blocknr to be correct.
Taking them out and looking at the values with kdb will show the wrong values.
One way arond this is to call a function that does the shift.
I didn't spend to much time looking at the asm code to try and understand what
is going wrong.
If you spot something obvious let me know.
-Russell
|