Re: [PATCH v8] run QEMU as non-root

[Jim Fehlig <jfehlig@suse.com>]

Ian Campbell wrote:
> On Tue, 2015-10-06 at 14:13 +0100, Stefano Stabellini wrote:
>> On Mon, 5 Oct 2015, Ian Campbell wrote:
>>> On Mon, 2015-10-05 at 16:53 +0100, Stefano Stabellini wrote:
>>>>> Wasn't there some code to plumb this into xl at one point? Did that
>>>>> get
>>>>> dropped along the way?
>>>> device_model_user is added to the idl by this patch, I think that is
>>>> enough, right?
>>> Depends what you mean by "enough", it adds it to the _libxl_ API, which
>>> is
>>> sufficient for the patch to be useful but it doesn't actually cause the
>>> option to be available to users of xl via a cfg file, which arguably it
>>> should.
>>>
>>> To make it available to xl users patches to tools/libxl/xl_cmdimpl.c
>>> would
>>> be needed (plus docs changes)
>> I see. No, xl_cmdimpl.c support for the option was never written, as it
>> was introduced principally for libvirt's benefit.
> 
> Ah, I was wondering how this had been tested...

Currently, there is no libivrt code to using this. But surely users would like
to specify the qemu user in their xl config right?

Regards,
Jim