From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [PATCH] Fix stack corruption on some architectures Date: Mon, 19 Aug 2013 15:23:38 -0700 Message-ID: <20130819222338.GA3041@kroah.com> References: <1376944647.5082.12.camel@chimera> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Dumon , linux-usb@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Daniel Gimpelevich Return-path: Content-Disposition: inline In-Reply-To: <1376944647.5082.12.camel@chimera> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Mon, Aug 19, 2013 at 01:37:27PM -0700, Daniel Gimpelevich wrote: > There is no need to get an interface specification if we know it's the > wrong one; trivial change. The big thing, though, was explained in the > #mipslinux IRC channel: > [Mon 2013-08-19 12:28:21 PM PDT] guys, are you sure it's not "DMA off stack" case? > [Mon 2013-08-19 12:28:35 PM PDT] it's a known stack corruptor on non-coherent arches > [Mon 2013-08-19 12:31:48 PM PDT] headless: for usb/ehci? > [Mon 2013-08-19 12:34:11 PM PDT] headless: explain > [Mon 2013-08-19 12:35:38 PM PDT] usb_control_msg() (or other such func) should not use buffer on stack. DMA from/to stack is prohibited > [Mon 2013-08-19 12:35:58 PM PDT] and EHCI uses DMA on control xfers (as well as all the others) > > Signed-off-by: Daniel Gimpelevich Acked-by: Greg Kroah-Hartman