Re: [Qemu-devel] [PATCH 06/16] net: Convert qdev_prop_vlan to peer with hub

[Laszlo Ersek <lersek@redhat.com>]

On 07/20/12 14:01, Stefan Hajnoczi wrote:

> @@ -638,11 +642,17 @@ static void get_vlan(Object *obj, Visitor *v, void *opaque,
>  {
>      DeviceState *dev = DEVICE(obj);
>      Property *prop = opaque;
> -    VLANState **ptr = qdev_get_prop_ptr(dev, prop);
> -    int64_t id;
> +    VLANClientState **ptr = qdev_get_prop_ptr(dev, prop);
> +    int64_t id = -1;
>  
> -    id = *ptr ? (*ptr)->id : -1;
> -    visit_type_int64(v, &id, name, errp);
> +    if (*ptr) {
> +        unsigned int hub_id;
> +        if (!net_hub_id_for_client(*ptr, &hub_id)) {
> +            id = (int64_t)hub_id;
> +        }
> +    }
> +
> +    visit_type_int(v, &id, name, errp);
>  }

Should we use uint32 here? (No particular reason, just for "cleanliness"
or whatever.)

Thanks,
Laszlo