Procfs is not support process share memory monitoring

[SilenceAdele]

• In the top command, the size of the process's shared memory can be obtained through SHR.

PID USER      PR  NI    VIRT    RES    SHR S  %CPU  %MEM     TIME+ COMMAND
58984 diag      20   0 2119424 185436  35180 S   1.0   0.1 234:02.02 qemu-kvm

• The role of shared memory monitoring of processes in actual production

1. In a virtualized or containerized environment, multiple virtual machines or containers may share the host machine's memory. If a virtual machine or container occupies too much shared memory, it may affect the memory resources of other virtual machines or containers, resulting in limited memory access, delays or crashes.
2. If a long-running virtual machine or container does not release shared memory correctly, it may cause the shared memory to continue to grow, eventually affecting the stability of the entire system.

Looking forward to your reply！

[SilenceAdele]

ping...

[discordianfish]

Are you sure its not part of the MemInfo struct? https://pkg.go.dev/github.com/prometheus/procfs#Meminfo

[SilenceAdele]

// Meminfo returns an information about current kernel/system memory statistics.
// See https://www.kernel.org/doc/Documentation/filesystems/proc.txt
func (fs FS) Meminfo() (Meminfo, error) {
	b, err := util.ReadFileNoStat(fs.proc.Path("meminfo"))
	if err != nil {
		return Meminfo{}, err
	}

	m, err := parseMemInfo(bytes.NewReader(b))
	if err != nil {
		return Meminfo{}, fmt.Errorf("%w: %w", ErrFileParse, err)
	}

	return *m, nil
}

• https://github.com/prometheus/procfs/blob/v0.15.1/meminfo.go#L195 From the meminfo.go code, we can see that Procfs obtains the system memory information in the /proc/meminfo directory. Shmem here is the size of the system shared memory, rather than the shared memory of a specific process.
• I want to get the shared memory of each pid. It should be obtained from /proc/pid/statm.Therefore, Procfs has not yet achieved all process sharing of memory monitoring. @discordianfish

[discordianfish]

Ah I see, well yeah than this seems reasonable. PRs welcome! :)