[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/11] NVMe: Initial commit for NVM Express devi
|
From: |
Markus Armbruster |
|
Subject: |
Re: [Qemu-devel] [PATCH 01/11] NVMe: Initial commit for NVM Express device |
|
Date: |
Wed, 27 Feb 2013 13:02:54 +0100 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 27.02.2013 um 10:48 hat Paolo Bonzini geschrieben:
>> Finally, and unrelated to the code above, we usually do not add new "if"
>> values. It's enough to have access via if=none & -device. In any case
>> it should be a separate patch, so it's easy to drop it if that's the
>> decision.
>
> -device is an unfriendly interface for disks, so I would support adding
> a new if=nvme convenience option. But you're right that having a
> separate patch for it should be possible.
Convenience syntax such as new interface types needs justification.
Thus, definitely a separate patch.
- Re: [Qemu-devel] [PATCH 09/11] QEMU NVMe: Implement discontiguous queues, (continued)
Re: [Qemu-devel] [PATCH 00/11] *** SUBJECT HERE ***, Stefan Hajnoczi, 2013/02/27