Re: [Xen-devel] Possible bug with pass-through hot-plug?

[Tom Rotenberg <tom.rotenberg@xxxxxxxxx>]

Simon,

I can confirm that the original patch u sent is working, and it fixed
the problem.
Thanks!

Tom

On Fri, Nov 27, 2009 at 12:47 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote:
> On Thu, Nov 26, 2009 at 01:54:24PM +0000, Keir Fraser wrote:
>> On 24/11/2009 05:48, "Simon Horman" <horms@xxxxxxxxxxxx> wrote:
>>
>> > sorry for the extended delay, I was first busy with travel
>> > and then with family. I'm finally back on deck and sifting through
>> > my todo list.
>> >
>> > Can you see if the patch below solves the problem that you were
>> > observing. It seems to help in my test environment.
>> >
>> > -----------------------------------------------------------------------------
>> >
>> > hvmloader: pass-through: use O(log(n)) logic for hotplug events
>>
>> This looked good to me, but I think that all this per-slot logic in DSDT is
>> a pain to manage manually. So I've applied a patch (xen-unstable:20510) to
>> auto-generate those parts of the DSDT. Simon: you might want to take a look
>> and also check that it does actually work. ;-)
>
> Hi Keir,
>
> that seems to work with the following minor modification in place.
>
> -------------------------------------------------------------------
>
> build: Execute mk_dsdt with path
>
> Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx>
>
>
> Index: xen-unstable.hg/tools/firmware/hvmloader/acpi/Makefile
> ===================================================================
> --- xen-unstable.hg.orig/tools/firmware/hvmloader/acpi/Makefile 2009-11-27 
> 08:59:33.000000000 +1100
> +++ xen-unstable.hg/tools/firmware/hvmloader/acpi/Makefile      2009-11-27 
> 09:45:32.000000000 +1100
> @@ -36,7 +36,7 @@ dsdt.c: dsdt.asl mk_dsdt.c
>        $(MAKE) iasl
>        $(HOSTCC) $(HOSTCFLAGS) -o mk_dsdt mk_dsdt.c
>        head -n -1 $< >_dsdt.asl
> -       mk_dsdt >>_dsdt.asl
> +       ./mk_dsdt >>_dsdt.asl
>        iasl -tc _dsdt.asl
>        mv _dsdt.hex dsdt.c
>        echo "int DsdtLen=sizeof(AmlCode);" >> dsdt.c
>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel