Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100 Root Port

From	Sinan Kaya <okaya@codeaurora.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100 Root Port
Date	2017-07-12 15:10 +0200
Message-ID	<u2pyW-cE-39@gated-at.bofh.it> (permalink)
References	<u25gS-41E-5@gated-at.bofh.it> <u29ua-6y9-21@gated-at.bofh.it> <u29Nv-6TI-7@gated-at.bofh.it> <u2agx-73x-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Bjorn,

On 7/11/2017 4:39 PM, Bjorn Helgaas wrote:
> My proposal handles endpoints, too.  The pci_walk_bus() in the quirk
> handles all devices we've already enumerated, and all devices we'll
> enumerate in the future are handled in pci_configure_device().

Code clears the endpoint's extended tag capability only if a quirky host
bridge is found. 

The question here was 

"what if you have an endpoint, it may declare extended tags capability
and has a bug even though the host bridge is just fine"

Code will enable extended tags on both the host bridge and endpoint
if it is supported.

The host bridge will start generating 256 tags towards the endpoint
but endpoint is unable to catch up with it. 

Same thing is possible with two endpoints that try to do peer-to-peer
communication. The first endpoint may generate 256 requests, second
endpoint may not handle it.

Again, this is a hypothetical condition with no known endpoints. I
suggest we deal with this when time comes.

Sinan 

-- 
Sinan Kaya
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.

Thread

[PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100 Root Port Sinan Kaya <okaya@codeaurora.org> - 2017-07-11 17:30 +0200
  Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Bjorn Helgaas <helgaas@kernel.org> - 2017-07-11 22:00 +0200
    Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Sinan Kaya <okaya@codeaurora.org> - 2017-07-11 22:20 +0200
      Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Bjorn Helgaas <helgaas@kernel.org> - 2017-07-11 22:50 +0200
        Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Sinan Kaya <okaya@codeaurora.org> - 2017-07-12 15:10 +0200
          Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Bjorn Helgaas <helgaas@kernel.org> - 2017-07-12 21:50 +0200
            Re: [PATCH V3 1/2] PCI: Add Extended Tags quirk for Broadcom HT2100  Root Port Sinan Kaya <okaya@codeaurora.org> - 2017-07-12 21:50 +0200

csiph-web