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

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

 




>>> On 12/18/2014 at 01:00 PM, in message
<5492D0080200006600086404@xxxxxxxxxxxxxxxxxxxxx>, "Chun Yan Liu"
<cyliu@xxxxxxxx> wrote: 

>  
>>>> On 12/17/2014 at 06:52 PM, in message <20141217105227.GQ136165@xxxxxxxxxx>, 
> Jiri Denemark <jdenemar@xxxxxxxxxx> wrote:  
> > On Wed, Dec 17, 2014 at 16:48:52 +0800, Chunyan Liu wrote:  
> > > Add public API virDomainSendSysrq for sending SysRequest key.  
> > >   
> > > Signed-off-by: Chunyan Liu <cyliu@xxxxxxxx>  
> > > ---  
> > > changes:  
> > >   * add 'flags' to the new API  
> > >   * change parameter from 'const char *key' to 'char key'  
> > >   * change version number from 1.2.11 to 1.2.12  
> > >   
> > >  include/libvirt/libvirt-domain.h |  3 +++  
> > >  src/driver-hypervisor.h          |  4 ++++  
> > >  src/libvirt-domain.c             | 39   
> > +++++++++++++++++++++++++++++++++++++++  
> > >  src/libvirt_public.syms          |  5 +++++  
> > >  4 files changed, 51 insertions(+)  
> > >   
> > > diff --git a/include/libvirt/libvirt-domain.h   
> > b/include/libvirt/libvirt-domain.h  
> > > index baef32d..5f72850 100644  
> > > --- a/include/libvirt/libvirt-domain.h  
> > > +++ b/include/libvirt/libvirt-domain.h  
> > > @@ -3526,6 +3526,9 @@ int virDomainGetFSInfo(virDomainPtr dom,  
> > >                         virDomainFSInfoPtr **info,  
> > >                         unsigned int flags);  
> > >    
> > > +/* virDomainSendSysrq */  
> > > +int virDomainSendSysrq(virDomainPtr dom, char key, unsigned int flags);  
> > > +  
> >   
> > I think quite a few reviewers (Daniel, Eric, and I) agreed on using an  
> > enum instead of char so that the API is more general.  
>  
> Sorry, I missed this part. I'll update. One left question: 
> How about 'virsh sysrq' parameters? What would we expect users to pass?

Any thoughts on that?
 
>  
> >   
> > Jirka  
> >   
> >   
>  
>  
>  
> -- 
> libvir-list mailing list 
> libvir-list@xxxxxxxxxx 
> https://www.redhat.com/mailman/listinfo/libvir-list 
>  
>  



--
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]