[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v3 2/3] vus: Introduce vhost-user-scsi host devi

From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH v3 2/3] vus: Introduce vhost-user-scsi host device
Date: Mon, 16 Jan 2017 17:43:10 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1

On 13/01/2017 01:31, Felipe Franciosi wrote:
>> Please place CONFIG_VHOST_USER_SCSI=$(CONFIG_POSIX) symbol in
>> default-configs/ (so that it is enabled by default on non-Windows hosts)
>> instead of having the configure option.  Otherwise, the patches look good!
> Which architectures would you like me to add this to? How about putting it in 
> "pci.mak"?

I think it should be in every configuration that includes virtio-scsi.

> And when you say "instead", do you mean on top of? I imagine having a 
> --disable switch is at least desirable, so I'd rather not remove the commands 
> from 'configure'.

No, I really meant "instead".  Even though the default-configs/
mechanisms is not very user-friendly, having two ways to do
configuration is messy.  Generally --disable is left for features that
require a third-party library.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]