Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-28 Thread Stefan Agner
On 28.11.2018 02:28, Andrey Smirnov wrote: > On Tue, Nov 27, 2018 at 5:12 PM Fabio Estevam wrote: >> >> Hi Andrey, >> >> On Tue, Nov 27, 2018 at 10:57 PM Andrey Smirnov >> wrote: >> >> > Could this be a regression? Prior to 415b6185c541 ("PCI: imx6: Fix >> > config read timeout handling") all of

Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-27 Thread Andrey Smirnov
On Tue, Nov 27, 2018 at 5:12 PM Fabio Estevam wrote: > > Hi Andrey, > > On Tue, Nov 27, 2018 at 10:57 PM Andrey Smirnov > wrote: > > > Could this be a regression? Prior to 415b6185c541 ("PCI: imx6: Fix > > config read timeout handling") all of the imprecise aborts were caught > > and handled via

Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-27 Thread Fabio Estevam
Hi Andrey, On Tue, Nov 27, 2018 at 10:57 PM Andrey Smirnov wrote: > Could this be a regression? Prior to 415b6185c541 ("PCI: imx6: Fix > config read timeout handling") all of the imprecise aborts were caught > and handled via no-op handler. I did an experiment on i.MX6Q board > that I have (ZII

Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-27 Thread Andrey Smirnov
On Tue, Nov 20, 2018 at 9:43 AM Stefan Agner wrote: > > Define the length of the DBI registers. This makes sure that > the kernel does not access registers beyond that point, avoiding > the following abort on a i.MX 6Quad: > # cat /sys/devices/soc0/soc/1ffc000.pcie/pci\:00/\:00\:00.0/con

Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-20 Thread Lucas Stach
Am Dienstag, den 20.11.2018, 14:27 +0100 schrieb Stefan Agner: > Define the length of the DBI registers. This makes sure that > the kernel does not access registers beyond that point, avoiding > the following abort on a i.MX 6Quad: >   # cat /sys/devices/soc0/soc/1ffc000.pcie/pci\:00/\:00\:

[PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-20 Thread Stefan Agner
Define the length of the DBI registers. This makes sure that the kernel does not access registers beyond that point, avoiding the following abort on a i.MX 6Quad: # cat /sys/devices/soc0/soc/1ffc000.pcie/pci\:00/\:00\:00.0/config [ 100.021433] Unhandled fault: imprecise external abort