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)
Sun, Dec 15, 10:05 AM
Unknown Object (File)
Tue, Dec 10, 7:57 PM
Unknown Object (File)
Thu, Dec 5, 7:39 PM
Unknown Object (File)
Wed, Dec 4, 7:09 PM
Unknown Object (File)
Oct 27 2024, 5:52 AM
Unknown Object (File)
Oct 27 2024, 12:21 AM
Unknown Object (File)
Oct 26 2024, 9:50 PM
Unknown Object (File)
Oct 18 2024, 5:51 PM

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.