<div><br><div class="gmail_quote"><div dir="auto">чт, 3 авг. 2017 г. в 15:52, Michael S. Tsirkin <<a href="mailto:mst@redhat.com">mst@redhat.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Jul 29, 2017 at 02:37:49AM +0300, Aleksandr Bezzubikov wrote:<br>
> Unmask previously masked SHPC feature in _OSC method.<br>
><br>
> Signed-off-by: Aleksandr Bezzubikov <<a href="mailto:zuban32s@gmail.com" target="_blank">zuban32s@gmail.com</a>><br>
<br>
This does not do what the subject says - it enables<br>
SHPC unconditionally. And I think it will actually break<br>
ACPI hotplug for the PC unless we add an interface to<br>
disable ACPI hotplug and enable SHPC.<br>
<br>
Pls limit to Q35 only.<br>
<br>
</blockquote><div dir="auto"><br></div><div dir="auto">But this function (build_q35_osc_method) is called only for q35, isn't it?</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> ---<br>
>  hw/i386/acpi-build.c | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c<br>
> index 6b7bade..2ab32f9 100644<br>
> --- a/hw/i386/acpi-build.c<br>
> +++ b/hw/i386/acpi-build.c<br>
> @@ -1848,9 +1848,9 @@ static Aml *build_q35_osc_method(void)<br>
><br>
>      /*<br>
>       * Always allow native PME, AER (no dependencies)<br>
> -     * Never allow SHPC (no SHPC controller in this system)<br>
> +     * Allow SHPC (PCI bridges can have SHPC controller)<br>
>       */<br>
> -    aml_append(if_ctx, aml_and(a_ctrl, aml_int(0x1D), a_ctrl));<br>
> +    aml_append(if_ctx, aml_and(a_ctrl, aml_int(0x1F), a_ctrl));<br>
><br>
>      if_ctx2 = aml_if(aml_lnot(aml_equal(aml_arg(1), aml_int(1))));<br>
>      /* Unknown revision */<br>
> --<br>
> 2.7.4<br>
</blockquote></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature">Aleksandr Bezzubikov</div>