On Wed, 2008-01-09 at 12:15 -0500, David Frascone wrote: > The -u flag to top is supposed to limit uid lookups. But, it was still > spinning through the entire password database with while (getpwent()). At > Cisco, which has a fairly large NIS database, this takes forever, and top > never starts. > > I filed this bug to address it: > http://www.freebsd.org/cgi/query-pr.cgi?pr=119490 > > The attached patch fixes it. This is my first submission, so let me know if > this is the proper order, etc, to post the patches (i.e. file the bug, the > post a fix) This is a longstanding bug, sadly. bin/20799 seems to be the first to mention it, and is over 7 years old now. Every system I deploy at work has to have top(1) patched to avoid this. I think your solution of not sizing the usernames when -u is specified is a sensible one, although I think some thought really needs to be put into whether sizing is necessary at all - maybe the solution is to just limit the username field to 8 or 10 characters or so, and either trim longer names or show them as userids. Parsing a huge (in my case ~25,000 users) password map when top is run seems stupid, especially given that many invocations of top are at exactly the time when significant extra load on a system is not a good idea. It would probably be best to submit your patch as a followup to PR bin/89762. GavinReceived on Wed Jan 09 2008 - 17:59:34 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:39:25 UTC