<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2017-07-24 12:42 GMT+03:00 Gerd Hoffmann <span dir="ltr"><<a href="mailto:kraxel@redhat.com" target="_blank">kraxel@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, 2017-07-23 at 22:44 +0300, Alexander Bezzubikov wrote:<br>
> By the way, any ideas on how to avoid 'bus overstealing' would <br>
> be greatly appreciated.<br>
> Static BIOS variable isn't applicable since its value isn't saved<br>
> across reboots.<br>
<br>
</span>I think the reservation hints should be a absolute number, not a<br>
increment.  i.e. if qemu suggests to reserve three extra bus numbers<br>
seabios should reserve three, no matter whenever there are zero, one,<br>
two or three child busses present.  And I guess seabios should<br>
interpret that as minimum, so in case it finds five child busses it<br>
will allocate five bus numbers of course ...<br></blockquote><div><br></div><div>Personally I have nothing against it. Marcel, Michael, what do you think?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Same with the other limit hints.  If the hint says to allocate 16M, and<br>
existing device bars sum up to 4M, allocate 16M (and therefore leave<br>
12M address space for hotplug).  If the device bars sum up to 32M,<br>
allocate that.<br>
<br>
While being at it:  I have my doubts the capability struct layout<br>
(which mimics register layout) buys us that much, seabios wouldn't<br>
blindly copy over the values anyway.  Having regular u32 fields looks<br>
more useful to me.<br>
<br></blockquote><div><br></div><div>Again, if nobody has any objections, I can change it in v3.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
cheers,<br>
  Gerd<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Alexander Bezzubikov</div></div></div></div>
</div></div>