Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757553Ab1CRV2c (ORCPT ); Fri, 18 Mar 2011 17:28:32 -0400 Received: from ist.d-labs.de ([213.239.218.44]:54821 "EHLO mx01.d-labs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757444Ab1CRV2Z (ORCPT ); Fri, 18 Mar 2011 17:28:25 -0400 Date: Fri, 18 Mar 2011 22:28:19 +0100 From: Florian Mickler To: Antti Palosaari Cc: mchehab@infradead.org, oliver@neukum.org, jwjstone@fastmail.fm, linux-kernel@vger.kernel.org, Mauro Carvalho Chehab , linux-media@vger.kernel.org Subject: Re: [PATCH 06/16] [media] ce6230: get rid of on-stack dma buffer Message-ID: <20110318222819.3ab2424a@schatten.dmk.lab> In-Reply-To: <4D8389FB.6040707@iki.fi> References: <20110315093632.5fc9fb77@schatten.dmk.lab> <1300178655-24832-1-git-send-email-florian@mickler.org> <1300178655-24832-6-git-send-email-florian@mickler.org> <4D8389FB.6040707@iki.fi> X-Mailer: Claws Mail 3.7.8 (GTK+ 2.22.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1009 Lines: 32 On Fri, 18 Mar 2011 18:36:11 +0200 Antti Palosaari wrote: > On 03/15/2011 10:43 AM, Florian Mickler wrote: > > usb_control_msg initiates (and waits for completion of) a dma transfer using > > the supplied buffer. That buffer thus has to be seperately allocated on > > the heap. > > > > In lib/dma_debug.c the function check_for_stack even warns about it: > > WARNING: at lib/dma-debug.c:866 check_for_stack > > > > Note: This change is tested to compile only, as I don't have the hardware. > > > > Signed-off-by: Florian Mickler > > Acked-by: Antti Palosaari > Reviewed-by: Antti Palosaari > Tested-by: Antti Palosaari > > > t. Antti > Thanks for your feedback! Regards, Flo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/