Re: [PATCH V2 1/5] Add public API virDomainSendSysrq

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/12/14 10:18, Peter Krempa wrote:
> On 12/12/14 10:04, Chunyan Liu wrote:
>> Add public API virDomainSendSysrq for sending SysRequest key.
>>
>> Signed-off-by: Chunyan Liu <cyliu@xxxxxxxx>
>> ---
>>  include/libvirt/libvirt-domain.h |  3 +++
>>  src/driver-hypervisor.h          |  4 ++++
>>  src/libvirt-domain.c             | 38 ++++++++++++++++++++++++++++++++++++++
>>  src/libvirt_public.syms          |  1 +
>>  4 files changed, 46 insertions(+)
>>
> 
> [...]
> 
>> diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
>> index cb76d8c..4658fd7 100644
>> --- a/src/libvirt-domain.c
>> +++ b/src/libvirt-domain.c
>> @@ -11192,3 +11192,41 @@ virDomainFSInfoFree(virDomainFSInfoPtr info)
>>          VIR_FREE(info->devAlias[i]);
>>      VIR_FREE(info->devAlias);
>>  }
>> +
>> +
>> +/**
>> + * virDomainSendSysrq:
>> + * @domain:    pointer to domain object, or NULL for Domain0
>> + * @key:    SysRq key, like h, c, ...
>> + *
>> + * Send SysRq key to the guest.
>> + *
>> + * Returns 0 in case of success, -1 in case of failure.
>> + */
>> +int
>> +virDomainSendSysrq(virDomainPtr domain, const char *key)
> 
> The new API should definitely have a 'flags' argument although it may be
> unused for now.
> 

As an additional thought. Isn't this possible with the existing
virDomainSendKey API? The sysrq combination is a key code, so you should
be able to use that API to do the same thing. In that case all you need
is a virsh wrapper and an implementation of the API for the XEN driver.

Peter


Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]