On Wed, May 15, 2019 at 12:42:44PM +0100, Andrew Jones wrote: > On Wed, May 15, 2019 at 12:00:45PM +0100, Dave Martin wrote: > > On Wed, May 15, 2019 at 09:18:54AM +0100, Andrew Jones wrote: > > > On Tue, May 14, 2019 at 04:48:38PM +0200, Igor Mammedov wrote: > > > > On Tue, 14 May 2019 11:02:25 +0200 > > > > Andrew Jones <drjo...@redhat.com> wrote: > > > > > My thought is primarily machines. If a human wants to use the command > > > > > line and SVE, then I'm assuming they'll be happy with sve-max-vq or > > > > > figuring out a map they like once and then sticking to it. > > > > > > > > maybe naive question, but why not use a property/bit as user facing > > > > interface, > > > > in line with what we do with CPUID bits. (that's assuming that bits have > > > > fixed meaning). > > > > Yes, it's verbose but follows current practice and works fine with -cpu > > > > and > > > > -device. > > > > (I really hate custom preprocessing of -cpu and we were working hard to > > > > remove > > > > that in favor of canonical properties at the expense of more verbose > > > > CLI). > > > > > > > > > > Are you asking if we should do something like the following? > > > > > > -cpu host,sve1=on,sve=2=on,sve3=off,sve4=on > > > > Note, there is nothing SVE-specific about this. > > In the above example there is some specific SVE stuff there. If the > command line has sve4=on, then it must also have sve1=on and sve2=on, > per the architecture requiring all smaller power-of-2 vector lengths. > Only sve3 is optional, but because it's optional we have to explicitly > state when it's on or off in order to ensure we can cleanly fail a > migration to a host that doesn't support that option. > > > > > Either enabling features on a per-vcpu basis is justified, or it isn't: > > if it's justified, then it would be better to have a general way of > > specifying per-vcpu properties, rather than it being reinvented per > > feature. > > > > Creating mismatched configurations is allowed by the architecture and so > > it's useful for testing the kernel, but probably less useful for real- > > world use cases today. > > > > So it may be a good idea to get the symmetric support sorted out first > > before thinking about whether and how to specify asymmetric > > configurations. > > These properties are per-vcpu for KVM only. QEMU doesn't have a way > to allow per-vcpu features to be described on the command line yet. > With '-cpu host,...' The '...' applies to all vcpus. So we are "just" > working on the symmetric support now.
OK, I think I misunderstood what was being proposed here. Until/unless someone comes up with a compelling use case, I think it's entirely reasonable for QEMU not to support asymmetry of this sort. Cheers ---Dave