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. Paolo
