Hi Ralph, On 4/27/2017 7:05 PM, Sricharan R wrote: > Hi, > > On 4/26/2017 9:45 PM, Ralph Sennhauser wrote: >> Hi Sricharan R, >> >> Commit 09515ef5ddad ("of/acpi: Configure dma operations at probe time >> for platform/amba/pci bus devices") causes a kernel panic as in the log >> below on an armada-385. Reverting the commit fixes the issue. >> >> Regards >> Ralph > > Somehow not getting a obvious clue on whats going wrong with the logs > below. From the log and looking in to dts, the drivers seems to the one for > "marvell,armada-370-neta". Issue looks the data from the dma has gone bad > and subsequently referring the wrong data has resulted in the crash. > Looks like the dma_masks is the one going wrong. > Can i get some logs from mvneta_probe, about dev->dma_mask, > dev->coherent_dma_mask and dev->dma_ops with and without the patch > to see whats the difference ? > Attached the patch with debug prints. Regards, Sricharan
From fe77d3968d6bc35b46e5f30d5c67c7603aa10597 Mon Sep 17 00:00:00 2001 From: Sricharan R <sricharan@xxxxxxxxxxxxxx> Date: Fri, 28 Apr 2017 09:43:26 +0530 Subject: [PATCH] Debug prints Signed-off-by: Sricharan R <sricharan@xxxxxxxxxxxxxx> --- drivers/net/ethernet/marvell/mvneta.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c index d297011..250f0b2 100644 --- a/drivers/net/ethernet/marvell/mvneta.c +++ b/drivers/net/ethernet/marvell/mvneta.c @@ -4146,6 +4146,12 @@ static int mvneta_probe(struct platform_device *pdev) int phy_mode; int err; int cpu; + struct device *ddev = &pdev->dev; + + + dev_err(ddev, "dev->dma_mask 0x%llx\n", *(ddev->dma_mask)); + dev_err(ddev, "dev->coherent_dma_mask 0x%llx\n", ddev->coherent_dma_mask); + dev_err(ddev, "dev->dma_ops 0x%llx\n", ddev->dma_ops); dev = alloc_etherdev_mqs(sizeof(struct mvneta_port), txq_number, rxq_number); if (!dev) -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation