Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

On Mon, Oct 16, 2017 at 04:23:20PM -0500, Bjorn Helgaas wrote: Hi Bjorn, > [+cc David, Manish] > > Please use a subject line that tells more about what's going on. > "Update quirk" doesn't really convey any useful information. > Something like "Apply Cavium ThunderX ACS quirk only to Root

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

On Mon, Oct 16, 2017 at 04:23:20PM -0500, Bjorn Helgaas wrote: Hi Bjorn, > [+cc David, Manish] > > Please use a subject line that tells more about what's going on. > "Update quirk" doesn't really convey any useful information. > Something like "Apply Cavium ThunderX ACS quirk only to Root

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

[+cc David, Manish] Please use a subject line that tells more about what's going on. "Update quirk" doesn't really convey any useful information. Something like "Apply Cavium ThunderX ACS quirk only to Root Ports". On Wed, Sep 27, 2017 at 11:20:39AM -0700, Vadim Lomovtsev wrote: > This commit

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

[+cc David, Manish] Please use a subject line that tells more about what's going on. "Update quirk" doesn't really convey any useful information. Something like "Apply Cavium ThunderX ACS quirk only to Root Ports". On Wed, Sep 27, 2017 at 11:20:39AM -0700, Vadim Lomovtsev wrote: > This commit

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Bjorn, On 27.09.17 11:20:39, Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk applicable only to Cavium > ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities > in terms of no ACS support advertisement. However, the RTL internally > implements similar

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Bjorn, On 27.09.17 11:20:39, Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk applicable only to Cavium > ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities > in terms of no ACS support advertisement. However, the RTL internally > implements similar

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi Bjorn, Would you please consider to take this patch ? It is required to provide correct ACS mask for CN8xxx PCI root ports. WBR, Vadim On Wed, Sep 27, 2017 at 02:18:54PM -0600, Alex Williamson wrote: > On Wed, 27 Sep 2017 11:20:39 -0700 > Vadim Lomovtsev

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi Bjorn, Would you please consider to take this patch ? It is required to provide correct ACS mask for CN8xxx PCI root ports. WBR, Vadim On Wed, Sep 27, 2017 at 02:18:54PM -0600, Alex Williamson wrote: > On Wed, 27 Sep 2017 11:20:39 -0700 > Vadim Lomovtsev wrote: > > > This commit makes

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi Bjorn, On Wed, Sep 27, 2017 at 02:18:54PM -0600, Alex Williamson wrote: > On Wed, 27 Sep 2017 11:20:39 -0700 > Vadim Lomovtsev wrote: > > > This commit makes Cavium PCI ACS quirk applicable only to Cavium > > ThunderX (CN8XXX) family PCIE Root Ports which

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi Bjorn, On Wed, Sep 27, 2017 at 02:18:54PM -0600, Alex Williamson wrote: > On Wed, 27 Sep 2017 11:20:39 -0700 > Vadim Lomovtsev wrote: > > > This commit makes Cavium PCI ACS quirk applicable only to Cavium > > ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities > > in

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

On Wed, 27 Sep 2017 11:20:39 -0700 Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk applicable only to Cavium > ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities > in terms of no ACS support advertisement. However, the

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

On Wed, 27 Sep 2017 11:20:39 -0700 Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk applicable only to Cavium > ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities > in terms of no ACS support advertisement. However, the RTL internally > implements similar

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi guys, I've found one typo (0xa00 instead of 0xa000 at code comment) and v6 has it fixed. I bring my apologies for that, could you please review this patch once again. WBR, Vadim On Wed, Sep 27, 2017 at 11:20:39AM -0700, Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk

Re: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

Hi guys, I've found one typo (0xa00 instead of 0xa000 at code comment) and v6 has it fixed. I bring my apologies for that, could you please review this patch once again. WBR, Vadim On Wed, Sep 27, 2017 at 11:20:39AM -0700, Vadim Lomovtsev wrote: > This commit makes Cavium PCI ACS quirk

[PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

This commit makes Cavium PCI ACS quirk applicable only to Cavium ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities in terms of no ACS support advertisement. However, the RTL internally implements similar protection as if ACS had completion/request redirection, upstream

[PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation

This commit makes Cavium PCI ACS quirk applicable only to Cavium ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities in terms of no ACS support advertisement. However, the RTL internally implements similar protection as if ACS had completion/request redirection, upstream