|
|
Log in / Subscribe / Register

Digging in the kernel dust

Digging in the kernel dust

Posted Oct 24, 2017 15:32 UTC (Tue) by firasha (guest, #4230)
Parent article: Digging in the kernel dust

A very enjoyable read, it's always great to hear what develops out of the RT work! Those interested in seeing Thomas's talk can do so here.

Sadly, a week before Kernel Recipes 2017, and more than a year after the rework patches were merged, a user-space-visible regression surfaced. This issue is ongoing, because members of the network community (who were most affected by this regression) initially responded by arguing that the right fix was the removal of the whole timer wheel rework — until it was pointed out that the network subsystem would suffer the biggest performance hit from doing so.

I was curious about the above issue, so went digging in the mailing list dust for the original threads:

[PATCH RESEND 0/2] enable hires timer to timeout datagram socket

[PATCH v2 0/2] enable hires timer to timeout datagram socket

[In future articles it would be great if you could include links to the talk(s) and mailing list discussion(s), to make it easy for interested readers to get the details.]

It's been almost a month since Thomas's talk, and as far as I can tell discussion about the above issue died after Thomas's last email of 25 September. No new mailing list threads on the subject have appeared (at least, that I could find) and the only change to /net/core/datagram.c since then is unrelated.


to post comments

Digging in the kernel dust

Posted Oct 30, 2017 16:47 UTC (Mon) by nayna (subscriber, #91044) [Link]

Thank You, Tom, for this article and, firasha, for posting these links.

The TPM Device Driver is another subsystem impacted by this change. The original discussion and a comparison between msleep()/usleep-range()
can be found here:

https://sourceforge.net/p/linux-ima/mailman/message/35685...
https://sourceforge.net/p/linux-ima/mailman/message/35773...

As a result of the non-cascading timer wheel change, we had to replace msleep() with usleep_range().


Copyright © 2026, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds