From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Kirsher Subject: Re: [PATCH 1/2] e1000: Fix DMA mapping error handling on TX Date: Sat, 23 Jan 2010 19:58:07 -0800 Message-ID: <9929d2391001231958n3f8d5165yaeed9ec1e0d7121a@mail.gmail.com> References: <20100121114244.GC32259@kryten> <9929d2391001221940r4482dde4k8e5409bc7933bc21@mail.gmail.com> <20100124004753.GB2996@kryten> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Yi Zou , e1000-devel@lists.sourceforge.net, Bruce Allan , Jesse Brandeburg , John Ronciak , netdev@vger.kernel.org, Roel Kluin To: Anton Blanchard Return-path: In-Reply-To: <20100124004753.GB2996@kryten> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: e1000-devel-bounces@lists.sourceforge.net List-Id: netdev.vger.kernel.org On Sat, Jan 23, 2010 at 16:47, Anton Blanchard wrote: > > Hi, > >> This patch does not apply to the current e1000 driver in net-2.6, much >> of this patch has already been corrected (applied) by Roel Kluin >> recent patch. > > Sorry I was basing off net-next. I just compared it to my fix and looks like > the patch in net-2.6 has an off by one error doesn't it? > > Anton > > -- > This was discussed during our code review of Roel's patch, and it was found that there was not an issue. But I will review the code again to ensure that there is not "an off by one error". Thanks for looking at this. -- Cheers, Jeff ------------------------------------------------------------------------------ Throughout its 18-year history, RSA Conference consistently attracts the world's best and brightest in the field, creating opportunities for Conference attendees to learn about information security's most important issues through interactions with peers, luminaries and emerging and established companies. http://p.sf.net/sfu/rsaconf-dev2dev _______________________________________________ E1000-devel mailing list E1000-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired