Am 16.09.2014 um 20:12 hat Markus Armbruster geschrieben:
> This is consistent with how VirtIOFOOConf variables are named
> elsewhere, and makes blk available for BlockBackend variables.
>
> Signed-off-by: Markus Armbruster
Reviewed-by: Kevin Wolf
On 16.09.2014 20:12, Markus Armbruster wrote:
This is consistent with how VirtIOFOOConf variables are named
elsewhere, and makes blk available for BlockBackend variables.
Signed-off-by: Markus Armbruster
---
hw/block/dataplane/virtio-blk.c | 33 +-
hw/block/dataplane/
Benoît Canet writes:
>> -if (size % dev->blk.conf.logical_block_size) {
>> +if (size % dev->conf.conf.logical_block_size) {
>
> This look strange (conf.conf)
Yeah, it does. I picked conf anyway, for consistency with the existing
similar members of sibling structs VirtIOSCSICommon and Vi
> -if (size % dev->blk.conf.logical_block_size) {
> +if (size % dev->conf.conf.logical_block_size) {
This look strange (conf.conf)
Anyway this seems correct.
Reviewed-by: Benoit Canet
This is consistent with how VirtIOFOOConf variables are named
elsewhere, and makes blk available for BlockBackend variables.
Signed-off-by: Markus Armbruster
---
hw/block/dataplane/virtio-blk.c | 33 +-
hw/block/dataplane/virtio-blk.h | 2 +-
hw/block/virtio-blk.c