Jeff: Here is the original discussion with John Linville on this topic: http://marc.theaimsgroup.com/?l=linux-netdev&m=111722587223104&w=2
ganesh. >-----Original Message----- >From: Chilakala, Mallikarjuna >Sent: Thursday, August 25, 2005 1:40 PM >To: Venkatesan, Ganesh; Ronciak, John >Subject: FW: PATCH net-drivers-2.6 4/8] e1000: Backout watchdog task patch >from John Linville > > > >-----Original Message----- >From: Jeff Garzik [mailto:[EMAIL PROTECTED] >Sent: Thursday, August 25, 2005 1:39 PM >To: Chilakala, Mallikarjuna >Cc: netdev >Subject: Re: PATCH net-drivers-2.6 4/8] e1000: Backout watchdog task patch >from John Linville > >Malli Chilakala wrote: >> Backout watchdog task patch from John Linville >> >> Signed-off-by: Mallikarjuna R Chilakala ><[EMAIL PROTECTED]> >> Signed-off-by: Ganesh Venkatesan <[EMAIL PROTECTED]> >> Signed-off-by: John Ronciak <[EMAIL PROTECTED]> > >Rejected for two reasons: > >1) No explanation why this back-out is needed. > >2) You should fix problems rather than running from them. We want as >much slow path code as possible -- such as reset or phy code -- to be in >process context. > - 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
