On Sun, Jun 25, 2006 at 05:44:00PM -0600, Grant Grundler wrote: > On Sun, Jun 25, 2006 at 10:31:08AM +0100, Martin Michlmayr wrote: > > * [EMAIL PROTECTED] <[EMAIL PROTECTED]> [2006-06-25 01:45]: > > > Cc: Valerie Henson <[EMAIL PROTECTED]> > > > > > > [akpm: this is a previously-nacked patch, but the problem is real] > > ia64 and parisc need the changes to tulip_select_media() too. > > > > And, for the record, Jeff's suggestion as to how to rework the patch: > > > > | Answer hasn't changed since this was last discussed: sleep, rather than > > | delay for an extra-long time. That's the only hurdle for the tulip > > | patches you keep resending. > > > > | Francois Romieu even had an untested patch that attempted this, > > | somewhere. > > > > Now we just need someone to complete this work... > > Kyle McMartin has the work queue patch merged into the cvs.parisc-linux.org > CVS tree. It would be easiest to accept this patch and then add the > tulip work queue patch next.
I like the work queue approach; I'll be talking with Kyle and Grant about it. > Note that the work queue does NOT solve all of the "spin delay with > interrupts blocked" problems. tulip_stop_rxtx() polls for nearly as long > and tulip_select_media(). Let's chat... -VAL - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
