Re: [Qemu-devel] [PULL v2 58/60] qdev: add 64bit properties

2015-06-23 Thread Markus Armbruster
Gonglei arei.gong...@huawei.com writes: On 2015/6/21 5:10, Paolo Bonzini wrote: On 01/06/2015 14:25, Michael S. Tsirkin wrote: +static uint64_t qdev_get_prop_mask64(Property *prop) +{ +assert(prop-info == qdev_prop_bit); +return 0x1 prop-bitnr; This needs to be 1ull. Yes,

Re: [Qemu-devel] [PULL v2 58/60] qdev: add 64bit properties

2015-06-23 Thread Gonglei
On 2015/6/23 14:55, Markus Armbruster wrote: Gonglei arei.gong...@huawei.com writes: On 2015/6/21 5:10, Paolo Bonzini wrote: On 01/06/2015 14:25, Michael S. Tsirkin wrote: +static uint64_t qdev_get_prop_mask64(Property *prop) +{ +assert(prop-info == qdev_prop_bit); +return 0x1

Re: [Qemu-devel] [PULL v2 58/60] qdev: add 64bit properties

2015-06-22 Thread Gonglei
On 2015/6/21 5:10, Paolo Bonzini wrote: On 01/06/2015 14:25, Michael S. Tsirkin wrote: +static uint64_t qdev_get_prop_mask64(Property *prop) +{ +assert(prop-info == qdev_prop_bit); +return 0x1 prop-bitnr; This needs to be 1ull. Yes, and coverity spot it and some other

Re: [Qemu-devel] [PULL v2 58/60] qdev: add 64bit properties

2015-06-20 Thread Paolo Bonzini
On 01/06/2015 14:25, Michael S. Tsirkin wrote: +static uint64_t qdev_get_prop_mask64(Property *prop) +{ +assert(prop-info == qdev_prop_bit); +return 0x1 prop-bitnr; This needs to be 1ull. Paolo +} +

[Qemu-devel] [PULL v2 58/60] qdev: add 64bit properties

2015-06-01 Thread Michael S. Tsirkin
From: Gerd Hoffmann kra...@redhat.com Needed for virtio features which go from 32bit to 64bit with virtio 1.0 Signed-off-by: Gerd Hoffmann kra...@redhat.com Reviewed-by: Michael S. Tsirkin m...@redhat.com Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/hw/qdev-properties.h | 10