Re: [PATCH v7 2/3] PCI: Enable PCIe Relaxed Ordering if supported

From: Alex Williamson
Date: Mon Jul 24 2017 - 11:05:37 EST


On Sat, 22 Jul 2017 12:19:38 +0800
Ding Tianhong <dingtianhong@xxxxxxxxxx> wrote:

> Hi Sinan, Bjorn:
>
> On 2017/7/14 21:54, Sinan Kaya wrote:
> > On 7/13/2017 9:26 PM, Ding Tianhong wrote:
> >> There is no code to enable the PCIe Relaxed Ordering bit in the configuration space,
> >> it is only be enable by default according to the PCIe Standard Specification, what we
> >> do is to distinguish the RC problematic platform and clear the Relaxed Ordering bit
> >> to tell the PCIe EP don't send any TLPs with Relaxed Ordering Attributes to the Root
> >> Complex.
> >
> > Maybe, you should change the patch commit as
> > "Disable PCIe Relaxed Ordering if not supported"...
>
> I agree that to use the new commit title as your suggested, thanks. :)
>
> @Bjorn do you want me to spawn a new patchset with the new commit title
> and the Reviewed-by from Casey on the patch 3, or maybe you could pick this
> up and modify it own ? thanks.

Hi Ding,

Bjorn is currently on holiday so it might be a good idea to respin the
series with any updates so nothing is lost. Thanks,

Alex