<div dir="ltr">Thanks,<div>that means it is just as I expected and I only needed the eye opener where to look at.</div><div>Thanks Paolo and Gerd!</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 11, 2018 at 3:53 PM, Paolo Bonzini <span dir="ltr"><<a href="mailto:pbonzini@redhat.com" target="_blank">pbonzini@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11/06/2018 15:21, Christian Ehrhardt wrote:<br>
> Hi,<br>
> I was asked about x86 Guests of >1TB in size. And while some discussions<br>
> where around qemu/libvirt and host-phys-bits [1] I realized that in<br>
> Seabios I need exactly what is already in CentOS/RHEL [2] to get the<br>
> phys-bits passed on.<br>
> <br>
> The change [2] itself is rather old, so I wondered if I'm missing that<br>
> this was implemented in a totally different way. Do I have to switch/set<br>
> options these days instead of using that patch?<br>
<br>
</span>That patch is not needed anymore.  It is only there to support old<br>
machine types.  In newer versions of QEMU, QEMU builds the e820 memory<br>
map for SeaBIOS, and that is enough to support >=1TB guests nicely.<br>
<span class="HOEnZb"><font color="#888888"><br>
Paolo<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> But I saw that it is still applied even to rather recent versions.<br>
> <br>
> So the question become why the change is not upstream yet?<br>
> Was it maybe discussed in the past and Nack'ed for some reason?<br>
> I didn't find the discussion if that is the case and would appreciate<br>
> the pointer.<br>
> <br>
> We are closing in to make 1TB more common rather quickly, so I wonder if<br>
> really nothing would speak against it - would it be reasonable to<br>
> consider committing that upstream to Seabios these days?<br>
> <br>
> [1]: <a href="https://bugs.launchpad.net/ubuntu/+source/qemu/+bug/1769053" rel="noreferrer" target="_blank">https://bugs.launchpad.<wbr>net/ubuntu/+source/qemu/+bug/<wbr>1769053</a><br>
> [2]: <a href="https://git.centos.org/blob/rpms!seabios.git/14f0fd75785bc5f1468fa84fbd3a1627f3433032/SOURCES!0002-allow-1TB-of-RAM.patch" rel="noreferrer" target="_blank">https://git.centos.org/<wbr>blob/rpms!seabios.git/<wbr>14f0fd75785bc5f1468fa84fbd3a16<wbr>27f3433032/SOURCES!0002-allow-<wbr>1TB-of-RAM.patch</a><br>
> <br>
> P.S. Subscribing people acking the original patch as they might have old<br>
> context to provide on this.<br>
> <br>
> -- <br>
> Christian Ehrhardt<br>
> Software Engineer, Ubuntu Server<br>
> Canonical Ltd<br>
<br>
</div></div></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"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div>