All of lore.kernel.org
 help / color / mirror / Atom feed
From: manish jaggi <manishjaggi.oss@gmail.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Ian Campbell <Ian.Campbell@citrix.com>,
	Vijay Kilari <vijay.kilari@gmail.com>,
	Prasun Kapoor <prasun.Kapoor@caviumnetworks.com>,
	manish.jaggi@caviumnetworks.com,
	Julien Grall <julien.grall@linaro.org>,
	xen-devel <xen-devel@lists.xen.org>,
	psawargaonkar@linaro.org, anup.patel@linaro.org
Subject: Re: [RFC + Queries] Flow of PCI passthrough in ARM
Date: Wed, 24 Sep 2014 16:23:43 +0530	[thread overview]
Message-ID: <CAAiw7JkmZtgSup0bD01c=RtFk0hnq_8YP8kjp66ZORuCzPqwZQ@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1409221057490.29862@kaball.uk.xensource.com>

On 22 September 2014 16:15, Stefano Stabellini
<stefano.stabellini@eu.citrix.com> wrote:
> On Thu, 18 Sep 2014, manish jaggi wrote:
>> Hi,
>> Below is the flow I am working on, Please provide your comments, I
>> have a couple of queries as well..
>>
>> a) Device tree has smmu nodes and each smmu node has the mmu-master property.
>> In our Soc DT the mmu-master is a pcie node in device tree.
>
> Do you mean that both the smmu nodes and the pcie node have the
> mmu-master property? The pcie node is the pcie root complex, right?
>
pci-node is the pcie root complex. pci node is the mmu master in smmu node.

  smmu1@0x8310,00000000 {
...

                 mmu-masters = <&pcie1 0x100>;
         };

>> b) Xen parses the device tree and prepares a list which stores the pci
>> device tree node pointers. The order in device tree is mapped to
>> segment number in subsequent calls. For eg 1st pci node found is
>> segment 0, 2nd segment 1
>
> What's a segment number? Something from the PCI spec?
> If you have several pci nodes on device tree, does that mean that you
> have several different pcie root complexes?
>
yes.
segment is the pci rc number.
>
>> c) During SMMU init the pcie nodes in DT are saved as smmu masters.
>
> At this point you should also be able to find via DT the stream-id range
> supported by each SMMU and program the SMMU with them, assigning
> everything to dom0.
Currently pcie enumeration is not done in xen, it is done in dom0.
>
>
>> d) Dom0 Enumerates PCI devices, calls hypercall PHYSDEVOP_pci_device_add.
>>  - In Xen the SMMU iommu_ops add_device is called. I have implemented
>> the add_device function.
>> - In the add_device function
>>  the segment number is used to locate the device tree node pointer of
>> the pcie node which helps to find out the corresponding smmu.
>> - In the same PHYSDEVOP the BAR regions are mapped to Dom0.
>>
>> Note: The current SMMU driver maps the complete Domain's Address space
>> for the device in SMMU hardware.
>>
>> The above flow works currently for us.
>
> It would be nice to be able to skip d): in a system where all dma capable
> devices are behind smmus, we should be capable of booting dom0 without
> the 1:1 mapping hack. If we do that, it would be better to program the
> smmus before booting dom0. Otherwise there is a risk that dom0 is going
> to start using these devices and doing dma before we manage to secure
> the devices via smmus.
>
In our current case we are programming smmu in
PHYSDEVOP_pci_device_add flow so before the domain 0 accesses the
device it is mapped, otherwise xen gets a SMMU fault.

> Of course we can do that if there are no alternatives. But in our case
> we should be able to extract the stream-ids from device tree and program
> the smmus right away, right?  Do we really need to wait for dom0 to call
> PHYSDEVOP_pci_device_add? We could just assign everything to dom0 for a
> start.
>
We cannot get streamid from device tree as enumeration is done for the same.

> I would like to know from the x86 guys, if this is really how it is
> supposed to work on PVH too. Do we rely on PHYSDEVOP_pci_device_add to
> program the IOMMU?
>
>
I was waiting but no one has commented

>> Now when I call pci-assignable-add I see that the iommu_ops
>> remove_device in smmu driver is not called. If that is not called the
>> SMMU would still have the dom0 address space mappings for that device
>>
>> Can you please suggest the best place (kernel / xl-tools) to put the
>> code which would call the remove_device in iommu_opps in the control
>> flow from pci-assignable-add.
>>
>> One way I see is to introduce a DOMCTL_iommu_remove_device in
>> pci-assignable-add / pci-detach and DOMCTL_iommu_add_device in
>> pci-attach. Is that a valid approach  ?
>
> I am not 100% sure, but I think that before assigning a PCI device to
> another guest, you are supposed to bind the device to xen-pciback (see
> drivers/xen/xen-pciback, also see
> http://wiki.xen.org/wiki/Xen_PCI_Passthrough). The pciback driver is
> going hide the device from dom0 and as a consequence
> drivers/xen/pci.c:xen_remove_device ends up being called, that issues a
> PHYSDEVOP_pci_device_remove hypercall.

xen_remove_device is not called at all. in pci-attach
iommu_ops->assign_device is called.
In Xen the nomenclature is confusing and no comments are there is iommu.h
iommu_ops.add_device is when dom0 issues hypercall
iommu_ops.assign_dt_device is when xen attaches a device tree device to dom0
iommu_ops.assign_device is when xl pci-attach is called
iommu_ops.reassign_device is called when xl pci-detach is called

As of now we are able to assign devices to domU and driver in domU is
running, we did some hacks like
a) in xen pci front driver bus->msi is assigned to its msi_chip

