lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 17 Oct 2017 13:27:52 +0200 From: Marek Szyprowski <m.szyprowski@...sung.com> To: Alexander Kochetkov <al.kochet@...il.com>, dmaengine@...r.kernel.org, linux-kernel@...r.kernel.org Cc: Dan Williams <dan.j.williams@...el.com>, Vinod Koul <vinod.koul@...el.com>, Krzysztof Kozlowski <krzk@...nel.org> Subject: Re: [PATCH v2 2/2] !!! FOR TESTING ONLY !!! dmaengine: pl330: add verbose message and set NR_DEFAULT_DESC to 1 Hi Alexander, On 2017-10-04 13:37, Alexander Kochetkov wrote: > Commit add verbose output to pl330 showing what changes introduced by > commit 1/2 from series work as expected. You should see similar output > running modified kernel: > > The patch tested on rk3188 radxdarock. Could someone else test it on > other hardware with pl330 DMA? > > root@...t:~# dmesg | grep pl330 > [ 0.277520] dma-pl330 20018000.dma-controller: Loaded driver for PL330 DMAC-241330 > [ 0.277538] dma-pl330 20018000.dma-controller: DBUFF-32x8bytes Num_Chans-6 Num_Peri-12 Num_Events-12 > [ 0.279894] dma-pl330 20078000.dma-controller: Loaded driver for PL330 DMAC-241330 > [ 0.279910] dma-pl330 20078000.dma-controller: DBUFF-64x8bytes Num_Chans-7 Num_Peri-20 Num_Events-14 > [ 1.344804] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > [ 1.344832] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > [ 1.344853] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > [ 1.344873] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > [ 1.344893] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > [ 1.344912] dma-pl330 20078000.dma-controller: pl330_get_desc:2458 Allocated one more descriptor > --- rest of similar lines omitted --- > > Signed-off-by: Alexander Kochetkov <al.kochet@...il.com> Tested-by: Marek Szyprowski <m.szyprowski@...sung.com> Seems to be working fine on Exynos4412 OdroidU3 board: # dmesg | grep pl330 [ 0.725608] dma-pl330 12680000.pdma: Loaded driver for PL330 DMAC-141330 [ 0.725629] dma-pl330 12680000.pdma: DBUFF-32x4bytes Num_Chans-8 Num_Peri-32 Num_Events-32 [ 0.731338] dma-pl330 12690000.pdma: Loaded driver for PL330 DMAC-141330 [ 0.731357] dma-pl330 12690000.pdma: DBUFF-32x4bytes Num_Chans-8 Num_Peri-32 Num_Events-32 [ 0.733097] dma-pl330 12850000.mdma: Loaded driver for PL330 DMAC-141330 [ 0.733115] dma-pl330 12850000.mdma: DBUFF-64x8bytes Num_Chans-8 Num_Peri-1 Num_Events-32 [ 98.353073] dma-pl330 12680000.pdma: pl330_get_desc:2460 Allocated one more descriptor [ 98.360970] dma-pl330 12680000.pdma: pl330_get_desc:2460 Allocated one more descriptor [ 98.368867] dma-pl330 12680000.pdma: pl330_get_desc:2460 Allocated one more descriptor > --- > drivers/dma/pl330.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c > index deec4a4..3441c16 100644 > --- a/drivers/dma/pl330.c > +++ b/drivers/dma/pl330.c > @@ -266,7 +266,7 @@ enum pl330_byteswap { > > /* The number of default descriptors */ > > -#define NR_DEFAULT_DESC 16 > +#define NR_DEFAULT_DESC 1 > > /* Delay for runtime PM autosuspend, ms */ > #define PL330_AUTOSUSPEND_DELAY 20 > @@ -2455,6 +2455,9 @@ static struct dma_pl330_desc *pl330_get_desc(struct dma_pl330_chan *pch) > > desc = pluck_desc(&pool, &lock); > WARN_ON(!desc || !list_empty(&pool)); > + > + dev_err(pch->dmac->ddma.dev, "%s:%d Allocated one more descriptor\n", > + __func__, __LINE__); > } > > /* Initialize the descriptor */ Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland
Powered by blists - more mailing lists