linux-kernel - RE: [PATCH 01/11] dmaengine: add context parameter toprep_slave_sgand prep_dma

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1328624057.26182.144.camel@vkoul-udesk3>
Date:	Tue, 07 Feb 2012 19:44:17 +0530
From:	Vinod Koul <vinod.koul@...el.com>
To:	"Bounine, Alexandre" <Alexandre.Bounine@....com>
Cc:	Russell King <rmk@....linux.org.uk>, dan.j.williams@...el.com,
	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	Jassi Brar <jaswinder.singh@...aro.org>,
	Kumar Gala <galak@...nel.crashing.org>,
	Matt Porter <mporter@...nel.crashing.org>,
	Li Yang <leoli@...escale.com>
Subject: RE: [PATCH 01/11] dmaengine: add context parameter
 toprep_slave_sgand prep_dma_cyclic

On Tue, 2012-02-07 at 06:01 -0800, Bounine, Alexandre wrote:
> Currently function private_candidate() allows filtering function
> not to be specified and therefore skip the filter check.
> It may be a good safety measure to ensure that channels specified
> as DMA_SLAVE_EXT provide a corresponding filter routine so they
> positively identify dma channel that will accept intended extra
> parameter.
> I assume that a system may have more than one dmac registered as
> DMA_SLAVE_EXT. 
> 
> This is what I have for RapidIO - the filter call ensures that DMA
> channel is associated with a RapidIO controller that provides access
> to the specified SRIO device. 
For slave-dma it's better to have a filter :)
-- 
~Vinod

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/