From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36168) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VWnl1-0001ph-P4 for qemu-devel@nongnu.org; Thu, 17 Oct 2013 09:37:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VWnku-0003ZJ-EP for qemu-devel@nongnu.org; Thu, 17 Oct 2013 09:36:55 -0400 Received: from mail-pb0-f50.google.com ([209.85.160.50]:34272) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VWnku-0003Z9-7V for qemu-devel@nongnu.org; Thu, 17 Oct 2013 09:36:48 -0400 Received: by mail-pb0-f50.google.com with SMTP id uo15so901553pbc.23 for ; Thu, 17 Oct 2013 06:36:46 -0700 (PDT) Message-ID: <525FE7E8.5050902@ozlabs.ru> Date: Fri, 18 Oct 2013 00:36:40 +1100 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <525E1E5C.4090002@ozlabs.ru> <525E33DA.8040106@redhat.com> <525E4759.10402@ozlabs.ru> <525FDA47.9050306@ozlabs.ru> <525FDE06.1000609@redhat.com> In-Reply-To: Content-Type: text/plain; charset=KOI8-R Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] virtio-blk-pci: how to tell if it is CD or HDD? List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf , Paolo Bonzini Cc: "qemu-devel@nongnu.org" , Nikunj A Dadhania On 10/18/2013 12:02 AM, Alexander Graf wrote: > > On 17.10.2013, at 14:54, Paolo Bonzini wrote: > >> Il 17/10/2013 14:38, Alexey Kardashevskiy ha scritto: >>> qdev_get_fw_dev_path: >>> /spapr-vio-bridge/spapr-vscsi/channel@0/disk@3,2 suffix=(null) >>> /spapr-vio-bridge/spapr-vscsi/channel@0/disk@3,1 suffix=(null) >> >> You need to implement qdev_fw_get_path to change >> >> spapr-vio-bridge -> vdevice >> spapr-vscsi -> v-scsi@REG >> >>> /pci@800000020000000/ethernet@1 suffix=/ethernet-phy@0 >> >> The extra suffix is not a problem since you can parse a prefix successfully. >> >>> /pci@800000020000000/scsi@0/channel@0/disk@3,2 suffix=(null) >>> /pci@800000020000000/scsi@0/channel@0/disk@3,1 suffix=(null) >> >> I guess this is virtio-scsi. >> >>> SLOF: >>> 0 > devalias >>> cdrom123 : /pci@800000020000000/scsi@0/disk@103000100000000 >>> cdrom12 : /pci@800000020000000/scsi@0/disk@103000200000000 >>> hvterm : /vdevice/vty@71000100 >>> net : /pci@800000020000000/ethernet@1 >>> scsi : /vdevice/v-scsi@71000001 >>> cdrom1 : /vdevice/v-scsi@71000001/disk@8301000000000000 >>> cdrom : /vdevice/v-scsi@71000001/disk@8302000000000000 >>> nvram : /vdevice/nvram@71000000 ok >>> >>> >>> In ideal world I would want to get in QEMU what SLOF can understand and >>> pass this to SLOF. But QEMU APIs return something which cannot be converted >>> straight away. >>> >>> Or I could simply put bootindex to the device tree nodes (as >>> "qemu,bootindex") but in this case "wildcard" nodes support fails as there >>> is just a single node "/vdevice/v-scsi@71000001/disk" in the device tree >>> for all LUNs. And we definitely do not want to create nodes for all disk >>> devices. >>> >>> Or I can implement a "smart" converter from QEMU strings to OF pathnames. >>> >>> Or I can implement third set of callbacks, something like qdev_OF_dev_path(). >>> >>> Or not support "bootindex" at all. >>> >>> All possibilities suck but which one sucks less? :) Thanks! >> >> In general, try to make QEMU produce SLOF APIs by modifying the devices >> that instantiate the buses. > > But please make sure to not block the path for non-SLOF machines. -M mac99 should still be able to get different path names for PCI devices for example. Ok. Then question for you. I need to change root PHB name from spapr-pci-host-bridge (which is a class name) to something reasonable. For example, PCIHostBridgeClass::root_bus_path does the job. But this part of OF path is made by sysbus_get_fw_dev_path() from hw/core/sysbus.c which does not know about PCI. So it cannot call PCI callbacks. I workarounded this by setting PHB's "id" to what sysbus_get_fw_dev_path() returns but this is barely the proper fix. I could initialize my PHBs s->mmio[0].addr to BUID and sysbus_get_fw_dev_path() would return what I need but it won't be MMIO by any mean and I do not really want to be responsible for all side effect it may have :) So how to fix it correctly? Thanks. -- Alexey