Re: [Qemu-devel] [Xen-devel] Hvmloader: Add _STA for PCI hotplug slots

[Jan Beulich]

>>> On 10.10.13 at 15:32, "Gonglei (Arei)" <address@hidden> wrote:
> --- mk_dsdt.c (revision 90666)
> +++ mk_dsdt.c (working copy)
> @@ -437,7 +437,7 @@
>          indent(); printf("B0EJ, 32,\n");
>          pop_block();
>  
> -        stmt("OperationRegion", "SRMV, SystemIO, 0xae0c, 0x04");
> +        stmt("OperationRegion", "SRMV, SystemIO, 0xae00, 0x04");

Is this change really necessary? And if so, why would nothing else
require adjustment (i.e. is the value either arbitrary, or used only
here and nowhere else, and also is guaranteed to not conflict with
anything else)?

Jan

>          push_block("Field", "SRMV, DWordAcc, NoLock, WriteAsZeros");
>          indent(); printf("RMV, 32,\n");
>          pop_block();        
> @@ -451,10 +451,10 @@
>                  } pop_block();
>                  push_block("Method", "_STA, 0");{
>                     push_block("If", "And(RMV, ShiftLeft(1, %#06x))", slot);
> -                      stmt("Return", "0x1F");
> +                      stmt("Return", "0x0F");
>                     pop_block();
>                     push_block("Else", NULL);
> -                      stmt("Return", "0x1E");
> +                      stmt("Return", "0x00");
>                     pop_block();
>                  };pop_block();
>                  stmt("Name", "_SUN, %i", slot);
> 
> based on this patch, PCI hot plug/unplug is supported in Redhat5.8/win2008, 
> but the problem still exists in Redhat6.3.
> 
> More support are needed, Expecting your reply.
> 
> Best Regards,
> -Gonglei