---- pcifront_scan_root()
...
b = pci_scan_bus_parented(&pdev->xdev->dev, bus,
                  &pcifront_bus_ops, sd);
    if (!b) {
        dev_err(&pdev->xdev->dev,
            "Error creating PCI Frontend Bus!\n");
        err = -ENOMEM;
        pci_unlock_rescan_remove();
        goto err_out;
    }

    bus_entry->bus = b;
+        msi_node = of_find_compatible_node(NULL,NULL, "arm,gic-v3-its");
+        if(msi_node) {
+            b->msi = of_pci_find_msi_chip_by_node(msi_node);
+            if(!b->msi) {
+               printk(KERN_ERR"Unable to find bus->msi node \r\n");
+               goto err_out;
+            }
+        }else {
+               printk(KERN_ERR"Unable to find arm,gic-v3-its
compatible node \r\n");
+               goto err_out;
+        }

----

using this the ITS emulation code in xen is able to trap ITS command
writes by driver.
But we are facing a problem now, where your help is needed

The StreamID is generated by segment: bus : device: number which is
fed as DevID in ITS commands. In Dom0 the streamID is correctly
generated but in domU the Stream ID for a passthrough device is
0:0:0:0 now when emulating this in Xen it is a problem as xen does not
know how to get the physical stream id.

(Eg: xl pci-attach 1 0001:00:05.0
DomU has the device but in DomU the id is 0000:00:00.0.)

Could you suggest how to go about this.

-Regards
Manish

  parent reply	other threads:[~2014-09-24 10:53 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-18 11:34 [RFC + Queries] Flow of PCI passthrough in ARM manish jaggi
2014-09-22 10:45 ` Stefano Stabellini
2014-09-22 11:09   ` Ian Campbell
2014-09-24 10:56     ` manish jaggi
2014-09-24 10:53   ` manish jaggi [this message]
2014-09-24 12:13     ` Jan Beulich
2014-09-24 14:10     ` Stefano Stabellini
2014-09-24 18:32       ` manish jaggi
2014-09-25 10:27         ` Stefano Stabellini
2014-10-01 10:37           ` manish jaggi
2014-10-02 16:41             ` Stefano Stabellini
2014-10-02 16:59               ` Stefano Stabellini
2014-10-03  9:01                 ` Ian Campbell
2014-10-03  9:33                   ` manish jaggi
2014-10-03  9:32                 ` manish jaggi
2014-10-06 11:05                   ` manish jaggi
2014-10-06 14:11                     ` Stefano Stabellini
2014-10-06 15:38                       ` Ian Campbell
2014-10-06 17:39                         ` manish jaggi
2014-10-06 17:39                       ` manish jaggi
2014-10-07 18:17                         ` Stefano Stabellini
2014-10-08 11:46                           ` manish jaggi
2014-10-08 12:46                             ` Konrad Rzeszutek Wilk
2014-10-08 13:37                               ` manish jaggi
2014-10-08 13:45                                 ` Ian Campbell
2014-10-08 13:47                                   ` manish jaggi
2014-10-08 13:58                                     ` Ian Campbell
2014-10-08 14:51                                     ` Konrad Rzeszutek Wilk
2014-10-20 13:30                                       ` manish jaggi
2014-10-20 14:54                                         ` Stefano Stabellini
2014-11-06 15:28                                           ` manish jaggi
2014-11-06 15:48                                             ` Stefano Stabellini
2014-11-06 15:55                                               ` manish jaggi
2014-11-06 16:02                                                 ` Julien Grall
2014-11-06 16:07                                                   ` Stefano Stabellini
2014-11-06 16:20                                                     ` manish jaggi
2014-11-07 10:29                                                       ` Julien Grall
2014-11-06 19:41                                             ` Konrad Rzeszutek Wilk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAAiw7JkmZtgSup0bD01c=RtFk0hnq_8YP8kjp66ZORuCzPqwZQ@mail.gmail.com' \
    --to=manishjaggi.oss@gmail.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=anup.patel@linaro.org \
    --cc=julien.grall@linaro.org \
    --cc=manish.jaggi@caviumnetworks.com \
    --cc=prasun.Kapoor@caviumnetworks.com \
    --cc=psawargaonkar@linaro.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=vijay.kilari@gmail.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.