cvs commit: src/sys/dev/nve if_nve.c

Robert Watson rwatson at FreeBSD.org
Fri Dec 2 13:28:48 GMT 2005


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


More information about the cvs-all mailing list