Re: [PATCH v7] PCI: imx6: limit DBI register length

2019-02-26 Thread Stefan Agner
On 25.02.2019 17:52, Trent Piepho wrote: > On Mon, 2019-02-25 at 16:15 +, Leonard Crestez wrote: >> On Mon, 2019-02-25 at 17:02 +0100, Stefan Agner wrote: >> > Define the length of the DBI registers and limit config space to its >> > length. This makes sure that the kernel does not access regis

Re: [PATCH v7] PCI: imx6: limit DBI register length

2019-02-26 Thread Stefan Agner
On 25.02.2019 21:19, Bjorn Helgaas wrote: > [+cc Thinh] > > On Mon, Feb 25, 2019 at 10:52 AM Trent Piepho wrote: >> On Mon, 2019-02-25 at 16:15 +, Leonard Crestez wrote: >> > On Mon, 2019-02-25 at 17:02 +0100, Stefan Agner wrote: >> > > Define the length of the DBI registers and limit config

Re: [PATCH v7] PCI: imx6: limit DBI register length

2019-02-25 Thread Bjorn Helgaas
[+cc Thinh] On Mon, Feb 25, 2019 at 10:52 AM Trent Piepho wrote: > On Mon, 2019-02-25 at 16:15 +, Leonard Crestez wrote: > > On Mon, 2019-02-25 at 17:02 +0100, Stefan Agner wrote: > > > Define the length of the DBI registers and limit config space to its > > > length. This makes sure that the

Re: [PATCH v7] PCI: imx6: limit DBI register length

2019-02-25 Thread Trent Piepho
On Mon, 2019-02-25 at 16:15 +, Leonard Crestez wrote: > On Mon, 2019-02-25 at 17:02 +0100, Stefan Agner wrote: > > Define the length of the DBI registers and limit config space to its > > length. This makes sure that the kernel does not access registers > > beyond that point, avoiding the follo

Re: [PATCH v7] PCI: imx6: limit DBI register length

2019-02-25 Thread Leonard Crestez
On Mon, 2019-02-25 at 17:02 +0100, Stefan Agner wrote: > Define the length of the DBI registers and limit config space to its > length. This makes sure that the kernel does not access registers > beyond that point, avoiding the following abort on a i.MX 6Quad: > > +static void imx6_pcie_quirk(stru

[PATCH v7] PCI: imx6: limit DBI register length

2019-02-25 Thread Stefan Agner
Define the length of the DBI registers and limit config space to its length. 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] Unhan