From: Nate Watterson <nwatters@codeaurora.org>
To: Sinan Kaya <okaya@codeaurora.org>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
linux-pci@vger.kernel.org, Robin Murphy <robin.murphy@arm.com>,
will.deacon@arm.com, linux-acpi@vger.kernel.org,
iommu@lists.linux-foundation.org, linux-arm-msm@vger.kernel.org,
bhelgaas@google.com, Tomasz Nowicki <tn@semihalf.com>,
Sricharan R <sricharan@codeaurora.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH V7 08/11] drivers: acpi: Handle IOMMU lookup failure with deferred probing or error
Date: Mon, 30 Jan 2017 09:23:29 -0500 [thread overview]
Message-ID: <5ba9f366d6e25397cdef8ad95b49e199@codeaurora.org> (raw)
In-Reply-To: <14751205-f034-7f0d-442a-854c3909425c@codeaurora.org>
On 2017-01-30 08:59, Sinan Kaya wrote:
> On 1/30/2017 7:22 AM, Robin Murphy wrote:
>> On 29/01/17 17:53, Sinan Kaya wrote:
>>> On 1/24/2017 7:37 AM, Lorenzo Pieralisi wrote:
>>>> [+hanjun, tomasz, sinan]
>>>>
>>>> It is quite a key patchset, I would be glad if they can test on
>>>> their
>>>> respective platforms with IORT.
>>>>
>>>
>>> Tested on top of 4.10-rc5.
>>>
>>> 1. Platform Hidma device passed dmatest
>>> 2. Seeing some USB stalls on a platform USB device.
>>> 3. PCIe NVME drive probed and worked fine with MSI interrupts after
>>> boot.
>>> 4. NVMe driver didn't probe following a hotplug insertion and
>>> received an
>>> SMMU error event during the insertion.
>>
>> What was the SMMU error - a translation/permission fault (implying the
>> wrong DMA ops) or a bad STE fault (implying we totally failed to tell
>> the SMMU about the device at all)?
>>
>
> root@ubuntu:/sys/bus/pci/slots/4# echo 0 > power
>
> [__204.698522]_iommu:_Removing_device_0003:01:00.0_from_group_0
> [ 204.708704] pciehp 0003:00:00.0:pcie004: Slot(4): Link Down
> [ 204.708723] pciehp 0003:00:00.0:pcie004: Slot(4): Link Down event
> ignored; already powering off
>
> root@ubuntu:/sys/bus/pci/slots/4#
>
> [__254.820440]_iommu:_Adding_device_0003:01:00.0_to_group_8
> [ 254.820599] nvme nvme0: pci function 0003:01:00.0
> [ 254.820621] nvme 0003:01:00.0: enabling device (0000 -> 0002)
> [ 261.948558] arm-smmu-v3 arm-smmu-v3.0.auto: event 0x0a received:
> [ 261.948561] arm-smmu-v3 arm-smmu-v3.0.auto: 0x000001000000000a
> [ 261.948563] arm-smmu-v3 arm-smmu-v3.0.auto: 0x0000000000000000
> [ 261.948564] arm-smmu-v3 arm-smmu-v3.0.auto: 0x0000000000000000
> [ 261.948566] arm-smmu-v3 arm-smmu-v3.0.auto: 0x0000000000000000
Looks like C_BAD_CD. Can you please try with:
iommu/arm-smmu-v3: Clear prior settings when updating STEs
> root@ubuntu:/sys/bus/pci/slots/4#
>
> root@ubuntu:/sys/bus/pci/slots/4#ls /dev/nvme*
> /dev/nvme0
>
> I should have seen /dev/nvme0n1 partition here.
>
>> Robin.
>>
>>>
>>> /sys/bus/pci/slots/4 #
>>> /sys/bus/pci/slots/4 # dmesg | grep nvme
>>> [ 14.041357] nvme nvme0: pci function 0003:01:00.0
>>> [ 198.399521] nvme nvme0: pci function 0003:01:00.0
>>> [__198.416232]_nvme_0003:01:00.0:_enabling_device_(0000_->_0002)
>>> [ 264.402216] nvme nvme0: I/O 228 QID 0 timeout, disable controller
>>> [ 264.402313] nvme nvme0: Identify Controller failed (-4)
>>> [ 264.421270] nvme nvme0: Removing after probe failure status: -5
>>> /sys/bus/pci/slots/4 #
>>>
>>>
>>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-pci"
>> in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
--
Qualcomm Datacenter Technologies, Inc. on behalf of Qualcomm
Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a
Linux
Foundation Collaborative Project.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2017-01-30 14:23 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20170123161926epcas2p1cf464a2978d89d0c1fdd2f7cc6d38317@epcas2p1.samsung.com>
2017-01-23 16:18 ` [PATCH V7 00/11] IOMMU probe deferral support Sricharan R
2017-01-23 16:18 ` [PATCH V7 01/11] iommu/of: Refactor of_iommu_configure() for error handling Sricharan R
2017-01-25 17:17 ` Tomasz Nowicki
2017-01-25 17:35 ` Robin Murphy
2017-01-25 18:13 ` Tomasz Nowicki
2017-01-27 18:00 ` Sricharan
2017-01-27 18:19 ` Robin Murphy
2017-01-30 7:00 ` Sricharan
2017-01-30 18:42 ` Robin Murphy
2017-01-31 13:11 ` Sricharan
2017-01-23 16:18 ` [PATCH V7 02/11] iommu/of: Prepare for deferred IOMMU configuration Sricharan R
2017-01-23 16:18 ` [PATCH V7 03/11] of: dma: Move range size workaround to of_dma_get_range() Sricharan R
2017-01-23 16:18 ` [PATCH V7 04/11] of: dma: Make of_dma_deconfigure() public Sricharan R
2017-01-23 16:18 ` [PATCH V7 05/11] ACPI/IORT: Add function to check SMMUs drivers presence Sricharan R
2017-01-23 16:18 ` [PATCH V7 06/11] of/acpi: Configure dma operations at probe time for platform/amba/pci bus devices Sricharan R
2017-01-28 21:08 ` Bjorn Helgaas
2017-01-30 7:09 ` Rafael J. Wysocki
2017-01-23 16:18 ` [PATCH V7 07/11] iommu: of: Handle IOMMU lookup failure with deferred probing or error Sricharan R
2017-01-28 21:03 ` Bjorn Helgaas
2017-01-30 8:01 ` Sricharan
2017-01-29 16:36 ` Sinan Kaya
2017-01-30 12:00 ` Sricharan
2017-01-23 16:18 ` [PATCH V7 08/11] drivers: acpi: " Sricharan R
2017-01-24 12:37 ` Lorenzo Pieralisi
2017-01-24 13:14 ` Hanjun Guo
2017-01-25 7:31 ` Sricharan
2017-01-29 17:53 ` Sinan Kaya
2017-01-30 12:22 ` Robin Murphy
2017-01-30 13:59 ` Sinan Kaya
2017-01-30 14:23 ` Nate Watterson [this message]
2017-01-30 14:33 ` Sinan Kaya
2017-01-30 14:38 ` Will Deacon
2017-01-30 14:54 ` Nate Watterson
2017-01-30 15:46 ` Sinan Kaya
2017-01-30 16:51 ` Lorenzo Pieralisi
2017-01-30 20:03 ` Sinan Kaya
2017-02-01 18:52 ` Lorenzo Pieralisi
2017-02-01 19:10 ` Sinan Kaya
2017-02-02 19:01 ` Nate Watterson
2017-02-03 3:37 ` Hanjun Guo
2017-02-03 3:37 ` Sricharan
2017-01-23 16:18 ` [PATCH V7 09/11] arm64: dma-mapping: Remove the notifier trick to handle early setting of dma_ops Sricharan R
2017-01-28 21:06 ` Bjorn Helgaas
2017-01-30 8:04 ` Sricharan
2017-01-23 16:18 ` [PATCH V7 10/11] iommu/arm-smmu: Clean up early-probing workarounds Sricharan R
2017-01-25 17:28 ` Tomasz Nowicki
2017-01-28 21:07 ` Bjorn Helgaas
2017-01-30 8:05 ` Sricharan
2017-01-23 16:18 ` [PATCH V7 11/11] ACPI/IORT: Remove linker section for IORT entries probing Sricharan R
2017-01-24 7:40 ` [PATCH V7 00/11] IOMMU probe deferral support Marek Szyprowski
2017-01-24 11:15 ` Sricharan
2017-01-25 4:33 ` Hanjun Guo
2017-01-25 7:33 ` Sricharan
2017-01-28 21:10 ` Bjorn Helgaas
2017-01-30 9:20 ` Sricharan
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=5ba9f366d6e25397cdef8ad95b49e199@codeaurora.org \
--to=nwatters@codeaurora.org \
--cc=bhelgaas@google.com \
--cc=iommu@lists.linux-foundation.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=lorenzo.pieralisi@arm.com \
--cc=okaya@codeaurora.org \
--cc=robin.murphy@arm.com \
--cc=sricharan@codeaurora.org \
--cc=tn@semihalf.com \
--cc=will.deacon@arm.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).