[coreboot] fix for cbfs error for longer file names

Kevin O'Connor kevin at koconnor.net
Tue May 12 01:48:13 CEST 2009


On Mon, May 11, 2009 at 10:08:47AM -0700, ron minnich wrote:
> On Mon, May 11, 2009 at 10:01 AM, Myles Watson <mylesgw at gmail.com> wrote:
> 
> > I agree.  Let's do that first, since it would simplify this fix.  I'll
> > review and test.
> 
> Let's see what other comments we get ... anyone out there see an issue
> with 32 byte character names? If you want, we can go with 40, which
> will size the struct up to a nice simple 64 bytes ...

Just my 2 cents, but I like the variable length file names.

BTW, you can simplify the existing coreboot code a bit if you add a
"char filename[0];" entry to 'struct cbfs_file' - as SeaBIOS does.

-Kevin




More information about the coreboot mailing list