ipfw: remove locking workarounds in the table code

Before the "upper half lock" became sleepable the table manipulation code
needed sophisticated workarounds to recover from races, where the lock is
temporarily dropped to do malloc(M_WAITOK). Remove all these workarounds
as they are no longer needed.

Differential Revision: https://reviews.freebsd.org/D54580