[SeaBIOS] [PATCH 0/4] add guest hint for memory layout (pci bar placement)

Gerd Hoffmann posted 4 patches 4 weeks, 1 day ago
src/fw/paravirt.h | 1 +
src/fw/paravirt.c | 9 +++++++++
src/fw/pciinit.c  | 2 +-
src/Kconfig       | 7 +++++++
4 files changed, 18 insertions(+), 1 deletion(-)
[SeaBIOS] [PATCH 0/4] add guest hint for memory layout (pci bar placement)
Posted by Gerd Hoffmann 4 weeks, 1 day ago
This series adds an option to set the memory layout preference,
both at compile time (via kconfig) and runtime (via fwcfg).

Gerd Hoffmann (4):
  add guest hint variable
  use guest hint as additional condition
  add guest hint default kconfig
  add guest hint fw_cfg override

 src/fw/paravirt.h | 1 +
 src/fw/paravirt.c | 9 +++++++++
 src/fw/pciinit.c  | 2 +-
 src/Kconfig       | 7 +++++++
 4 files changed, 18 insertions(+), 1 deletion(-)

-- 
2.47.0

_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org
[SeaBIOS] Re: [PATCH 0/4] add guest hint for memory layout (pci bar placement)
Posted by Kevin O'Connor 4 weeks, 1 day ago
On Fri, Nov 15, 2024 at 12:40:06PM +0100, Gerd Hoffmann wrote:
> This series adds an option to set the memory layout preference,
> both at compile time (via kconfig) and runtime (via fwcfg).
> 
> Gerd Hoffmann (4):
>   add guest hint variable
>   use guest hint as additional condition
>   add guest hint default kconfig
>   add guest hint fw_cfg override

Thanks.  For what it is worth, I think it would be preferable to have
a more specific variable name, such as "etc/pci-pad-64bit-mappings"
and make the implementation specific to the pciinit.c code.  I fear a
generic name like "GuestHint" will be opaque to users and future
developers, and possibly encourage complicated heuristics.

It might also be nice to support forcing the option in both directions
- for example: 0 - use default heuristic, 1 - avoid 64bit padding, 2 -
force 64bit padding.

I'd also avoid adding a Kconfig setting for this - we don't use
Kconfig for any other defaults and I think it will be unwieldly to try
to use Kconfig in that role.

Cheers,
-Kevin
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org