2010/11/16 Gleb Natapov <gleb@xxxxxxxxxx>:
> On Mon, Nov 15, 2010 at 08:29:24PM +0000, Blue Swirl wrote:
>> 2010/11/15 Gleb Natapov <gleb@xxxxxxxxxx>:
>> > On Sun, Nov 14, 2010 at 10:50:13PM +0000, Blue Swirl wrote:
>> >> On Sun, Nov 14, 2010 at 3:39 PM, Gleb Natapov <gleb@xxxxxxxxxx> wrote:
>> >> >
>> >> > Signed-off-by: Gleb Natapov <gleb@xxxxxxxxxx>
>> >> > ---
>> >> > Âhw/fw_cfg.c | Â 14 ++++++++++++++
>> >> > Âhw/fw_cfg.h | Â Â4 +++-
>> >> > Âsysemu.h  Â|  Â1 +
>> >> > Âvl.c    Â|  51 +++++++++++++++++++++++++++++++++++++++++++++++++++
>> >> > Â4 files changed, 69 insertions(+), 1 deletions(-)
>> >> >
>> >> > diff --git a/hw/fw_cfg.c b/hw/fw_cfg.c
>> >> > index 7b9434f..f6a67db 100644
>> >> > --- a/hw/fw_cfg.c
>> >> > +++ b/hw/fw_cfg.c
>> >> > @@ -53,6 +53,7 @@ struct FWCfgState {
>> >> > Â Â FWCfgFiles *files;
>> >> > Â Â uint16_t cur_entry;
>> >> > Â Â uint32_t cur_offset;
>> >> > + Â ÂNotifier machine_ready;
>> >> > Â};
>> >> >
>> >> > Âstatic void fw_cfg_write(FWCfgState *s, uint8_t value)
>> >> > @@ -315,6 +316,15 @@ int fw_cfg_add_file(FWCfgState *s, Âconst char *filename, uint8_t *data,
>> >> > Â Â return 1;
>> >> > Â}
>> >> >
>> >> > +static void fw_cfg_machine_ready(struct Notifier* n)
>> >> > +{
>> >> > + Â Âuint32_t len;
>> >> > + Â Âchar *bootindex = get_boot_devices_list(&len);
>> >> > +
>> >> > + Â Âfw_cfg_add_bytes(container_of(n, FWCfgState, machine_ready),
>> >> > + Â Â Â Â Â Â Â Â Â Â FW_CFG_BOOTINDEX, (uint8_t*)bootindex, len);
>> >> > +}
>> >> > +
>> >> > ÂFWCfgState *fw_cfg_init(uint32_t ctl_port, uint32_t data_port,
>> >> > Â Â Â Â Â Â Â Â Â Â Â Â target_phys_addr_t ctl_addr, target_phys_addr_t data_addr)
>> >> > Â{
>> >> > @@ -343,6 +353,10 @@ FWCfgState *fw_cfg_init(uint32_t ctl_port, uint32_t data_port,
>> >> > Â Â fw_cfg_add_i16(s, FW_CFG_MAX_CPUS, (uint16_t)max_cpus);
>> >> > Â Â fw_cfg_add_i16(s, FW_CFG_BOOT_MENU, (uint16_t)boot_menu);
>> >> >
>> >> > +
>> >> > + Â Âs->machine_ready.notify = fw_cfg_machine_ready;
>> >> > + Â Âqemu_add_machine_init_done_notifier(&s->machine_ready);
>> >> > +
>> >> > Â Â return s;
>> >> > Â}
>> >> >
>> >> > diff --git a/hw/fw_cfg.h b/hw/fw_cfg.h
>> >> > index 856bf91..4d61410 100644
>> >> > --- a/hw/fw_cfg.h
>> >> > +++ b/hw/fw_cfg.h
>> >> > @@ -30,7 +30,9 @@
>> >> >
>> >> > Â#define FW_CFG_FILE_FIRST Â Â Â 0x20
>> >> > Â#define FW_CFG_FILE_SLOTS Â Â Â 0x10
>> >> > -#define FW_CFG_MAX_ENTRY Â Â Â Â(FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS)
>> >> > +#define FW_CFG_FILE_LAST_SLOT Â (FW_CFG_FILE_FIRST+FW_CFG_FILE_SLOTS)
>> >> > +#define FW_CFG_BOOTINDEX Â Â Â Â(FW_CFG_FILE_LAST_SLOT + 1)
>> >> > +#define FW_CFG_MAX_ENTRY Â Â Â ÂFW_CFG_BOOTINDEX
>> >>
>> >> This should be
>> >> #define FW_CFG_MAX_ENTRY Â Â Â Â(FW_CFG_BOOTINDEX + 1)
>> >> because the check is like this:
>> >> Â Â if ((key & FW_CFG_ENTRY_MASK) >= FW_CFG_MAX_ENTRY) {
>> >> Â Â Â Â s->cur_entry = FW_CFG_INVALID;
>> >>
>> > Yeah, will fix.
>> >
>> >> With that change, I got the bootindex passed to OpenBIOS:
>> >> OpenBIOS for Sparc64
>> >> Configuration device id QEMU version 1 machine id 0
>> >> kernel cmdline
>> >> CPUs: 1 x SUNW,UltraSPARC-IIi
>> >> UUID: 00000000-0000-0000-0000-000000000000
>> >> bootindex num_strings 1
>> >> bootindex /pbm@000001fe00000000/ide@5/drive@1/disk@0
>> >>
>> >> The device path does not match exactly, but it's close:
>> >> /pci@1fe,0/pci-ata@5/ide1@600/disk@0
>> >
>> > pbm->pci should be solvable by the patch at the end. Were in the spec
>> > it is allowed to abbreviate 1fe00000000 as 1fe,0? Spec allows to drop
>> > starting zeroes but TARGET_FMT_plx definition in targphys.h has 0 after
>> > %. I can define another one without leading zeroes. Can you suggest
>> > a name?
>>
>> I think OpenBIOS for Sparc64 is not correct here, so it may be a bad
>> reference architecture. OBP on a real Ultra-5 used a path like this:
>> /pci@1f,0/pci@1,1/ide@3/disk@0,0
>>
>> pci@1f,0 specifies the PCI host bridge at UPA bus port ID of 0x1f.
> According to device name qemu creates pci controller is memory mapped
> at address 1fe00000000 and by looking at the code I can see that this
> is indeed the case. How is UPA naming works?
No idea.
>> pci@1,1 specifies a PCI-PCI bridge.
>>
>> >ÂTARGET_FMT_lx is poisoned. As of ATA there is no open firmware
>> > binding spec for ATA, so everyone does what he pleases. I based my
>> > implementation on what open firmware showing when running on qemu x86.
>> > "pci-ata" should be "ide" according to PCI binding spec :)
>>
>> Yes, for example there is no ATA in the Ultra-5 tree but in UltraAX it exists:
>> /pci@1f,4000/ide@3/ata@0,0/cmdk@0,0
>>
>> > diff --git a/hw/apb_pci.c b/hw/apb_pci.c
>> > index c619112..643aa49 100644
>> > --- a/hw/apb_pci.c
>> > +++ b/hw/apb_pci.c
>> > @@ -453,6 +453,7 @@ static PCIDeviceInfo pbm_pci_host_info = {
>> >
>> > Âstatic SysBusDeviceInfo pbm_host_info = {
>> > Â Â .qdev.name = "pbm",
>> > + Â Â.qdev.fw_name = "pci",
>>
>> Perhaps the FW path should use device class names if no name is specified.
> What do you mean by "device class name". We can do something like this:
> if (dev->child_bus.lh_first)
> Â Â Â Âreturn dev->child_bus.lh_first->info->name;
>
> i.e if there is child bus use its bus name as fw name. This will make
> all pci devices to have "pci" as fw name automatically. The problem is
> that theoretically same device can provide different buses.
I meant PCI class name, like "display" for display controllers,
"network" for NICs etc.
>> I'll try Sparc32 to see how this fits there.
Except bootindex is not implemented for SCSI.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
