 cf4e3d000e
			
		
	
	
		cf4e3d000e
		
	
	
	
	
		
			
			virtio_crypto_pci_realize() and copies the value of vcrypto->vdev's
property "cryptodev" to vcrypto's property:
    object_property_set_link(OBJECT(vrng), "rng", OBJECT(vrng->vdev.conf.rng),
                             NULL);
Since it does so only after realize, this always fails, but the error
is ignored.
It's actually superfluous: vcrypto's property is an alias of
vcrypto->vdev's property, created by virtio_instance_init_common().
Drop the call.
Same for virtio_ccw_crypto_realize(), virtio_rng_pci_realize(),
virtio_ccw_rng_realize().
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20200721121153.1128844-1-armbru@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
		
	
			
		
			
				
	
	
		
			69 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			69 lines
		
	
	
		
			1.9 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
| /*
 | |
|  * virtio ccw random number generator implementation
 | |
|  *
 | |
|  * Copyright 2012, 2015 IBM Corp.
 | |
|  * Author(s): Cornelia Huck <cornelia.huck@de.ibm.com>
 | |
|  *
 | |
|  * This work is licensed under the terms of the GNU GPL, version 2 or (at
 | |
|  * your option) any later version. See the COPYING file in the top-level
 | |
|  * directory.
 | |
|  */
 | |
| 
 | |
| #include "qemu/osdep.h"
 | |
| #include "hw/qdev-properties.h"
 | |
| #include "hw/virtio/virtio.h"
 | |
| #include "qapi/error.h"
 | |
| #include "qemu/module.h"
 | |
| #include "virtio-ccw.h"
 | |
| 
 | |
| static void virtio_ccw_rng_realize(VirtioCcwDevice *ccw_dev, Error **errp)
 | |
| {
 | |
|     VirtIORNGCcw *dev = VIRTIO_RNG_CCW(ccw_dev);
 | |
|     DeviceState *vdev = DEVICE(&dev->vdev);
 | |
| 
 | |
|     if (!qdev_realize(vdev, BUS(&ccw_dev->bus), errp)) {
 | |
|         return;
 | |
|     }
 | |
| }
 | |
| 
 | |
| static void virtio_ccw_rng_instance_init(Object *obj)
 | |
| {
 | |
|     VirtIORNGCcw *dev = VIRTIO_RNG_CCW(obj);
 | |
| 
 | |
|     virtio_instance_init_common(obj, &dev->vdev, sizeof(dev->vdev),
 | |
|                                 TYPE_VIRTIO_RNG);
 | |
| }
 | |
| 
 | |
| static Property virtio_ccw_rng_properties[] = {
 | |
|     DEFINE_PROP_BIT("ioeventfd", VirtioCcwDevice, flags,
 | |
|                     VIRTIO_CCW_FLAG_USE_IOEVENTFD_BIT, true),
 | |
|     DEFINE_PROP_UINT32("max_revision", VirtioCcwDevice, max_rev,
 | |
|                        VIRTIO_CCW_MAX_REV),
 | |
|     DEFINE_PROP_END_OF_LIST(),
 | |
| };
 | |
| 
 | |
| static void virtio_ccw_rng_class_init(ObjectClass *klass, void *data)
 | |
| {
 | |
|     DeviceClass *dc = DEVICE_CLASS(klass);
 | |
|     VirtIOCCWDeviceClass *k = VIRTIO_CCW_DEVICE_CLASS(klass);
 | |
| 
 | |
|     k->realize = virtio_ccw_rng_realize;
 | |
|     device_class_set_props(dc, virtio_ccw_rng_properties);
 | |
|     set_bit(DEVICE_CATEGORY_MISC, dc->categories);
 | |
| }
 | |
| 
 | |
| static const TypeInfo virtio_ccw_rng = {
 | |
|     .name          = TYPE_VIRTIO_RNG_CCW,
 | |
|     .parent        = TYPE_VIRTIO_CCW_DEVICE,
 | |
|     .instance_size = sizeof(VirtIORNGCcw),
 | |
|     .instance_init = virtio_ccw_rng_instance_init,
 | |
|     .class_init    = virtio_ccw_rng_class_init,
 | |
| };
 | |
| 
 | |
| static void virtio_ccw_rng_register(void)
 | |
| {
 | |
|     type_register_static(&virtio_ccw_rng);
 | |
| }
 | |
| 
 | |
| type_init(virtio_ccw_rng_register)
 |