linux-kernel - Re: dma engine drivers for 2.6.25?

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]

Date:	Thu, 14 Feb 2008 22:29:11 -0700
From:	"Dan Williams" <dan.j.williams@...el.com>
To:	"Kumar Gala" <galak@...nel.crashing.org>
Cc:	"LKML Kernel" <linux-kernel@...r.kernel.org>,
	"Andrew Morton" <akpm@...ux-foundation.org>,
	"Zhang Wei" <wei.zhang@...escale.com>
Subject: Re: dma engine drivers for 2.6.25?

On Thu, Feb 14, 2008 at 8:44 PM, Kumar Gala <galak@...nel.crashing.org> wrote:
>
>  On Feb 14, 2008, at 12:14 PM, Dan Williams wrote:
>
>  > On Wed, Feb 13, 2008 at 8:52 PM, Kumar Gala
>  > <galak@...nel.crashing.org> wrote:
>  >> Dan,
>  >>
>  >> What's going on with the dma engine drivers for 2.6.25?  We had a
>  >> Freescale dma driver from Zhang Wei queued up but seems to have been
>  >> lost.
>  >
>  > I pulled it into my tree and am holding it until Zhang has an
>  > opportunity to address the pending review comments [1].  I also did
>  > not feel comfortable pushing it to Linus without a PPC maintainer's
>  > Acked-by/Reviewed-by.
>  >
>  > I have attached the version I am carrying.
>
>  What issues are still open.  I was under the belief that Zhang had
>  resolved all the issues.
>

The high priority review item is that the driver performs operation
completion callbacks in hardirq context.  Clients of the API assume
that callbacks will happen in softirq context.  Of lesser concern is
the use of GFP_ATOMIC in fsl_dma_alloc_descriptor.  Other drivers
preallocate a small pool of descriptors.

Regards,
Dan
--
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/