[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification
From: |
Benjamin Herrenschmidt |
Subject: |
Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification |
Date: |
Fri, 16 Sep 2011 12:51:31 -0300 |
> I've done some readings about this problem today, and I think I've got
> an idea what might be wrong here - seems like a bug in SLOF to me.
>
> First, according to the SLOF source code, it seems to me that its
> intention is to to scan target IDs, not channels (but as I haven't
> written that part, I am not 100% sure here).
>
> Then I compared how Linux and SLOF fill the 64-bit LUN field in the
> SRP_CMD request structure, and they both fill in the target ID at the
> same location - but Linux is additionally setting an additional bit in
> first byte (see the "lun_from_dev" function in ibmvscsi.c of the
> kernel).
>
> Looking at the "SCSI Architecture Model" specification, this additional
> bit is used to select the "Logical unit addressing method" instead of
> the "Peripheral device addressing method" that SLOF currently uses - and
> the "Logical unit addressing method" sounds more reasonable to me when
> looking at the places where SLOF tries to fill in the target ID.
>
> So I suggest that I change SLOF to also use the "Logical unit
> addressing method" like Linux does, which should result in the fact that
> SLOF tries to scan the target IDs instead of the channels/bus IDs.
.../...
Note that in addition to that, the PAPR spec specifies only one
"device" (whatever that means) per vscsi instance.
In fact, if we are ever to support proper sg, we need that, because the
way our vscsi client driver works in linux, we can only pass one max
request size down to the client from qemu, and without that information,
sg cannot be done reliably.
Cheers,
Ben.
- [Qemu-ppc] [PATCH 27/58] device tree: dont fail operations, (continued)
- [Qemu-ppc] [PATCH 27/58] device tree: dont fail operations, Alexander Graf, 2011/09/14
- [Qemu-ppc] [PATCH 58/58] KVM: Update kernel headers, Alexander Graf, 2011/09/14
- [Qemu-ppc] [PATCH 40/58] PPC: Fix sync instructions problem in SMP, Alexander Graf, 2011/09/14
- [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Alexander Graf, 2011/09/14
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, David Gibson, 2011/09/14
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Paolo Bonzini, 2011/09/15
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, David Gibson, 2011/09/15
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Paolo Bonzini, 2011/09/16
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Thomas Huth, 2011/09/16
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Paolo Bonzini, 2011/09/16
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification,
Benjamin Herrenschmidt <=
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Thomas Huth, 2011/09/19
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Paolo Bonzini, 2011/09/19
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, David Gibson, 2011/09/16
- Re: [Qemu-ppc] [PATCH 01/58] spapr: proper qdevification, Paolo Bonzini, 2011/09/19
[Qemu-ppc] [PATCH 29/58] MPC8544DS: Remove CPU nodes, Alexander Graf, 2011/09/14
[Qemu-ppc] [PATCH 56/58] PPC: Fix via-cuda memory registration, Alexander Graf, 2011/09/14
[Qemu-ppc] [PATCH 06/58] PPC: Extend MPIC MMIO range, Alexander Graf, 2011/09/14
[Qemu-ppc] [PATCH 54/58] openpic: Unfold write_IRQreg, Alexander Graf, 2011/09/14
[Qemu-ppc] [PATCH 18/58] PPC: E500: Remove mpc8544_copy_soc_cell, Alexander Graf, 2011/09/14
[Qemu-ppc] [PATCH 07/58] PPC: Fix IPI support in MPIC, Alexander Graf, 2011/09/14