From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarek Poplawski Subject: Re: [patch] genirq: fix simple and fasteoi irq handlers Date: Fri, 3 Aug 2007 11:10:08 +0200 Message-ID: <20070803091008.GC1582@ff.dom.local> References: <1185437431.3227.21.camel@chaos> <20070726083120.GA26910@elte.hu> <20070726085523.GA3423@ff.dom.local> <20070726091254.GA8063@elte.hu> <4bacf17f0707300029g5116e70bq4808059dc8b069f1@mail.gmail.com> <20070731155843.GA7033@elte.hu> <46B20E47.6020403@googlemail.com> <20070802201126.GA27000@elte.hu> <20070803060733.GA1582@ff.dom.local> <20070803080408.GA12222@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Gabriel C , Linus Torvalds , Thomas Gleixner , Jean-Baptiste Vignaud , linux-kernel , shemminger , linux-net , netdev , Andrew Morton , Alan Cox , marcin.slusarz@gmail.com To: Ingo Molnar Return-path: Content-Disposition: inline In-Reply-To: <20070803080408.GA12222@elte.hu> Sender: linux-net-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, Aug 03, 2007 at 10:04:08AM +0200, Ingo Molnar wrote: > > * Jarek Poplawski wrote: > > > I can't guarantee this is all needed to fix this bug, but I think this > > patch is necessary here. > > hmmm ... very interesting! Now _this_ is something we'd like to see > tested. Could you send a patch to Marcin that also undoes the workaround > we have in place now, so that he could check whether ne2k-pci works fine > with your fix alone? I'm not sure this is needed... Marcin got this patch, I hope, and I don't have another possibility to contact with him. Since he managed with this bisection and all the previous patches I don't think there could be any problems, so: Marcin! I'd be very glad if you could test this patch alone; this should apply without any problems to 2.6.21 (with some offset) and later "vanilla" versions (or try to revert Ingo's last patch with patch -p1 -R). Please, contact me on any problems (alas not during the weekend...). Thanks, Jarek P. PS: of course, I'm very curious of this testing too, but, on the other hand, as I've written earlier, I think this patch is needed for logical reasons only, and it really doesn't look like it could make any damage here.