From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750994AbZH0FiP (ORCPT ); Thu, 27 Aug 2009 01:38:15 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750772AbZH0FiO (ORCPT ); Thu, 27 Aug 2009 01:38:14 -0400 Received: from out01.mta.xmission.com ([166.70.13.231]:42290 "EHLO out01.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750759AbZH0FiN (ORCPT ); Thu, 27 Aug 2009 01:38:13 -0400 To: Francois Romieu Cc: David Dillow , Michael Riepe , Michael Buesch , Rui Santos , Michael B??ker , linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH] r8169: Reduce looping in the interrupt handler. References: <1251232848.9607.15.camel@lap75545.ornl.gov> <20090825221903.GA13630@electric-eye.fr.zoreil.com> <1251294974.14241.9.camel@obelisk.thedillows.org> <1251295175.14241.11.camel@obelisk.thedillows.org> <20090826213024.GA20428@electric-eye.fr.zoreil.com> <20090827052423.GA1709@electric-eye.fr.zoreil.com> From: ebiederm@xmission.com (Eric W. Biederman) Date: Wed, 26 Aug 2009 22:38:05 -0700 In-Reply-To: <20090827052423.GA1709@electric-eye.fr.zoreil.com> (Francois Romieu's message of "Thu\, 27 Aug 2009 07\:24\:23 +0200") Message-ID: User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-XM-SPF: eid=;;;mid=;;;hst=in01.mta.xmission.com;;;ip=76.21.114.89;;;frm=ebiederm@xmission.com;;;spf=neutral X-SA-Exim-Connect-IP: 76.21.114.89 X-SA-Exim-Rcpt-To: romieu@fr.zoreil.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, m.bueker@berlin.de, rsantos@grupopie.com, mb@bu3sch.de, michael.riepe@googlemail.com, dave@thedillows.org X-SA-Exim-Mail-From: ebiederm@xmission.com X-SA-Exim-Version: 4.2.1 (built Thu, 25 Oct 2007 00:26:12 +0000) X-SA-Exim-Scanned: No (on in01.mta.xmission.com); Unknown failure Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Francois Romieu writes: > [...] >> Right now I really don't have much more test bandwidth. So I tried >> for something simple that would address my problem without >> fundamentally changing the already tested logic. > > Fair enough. > > [...] >> Perhaps we should unconditionally ack everything after changing the >> interrupt mask ? If that would prevent races it sounds like a simple >> fix. > > I should be able to implement something right this evening, please hold on. Sounds good. Eric