[LinuxBIOS] cmos checksum in linuxbios table

[Stefan Reinauer]

Hi,

I suggest putting the cmos checksum areas into the LinuxBIOS table. 
This was obviously planned a long time ago and I already implemented
support for that in the lxbios utility.

See below patch to implement this thing in LinuxBIOS.

Stefan

[yhlu]

then who will check the checksum?

YH

On 12/3/05, Stefan Reinauer  wrote:
> Hi,
>
> I suggest putting the cmos checksum areas into the LinuxBIOS table.
> This was obviously planned a long time ago and I already implemented
> support for that in the lxbios utility.
>
> See below patch to implement this thing in LinuxBIOS.
>
> Stefan

[Stefan Reinauer]

Hi Yinghai,

Tools like LNXI's cmos_util and Dave Peterson's lxbios usually need
the cmos.layout file to be able to change CMOS settings.

The settings themselfes are stored in the LinuxBIOS table. The location
of the checksum is also stored. But there's no way to find out what 
area has to be checksummed. So these tools fail without cmos.layout.

Thus someone implemented a LinuxBIOS table entry type
LB_TAG_OPTION_CHECKSUM to pack this information into the LinuxBIOS table
as well.

But LinuxBIOS never created such an entry. I noticed this after
implementing reading this table entry in lxbios (lxbios.sourceforge.net)

If an LB_TAG_OPTION_CHECKSUM entry is created (which the patch does)
cmos.layout is not needed as an extra file to change cmos settings.
Everything can be read from the LinuxBIOS table.

Stefan

* yhlu  [051203 20:49]:
> then who will check the checksum?
>
> YH
>
> On 12/3/05, Stefan Reinauer  wrote:
> > Hi,
> >
> > I suggest putting the cmos checksum areas into the LinuxBIOS table.
> > This was obviously planned a long time ago and I already implemented
> > support for that in the lxbios utility.
> >
> > See below patch to implement this thing in LinuxBIOS.
> >
> > Stefan

Stefan Reinauer  writes:
> Hi,
>
> I suggest putting the cmos checksum areas into the LinuxBIOS table. 
> This was obviously planned a long time ago and I already implemented
> support for that in the lxbios utility.
>
> See below patch to implement this thing in LinuxBIOS.

It should already be there....

I know cmos util needs to check the location.

Eric

[Stefan Reinauer]

* Eric W. Biederman  [051203 22:22]:
> > See below patch to implement this thing in LinuxBIOS.
>
> It should already be there....
>
> I know cmos util needs to check the location.

I know, that's what I thought, and that's why I implemented parsing it
in the lxbios utility as well. But then again, go grep for
LB_TAG_OPTION_CHECKSUM in the source tree. You will find it is not
there.

I'll apply the patch.

BTW, I've added another patch at
https://openbios.org/roundup/linuxbios/issue42

after finding out that linuxbios_table.c is exactly the same on ppc and
i386 systems. We should definitely share code here.

Stefan