[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 12/16] IPMI: Add a PC ISA type structure
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 12/16] IPMI: Add a PC ISA type structure |
Date: |
Mon, 16 Jul 2012 09:16:39 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120615 Thunderbird/13.0.1 |
Il 15/07/2012 22:25, address@hidden ha scritto:
> + /* Clear the property from this device so we can put it elsewhere */
> + chr = isa->chr;
> + qdev_prop_set_chr(&dev->qdev, "chardev", NULL);
> +
> + if (chr) {
> + bdev = qdev_create(NULL, "ipmi-bmc-extern");
> + qdev_prop_set_chr(bdev, "chardev", chr);
> + } else {
> + bdev = qdev_create(NULL, "ipmi-bmc-sim");
> + }
> + snprintf(typename, sizeof(typename), "ipmi-interface-%s",
> isa->interface);
> + idev = qdev_create(&isa_bus->qbus, typename);
> + qdev_prop_set_ptr(bdev, "ipmiif", idev);
> + qdev_prop_set_uint64(idev, "iobase", isa->iobase);
> + qdev_prop_set_ptr(idev, "bmc", bdev);
> + qdev_prop_set_uint8(idev, "slave-addr", isa->slave_addr);
> + rc = qdev_init(idev);
I think this is adding a bit of extra complication. You do not need to
add explicit devices for this.
If you want to make ipmi-bmc-* and ipmi-interface-* objects, you can
create them with object_new, set the properties manually with the
returned pointer and add the object to the parent with
object_property_add_child.
Otherwise, it should also be fine if you just add them as function pointers.
Either way also lets you keep the ownership of the chardev in the ISA
IPMI device, and avoids that "info qtree" looks strange and does not
show the chardev you assigned.
Paolo
- [Qemu-devel] [PATCH 03/16] vl: Move init_timer_alarm() earlier, (continued)
- [Qemu-devel] [PATCH 03/16] vl: Move init_timer_alarm() earlier, minyard, 2012/07/15
- [Qemu-devel] [PATCH 14/16] IPMI: Add a BT low-level interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 16/16] IPMI: Add an external connection simulation interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 05/16] qemu-char: Allow a chardev to reconnect if disconnected, minyard, 2012/07/15
- [Qemu-devel] [PATCH 06/16] qemu-char: Fix a race reporting opens and closes, minyard, 2012/07/15
- [Qemu-devel] [PATCH 04/16] qemu-char: Allocate CharDriverState in qemu_chr_new_from_opts, minyard, 2012/07/15
- [Qemu-devel] [PATCH 11/16] Add a base IPMI interface, minyard, 2012/07/15
- [Qemu-devel] [PATCH 09/16] qdev: Add a pre-firmware init capability, minyard, 2012/07/15
- [Qemu-devel] [PATCH 08/16] qemu-char: Close fd at end of file, minyard, 2012/07/15
- [Qemu-devel] [PATCH 12/16] IPMI: Add a PC ISA type structure, minyard, 2012/07/15
- Re: [Qemu-devel] [PATCH 12/16] IPMI: Add a PC ISA type structure,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 13/16] IPMI: Add a KCS low-level interface, minyard, 2012/07/15
- Re: [Qemu-devel] [PATCH 01/16] smbios: Add a function to directly add an entry, Corey Minyard, 2012/07/15
- Re: [Qemu-devel] [PATCH 01/16] smbios: Add a function to directly add an entry, Eric Blake, 2012/07/16