On 18/09/2019 12:42, Andrew Murray wrote: > On Tue, Sep 17, 2019 at 08:07:13PM +0200, Steffen Liebergeld wrote: >> According to documentation [0] the correct offset for the >> Upstream Peer Decode Configuration Register (UPDCR) is 0x1014. >> It was previously defined as 0x1114. This patch fixes it. >> >> [0] >> https://www.intel.com/content/dam/www/public/us/en/documents/datasheets/4th-gen-core-family-mobile-i-o-datasheet.pdf >> (page 325) >> >> Signed-off-by: Steffen Liebergeld <steffen.liebergeld@xxxxxxxxxxxxxxx> > > You may also like to add: > > Fixes: d99321b63b1f ("PCI: Enable quirks for PCIe ACS on Intel PCH root ports") > Reviewed-by: Andrew Murray <andrew.murray@xxxxxxx> > > As well as CC'ing stable. Ok. Thank you. > I guess the side effect of this bug is that we claim to have peer > isolation when we do not. This fix ensures that we get the advertised > isolation. Yes, that is also my understanding. Should I explain that in the commit message? Best, Steffen -- Steffen Liebergeld +49-351-41 888 613 Kernkonzept GmbH. Sitz: Dresden. Amtsgericht Dresden, HRB 31129. Geschäftsführer: Dr.-Ing. Michael Hohmuth
