[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCHv8 00/16] boot order specification
From: |
Gleb Natapov |
Subject: |
[Qemu-devel] Re: [PATCHv8 00/16] boot order specification |
Date: |
Sat, 11 Dec 2010 20:02:23 +0200 |
On Sat, Dec 11, 2010 at 05:19:01PM +0000, Blue Swirl wrote:
> >> What should we do with
> >> address@hidden vs address@hidden
> > There is no available IDE OF binding spec, so I when with the way
> > OpenBIOS reports ata on qemu-x86. I have no idea what 600 in address@hidden
> > may mean, but looking at g3_beige_300.html there is no such node there
> > and looking at any other device tree in
> > http://penguinppc.org/historical/dev-trees-html/
> > I haven't found one that use this kind of addressing for pci-ata.
> > http://penguinppc.org/historical/dev-trees-html/g3bw_400.html for
> > instance has address@hidden/address@hidden/address@hidden/ata-4.
> > address@hidden kind of
> > addressing is used by devices on mac-io bus which I do not think we
> > emulate in qemu. So it looks like OpneBIOS is wrong here.
>
> We have PMAC IDE, but this device is CMD646, so mac-io bus addressing
> rules should not be used.
>
So you agree that OpenBIOS is wrong here?
> In this tree there are two disks connected to CMD646, named
> /address@hidden/address@hidden/address@hidden/ata-4/disk and
> /address@hidden/address@hidden/address@hidden/ata-4/address@hidden:
> http://penguinppc.org/historical/dev-trees-html/g4_pci_350.html
You are saying that qemu creates paths like:
/address@hidden/address@hidden/address@hidden/address@hidden
/address@hidden/address@hidden/address@hidden/address@hidden
I do not understand why qemu creates node address@hidden It should be
address@hidden
according to the code. I'll look at why unit-address is incorrect for
the node. But assuming that this problem is fixed then paths created by
qemu is very similar to the paths in g4_pci_350.html. It looks like in
g4_pci_350.html they omit unit address if it is zero.
--
Gleb.
- [Qemu-devel] [PATCHv8 04/16] Add get_fw_dev_path callback to ISA bus in qdev., (continued)
- [Qemu-devel] [PATCHv8 04/16] Add get_fw_dev_path callback to ISA bus in qdev., Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 12/16] Add bootindex parameter to net/block/fd device, Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 07/16] Add get_fw_dev_path callback for system bus., Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 06/16] Add get_fw_dev_path callback to IDE bus., Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 13/16] Change fw_cfg_add_file() to get full file path as a parameter., Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 10/16] Add get_fw_dev_path callback for usb bus., Gleb Natapov, 2010/12/08
- [Qemu-devel] [PATCHv8 01/16] Introduce fw_name field to DeviceInfo structure., Gleb Natapov, 2010/12/08
- [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Blue Swirl, 2010/12/11
- [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Gleb Natapov, 2010/12/11
- [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Blue Swirl, 2010/12/11
- [Qemu-devel] Re: [PATCHv8 00/16] boot order specification,
Gleb Natapov <=
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Gleb Natapov, 2010/12/11
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Blue Swirl, 2010/12/11
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Gleb Natapov, 2010/12/11
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Benjamin Herrenschmidt, 2010/12/11
- [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Blue Swirl, 2010/12/11
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Benjamin Herrenschmidt, 2010/12/11
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Andreas Färber, 2010/12/14
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Benjamin Herrenschmidt, 2010/12/14
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Alexander Graf, 2010/12/14
- Re: [Qemu-devel] Re: [PATCHv8 00/16] boot order specification, Andreas Färber, 2010/12/17