V2 Epia report

ron minnich rminnich at lanl.gov
Thu Oct 9 10:20:00 CEST 2003


On 8 Oct 2003, Eric W. Biederman wrote:

> The current device resource assignment code should cope with static
> resource assignments, so hopefully it should be a matter of plugging
> hard codes into the device tree.

no, greg and I will be talking to you about this. There is a problem with 
that code, which I have alluded to, in that you can not (in the current 
system) do device assignments etc. before pci enumeration, and it is 
essential that you be able to do that.

> Although looking at that code there is another issue. You are using
> dev_find_device in vt8231.c inappropriately.  dev_find_device should
> be virtually unnecessary in the freebios2 tree.  Except when you are
> very carefully using dev_find_device will fail to handle multiple
> instances of a device.  This is a very bad example to set when
> doing things properly causes everything to work transparently.

legacy code. Has to get fixed. 

Examples of proper usage welcomed. Although this actually points out a 
problem with the dynamic tree: it handles complex cases well, simple cases 
poorly. All I want to do is get into that device BEFORE pci enumeration 
and set some default values. You can't do that in the current scheme.

> This is also a limitation in FILO that it is scanning for devices only
> using the legacy port addresses.  Using those addresses is great but this
> problem would have remained hidden if FILO did a scan through pci devices
> like etherboot does.

yes.


ron




More information about the coreboot mailing list