Re: [Xen-devel] [PATCH] new commands "xl reboot" & "xl shutdown"

[Keir Fraser <keir.fraser@xxxxxxxxxxxxx>]

On 12/05/2010 18:09, "Gihan Munasinghe" <GMunasinghe@xxxxxxxxxxxx> wrote:

>> the if (/* hvm */ 0) is because the function never properly tested if
>> the domain is hvm or not. nowadays you have is_hvm(domid) function
>> that can handily replace the hardcoded value.
>> 
> Yes having is_hvm is much better than the hard coded values I have
> changed that bit of code see attached patch.

Please port your patch to the tip of xen-unstable and re-post with changeset
comment and signed-off-by-line.

 Thanks,
 Keir



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