Page MenuHomeFreeBSD

Account for the new "laundry" memory in ksysguard in preparation for 11.1
ClosedPublic

Authored by rezny on Jul 11 2017, 5:04 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 28, 10:54 PM
Unknown Object (File)
Fri, Nov 28, 2:09 PM
Unknown Object (File)
Wed, Nov 26, 2:06 AM
Unknown Object (File)
Fri, Nov 14, 2:04 PM
Unknown Object (File)
Wed, Nov 12, 4:45 AM
Unknown Object (File)
Sun, Nov 9, 9:58 PM
Unknown Object (File)
Oct 17 2025, 4:10 AM
Unknown Object (File)
Oct 11 2025, 12:29 AM

Details

Summary

Recent src changes have introduced another memory accounting variable that ksysguard needs to be aware of. As fas as I understand it, "laundry" is essentially a subset of "inactive" that is now reported separately. So, handle it in ksysguardd by adding "laundry" where "inactive" is reported to ksysguard.

Test Plan

used on 11-stable for several weeks

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tcberner added a subscriber: tcberner.

Looks good to me. Could you upstream it into the current ksysguard ?

This revision is now accepted and ready to land.Jul 11 2017, 7:10 AM

On second thought --

  • don't you need to check whether the sysctl exists? -- or has it trickled down to all the versions?
  • don't forget to bump portrevision
This revision now requires changes to proceed.Jul 11 2017, 7:15 AM
rezny edited edge metadata.

bump PORTREVISION

On second thought --

  • don't you need to check whether the sysctl exists? -- or has it trickled down to all the versions?

No need to check; if it doesn't exist (it'll be in 11.1, not sure if 10 gets it) sysctlbyname will return -1, set errno to ENOENT, and leave the output variable unchanged (remains zero).

  • don't forget to bump portrevision

of course :)

Thanks. In that case, go ahead :)

This revision is now accepted and ready to land.Jul 12 2017, 7:27 PM
adridg added a subscriber: adridg.

LGTM, from 10.3 -- it builds, it runs, it doesn't show anything immediately silly. That is, if the means to test is "start ksysguard, then look at the *system load* tab".

This revision was automatically updated to reflect the committed changes.