Keir Fraser wrote:
> On 12/05/2010 18:09, "Gihan Munasinghe" <GMunasinghe@flexiant.com> 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
>
>
>
>
>   
Sorry for the very long delay see updated patch


xl: adds shutdown and reboot commands
libxl : remote shutdown to work for pure hvm domains

Signed-off-by: Gihan Munasinghe <GMunasinghe@flexiant.com>