Bug #1027

add vminfo.updates to perl kstat module

Added by Mike Harsch almost 3 years ago. Updated almost 3 years ago.

Status:Resolved Start date:2011-05-12
Priority:Low Due date:
Assignee:- % Done:

0%

Category:lib - userland libraries Spent time: -
Target version:- Estimated time:1.00 hour
Difficulty:Bite-size Tags:needs-triage

Description

A recent change for CR 6882903 added the field 'updates' to vminfo_t (defined in sysinfo.h). kstat(1) doesn't show this field when the vminfo kstat is queried, because the perl kstat module hasn't been updated to include this statistic as part of the vminfo kstat.

I believe this just a 1-line change needed to /usr/src/cmd/perl/contrib/Sun/Solaris/Kstat/Kstat.xs
in the save_sysinfo function. I would like to contribute the patch for this myself if possible.

History

Updated by Mike Harsch almost 3 years ago

This issue is complicated by the fact that Bug #377 removed perl 5.8.4 from the build process. It turns out that kstat(1) uses perl 5.8.4, so any changes to the kstat module are not reflected in kstat(1) output, since the underlying kstat module (for perl 5.8.4) isn't built.

Updated by Mike Harsch almost 3 years ago

Updated by Mike Harsch almost 3 years ago

Mike Harsch wrote:

This issue is complicated by the fact that Bug #377 removed perl 5.8.4 from the build process. It turns out that kstat(1) uses perl 5.8.4, so any changes to the kstat module are not reflected in kstat(1) output, since the underlying kstat module (for perl 5.8.4) isn't built.

Correction: kstat(1) uses /usr/perl5/bin/perl which just happens to link to 5.8.4 on my system. As long as the Sun::Solaris modules are built with the same version provided by the build system (currently 5.10), everything should be fine.

Updated by Gordon Ross almost 3 years ago

  • Status changed from New to Resolved

changeset: 13373:d8a063b9ccba
tag: tip
user: Mike Harsch <>
date: Sun May 15 18:57:41 2011 -0600

description:
1027 add vminfo.updates to perl kstat module
Reviewed by: Garrett D'Amore <>
Reviewed by: Richard Elling <>
Approved by: Gordon Ross <>

Also available in: Atom PDF