[ofw] RE: [PATCH] Fix ND connection establishment races

Fab Tillier ftillier at windows.microsoft.com
Thu Mar 26 14:34:16 PDT 2009


> Why not just migrate ND over winverbs rather than changing several
> hundred lines
> of kernel driver code?  This patch is removing a lot of 'enhancements'
> originally made to the drivers to support ND in the first place.

ND over WinVerbs is the end goal, however there are ND drivers in the field right now, supported by the various IB OEMs, that need to be fixed.  WinVerbs however is not supported currently in any driver release.

I could have sent the patch to just the OEMs and bypassed the mailing list, but I don't think splitting the code bases is a valuable thing to do.

> Winverbs already separates the CEP and QP, does the IRP handling, has a
> NotifyDisconnect call available, and the codebase is about 1000 times
> simpler to
> understand than IBAL.  We could probably get ND support over winverbs
> by the 2.1
> feature freeze with.

I'm all for it.  However the OEM driver release cycle isn't gated on 2.1.  So this patch fixes existing bugs, without preventing a future change to WinVerbs.

> How long must we perfume the pig?

Until ND support is available over WV, would be my guess.

-Fab



More information about the ofw mailing list