Re: [Qemu-devel] Re: [PATCH RFC] virtio: add features qdev property

[Gerd Hoffmann]

On 12/14/09 14:30, Markus Armbruster wrote:
> > Then the property parser would accepts strings such as 'bit1|bit2' and
> > you can have
> >
> >    -device 'virtio-blk-pci,disable=blk-identify|ring-indirect'
> >
> > The driver will just do 'vdev->host_features&= ~disable'.
>
> Use of '|' in option argument syntax is user-hostile, because it
> requires quoting in the shell.  What about '+'?

I don't care that much.  I've picked '|' because it is 'or' in many 
programming languanges and thus sort of intuitive (at least to me). 
Using '+' is fine with me too.

cheers,
  Gerd