Re: [Autotest] [KVM-AUTOTEST PATCH] KVM test: refactor kvm_config.py

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

 



On Thu, Feb 10, 2011 at 01:03:53PM +0200, Avi Kivity wrote:
> On 02/10/2011 12:57 PM, Michael Goldish wrote:
> >>
> >>  I can't easily think of a case where this might cause confusion.  The
> >>  purpose of this is to allow people to write:
> >>
> >>  only qcow2..raw..rtl8139
> >>
> >>  without having to remember the order in which those were defined in
> >>  tests_base.cfg.
> >
> >Sorry, I meant something like
> >
> >only qcow2..hugepages..rtl8139
> >
> >Obviously qcow2 and raw can't coexist.
> 
> The config files describe a cartesian product, in which order matters.

Mathematically speaking, the ordering in the result is different, but BA
and AB are often equivalent for the user.

In many situations, people don't care in which order (as an example)
"qcow" and "ide" are defined on the base config, they just want to
exclude the combination of "qcow" and "ide".

> 
> [A B C] x [1 2] generates [A1 A2 B1 B2 C1 C2]; no confusion here if
> you specify A..1
> 
> however
> 
> [A B C] x [A B] generates [AA AB BA BB CA CB]; A..B is ambiguous

If you do the above and reuse keywords, "A" is also ambiguous, "B" is
also ambiguous. "A..B" being ambiguous is a consequence of "A" and "B"
being ambiguous. If you don't want to be ambiguous, just use "A.B" or
"B.A".

> 
> we might require that keywords be unique.

I wouldn't be against that. At least for the use cases I see, people
have been assuming that keywords are unique on most "only" and "no"
statements.

-- 
Eduardo
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux