Date: Fri, 2 Dec 2005 13:28:46 +0000 (GMT) From: Robert Watson <rwatson@FreeBSD.org> To: John Baldwin <jhb@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org, "David E. O'Brien" <obrien@freebsd.org> Subject: Re: cvs commit: src/sys/dev/nve if_nve.c Message-ID: <20051202132726.L77687@fledge.watson.org> In-Reply-To: <200512020749.06435.jhb@freebsd.org> References: <200510271539.j9RFdJdS096434@repoman.freebsd.org> <20051202114031.J77687@fledge.watson.org> <200512020749.06435.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 2 Dec 2005, John Baldwin wrote: >> I have had at least one report that this patch corrects repeated >> wedging of the if_nve interface on 6.x systems. Do you have plans to >> merge this patch to RELENG_6 and/or RELENG_6_0? The PR seems to >> suggest a larger set of changes that I've not dug into, and as a result >> fail to understand, however. > > I'd be interested in looking at the PR FWIW as I've been doing some > hacking on nve(4) recently. After some discussion, it sounds like the problem here is that this patch fixes a bug in a workaround for other serious problems -- i.e., that when you reset the interface to deal with the fact that the interface has wedged, it isn't being properly reset. So this patch is useful, in that it helps work around one or more other serious bugs better, but still leaves serious underlying problems. That said, I think it would be good to MFC the fix. Robert N M Watson
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20051202132726.L77687>