<div dir="ltr">Maybe rename Windows 2001, 2006, and 2009 to their real names?<br>Or use their NT kernel name instead, like NT 6.1, NT 6.2, etc.<br>Or add a comment in the source code like:<br>“Windows 2009” /* Windows 7 */ ||<br>
"Windows 2012" /* Windows Server 2012 */ ||<br> </div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 5, 2013 at 9:11 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"><div class="im"><br>
> > From my quick research, it looks like "Windows 2006" || "Windows<br>
> > 2006.1" || "Linux" would work, but I have not tested it.<br>
> ><br>
> > Paolo<br>
><br>
> This doesn't work in that it matches linux.<br>
<br>
</div>Note that the above was meant to be a condition for when to _show_<br>
the PCI hole, i.e. negated compared to your example.<br>
<div class="im"><br>
> ATM it looks like we should test<br>
>         "Windows 2000" ||<br>
>         "Windows 2001" ||<br>
>         "Windows 2001 SP1" ||<br>
>       "Windows 2001.1 SP1"<br>
<br>
</div>Including this may be too strict, what about 98/ME?<br>
<div class="im"><br>
>       && !(<br>
>         "Windows 2006" ||<br>
>         "Windows 2006.1" ||<br>
<br>
</div>We know that these are all implied by the following four:<br>
<div class="im"><br>
>         "Windows 2006 SP1" ||<br>
>         "Windows 2006 SP2" ||<br>
>         "Windows 2009" ||<br>
>         "Windows 2012" ||<br>
<br>
</div>So it is not necessary to test these four.<br>
<div class="im"><br>
>       "Linux" ||<br>
>       "FreeBSD"<br>
>       ) &&<br>
>       _OS == "Microsoft Windows NT"<br>
>       &&<br>
>       _REV == 0x1<br>
<br>
</div>Testing _OS and _REV is probably too strict.<br>
<div class="im"><br>
> This should match XP and 2003 as tightly as possible.<br>
> Please note "Linux" is there just in case, modern<br>
> Linux OSPM does not identify itself as "Linux".<br>
<br>
</div>Yeah, I know.  I didn't know about FreeBSD, and I agree it is<br>
better to include it just in case.<br>
<span class="HOEnZb"><font color="#888888"><br>
Paolo<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
SeaBIOS mailing list<br>
<a href="mailto:SeaBIOS@seabios.org">SeaBIOS@seabios.org</a><br>
<a href="http://www.seabios.org/mailman/listinfo/seabios" target="_blank">http://www.seabios.org/mailman/listinfo/seabios</a><br>
</div></div></blockquote></div><br></div>