On Fri, Dec 15, 2023 at 11:59:51AM +0100, Niklas Cassel wrote:
> From: Niklas Cassel <niklas.cassel@xxxxxxx>
>
> Use a unique test pattern for each BAR in. This makes it easier to
> detect/debug address translation issues, since a developer can dump
> the backing memory on the EP side, using e.g. devmem, to verify that
> the address translation for each BAR is actually correct.
>
> Signed-off-by: Niklas Cassel <niklas.cassel@xxxxxxx>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
- Mani
> ---
> drivers/misc/pci_endpoint_test.c | 14 ++++++++++++--
> 1 file changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/misc/pci_endpoint_test.c b/drivers/misc/pci_endpoint_test.c
> index a765a05f0c64..7ac1922475af 100644
> --- a/drivers/misc/pci_endpoint_test.c
> +++ b/drivers/misc/pci_endpoint_test.c
> @@ -263,6 +263,15 @@ static bool pci_endpoint_test_request_irq(struct pci_endpoint_test *test)
> return false;
> }
>
> +static const u32 bar_test_pattern[] = {
> + 0xA0A0A0A0,
> + 0xA1A1A1A1,
> + 0xA2A2A2A2,
> + 0xA3A3A3A3,
> + 0xA4A4A4A4,
> + 0xA5A5A5A5,
> +};
> +
> static bool pci_endpoint_test_bar(struct pci_endpoint_test *test,
> enum pci_barno barno)
> {
> @@ -280,11 +289,12 @@ static bool pci_endpoint_test_bar(struct pci_endpoint_test *test,
> size = 0x4;
>
> for (j = 0; j < size; j += 4)
> - pci_endpoint_test_bar_writel(test, barno, j, 0xA0A0A0A0);
> + pci_endpoint_test_bar_writel(test, barno, j,
> + bar_test_pattern[barno]);
>
> for (j = 0; j < size; j += 4) {
> val = pci_endpoint_test_bar_readl(test, barno, j);
> - if (val != 0xA0A0A0A0)
> + if (val != bar_test_pattern[barno])
> return false;
> }
>
> --
> 2.43.0
>
--
மணிவண்ணன் சதாசிவம்
