On Fri, Feb 28, 2014 at 09:14:03AM +0200, Alan Barrett wrote: > On Thu, 27 Feb 2014, Joerg Sonnenberger wrote: > >Modified Files: > > src/sys/dev/pci: if_ti.c > > > >Log Message: > >Remove impossible checks. > > If new bugs are introduced in the future, then these tests might no > longer be impossible. So I'd prefer to change them to KASSERT or > KDASSERT rather than deleting them.
Given the nature of this checks and the failure mode of such changes, I don't see a point in the assert. Joerg