[Crash-utility] [PATCH] SIAL extension: bitfield handling fixup
Luc Chouinard
Luc.Chouinard at trueposition.com
Thu Oct 30 16:58:34 UTC 2008
ACK'ed. Thanks for setting a long retransmit timeout for your protocol
config Dave :)
I wander about the history behind the API_GET_UINT64(). Should not have
made it in there in the first place. It probably was always failing
which kept 'val' intact.
Thanks Hedi.
-Luc
On Tue, 2008-10-21 at 09:45 -0400, Dave Anderson wrote:
> Thanks Hedi,
>
> Forwarding to Luc Chouinard for his ACK.
>
> Dave
>
>
> ----- "Hedi Berriche" <hedi at sgi.com> wrote:
>
> > This is a patch from Olaf Weber that fixes a bug in the SIAL extension
> > when
> > dealing with bitfields.
> >
> > Cheers,
> > Hedi.
> >
> > Signed-off-by: Olaf Weber <olaf at sgi.com>
> > Tested-by: Hedi Berriche <hedi at sgi.com>
> >
> > --- a/extensions/sial.c 2008-10-14 14:35:40.000000000 +0100
> > +++ b/extensions/sial.c 2008-10-21 13:16:39.024057612 +0100
> > @@ -363,7 +363,7 @@ int midx;
> > sial_member_soffset(m, TYPE_FIELD_BITPOS(type, midx)/8);
> > sial_member_ssize(m, TYPE_FIELD_TYPE(type, midx)->length);
> > sial_member_snbits(m, TYPE_FIELD_BITSIZE(type, midx));
> > - sial_member_sfbit(m, TYPE_FIELD_BITSIZE(type, midx));
> > + sial_member_sfbit(m, TYPE_FIELD_BITPOS(type, midx)%8);
> > sial_member_sname(m, TYPE_FIELD_NAME(type, midx));
> > LASTNUM=midx+1;
> > return drilldowntype(TYPE_FIELD_TYPE(type, midx), tm);
> > --- a/extensions/libsial/sial_type.c 2008-10-14 14:35:40.000000000
> > +0100
> > +++ b/extensions/libsial/sial_type.c 2008-10-21 13:16:48.823058008
> > +0100
> > @@ -278,9 +278,6 @@ get_bit_value(ull val, int nbits, int bo
> > int dosign=0;
> > int vnbits=size*8;
> >
> > -
> > - val = API_GET_UINT64(&val);
> > -
> > /* first get the value_t */
> > if (nbits >= 32) {
> > int upper_bits = nbits - 32;
> >
> > --
> > Hedi Berriche
> > Global Product Support
> >
>
More information about the Crash-utility
mailing list