- User Since
- Aug 24 2017, 3:21 PM (116 w, 4 d)
Ok, will add tests and fix the comments.
Sat, Nov 16
We use this implementation in the ice driver (current review at https://reviews.freebsd.org/D21959, will be updated to reflect moving the function here soon).
Fri, Nov 15
Tue, Nov 5
You should be able to verify this leak by doing something like:
I'm really not sure this is the correct fix. Possibly we should just add an ifa_free after the rtrequest1_fib()?
I found this while debugging an extra reference on the ifa refcount. I think I found a "solution" that isn't quite right, but this comment is misleading.
Mon, Nov 4
Remove ETHER_IS_ZERO definitions from USB networking drivers.
Fri, Nov 1
We're in agreement that the refactor is a better solution, so I'm going to abandon this one.
add an IFNET_WLOCK_ASSERT in if_freegroup
I think the refactored version to avoid code duplication is probably a better fix. If you go with this one, perhaps remove the whitespace changes and only commit the added free().
Also, yay.. yet another memory leak fixed while playing whack-a-mole trying to fix leaks during driver load/unload testing. I think we still have at least one more on M_IFADDR somewhere.
This is likely the better approach to solving the leak described in https://reviews.freebsd.org/D22215
This is the simpler straight forward fix to the same issue described in https://reviews.freebsd.org/D22214
Thu, Oct 31
The fact that the map pointer can be NULL is weird, but we found this to be true in several places, and without the removal of the NULL checks we end up leaking ~200 bytes of M_DEVBUF, and associated other DMA memory allocated outside of malloc.
Fri, Oct 25
We should be able to MFC this to 12-STABLE and 11-STABLE. Since the issue has existed since the beginning of iflib in-tree, I don't really think it's worth trying to rush it into 12.1... although it is a relatively small fix.
Wed, Oct 23
Oct 18 2019
Updated the review to add vlan event handler unregister calls before iflib_stop as well. I kept the call in iflib_deregister as well so that they get cleaned up properly during error flows. Now it uses the "unregister -> assign NULL" pattern so that only one of the flows will actually perform an unregister check.
also move VLAN event handlers
Oct 17 2019
Assuming this fix makes sense, this should be MFC'd to 11-STABLE and 12-STABLE too, IMHO.
I did manage to eventually trigger another stale ifp pointer, though it was not reliable:
Oct 10 2019
Oct 9 2019
Use the correct register value for the received pause frames
Oct 1 2019
Sep 9 2019
I agree with Eric, we should depend on the device to be configured properly, and push to get the images for the device updated if they're wrong.
Sep 5 2019
This is specifically a fix for 11-STABLE. It's already fixed in CURRENT and STABLE-12. It looks like it was an accidental miss in MFC for STABLE-11 a few months ago.
Aug 13 2019
Use bitwise OR instead of bitwise AND to correct implementation
Aug 12 2019
Jul 31 2019
Move the kobject refcount removal to a separate patch
Jul 23 2019
Jul 19 2019
This is built on top of https://reviews.freebsd.org/D21003
May 10 2019
We discovered this as part of our rename from ixlv to iavf.
Apr 25 2019
Ok, I updated this revision to just use the default value now instead of rounding down.
use the default value instead of rounding down
Apr 22 2019
Modify _iflib_assert to verify driver parameters are power of 2
I would be fine modifying the iflib code to avoid the power of 2 limitation entirely, but I don't think there is significant gain for that, because we would have to add a configuration knob indicating the increment. Additionally, since some hardware likely does require power-of-2 values, we'd want to have a way for those drivers to express that restriction as well. power-of-2 restriction can't be expressed with just an increment value, so that becomes a lot trickier.
Yea, I'll add MPASS to check that the values are power of 2.
Apr 11 2019
I have a couple of thoughts I had while writing this.
Apr 10 2019
Mar 19 2019
Mar 15 2019
Document the new CONST_STRING sysctl macros
Mar 14 2019
I like the addition of SYSCTL_ADD_CONST_STRING since it also ensures that we enforce that we don't have CTLFLAG_WR.
Use __DECONST instead of a poor reimplementation
This version actually compilers without the warnings, but it *is* rather ugly to have to do it this way
Introduce a __drop_const macro to handle the warning