Skip to content

Commit

Permalink
rename resctrlHostMount to resctrlMount
Browse files Browse the repository at this point in the history
  • Loading branch information
ppalucki committed May 23, 2024
1 parent 5c4bb9e commit 7c6a03c
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion deployment/pcm/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ kubectl logs ds/pcm
| perf-uncore | sys_perf_open() perf_paranoid<=0/privileged/CAP_ADMIN | PCM_USE_UNCORE_PERF | use perf for uncore | programPerfEvent(), PerfVirtualControlRegister() | |
| perf-topdown | /sys/bus/event_source/devices/cpu/events | sysMount | yes | cpucounters.cpp:perfSupportsTopDown() | TODO: conflicts with sys/fs/resctrl |
| RDT | uses "msr" or "resctrl" interface | PCM_NO_RDT | yes | cpucounters.cpp:isRDTDisabled()/QOSMetricAvailable() | |
| resctrl | RW: /sys/fs/resctrl | PCM_USE_RESCTRL | yes | resctrl.cpp | resctrlHostMount |
| resctrl | RW: /sys/fs/resctrl | PCM_USE_RESCTRL | yes | resctrl.cpp | resctrlMount |
| watchdog | RO/RW: /proc/sys/kernel/nmi_watchdog | PCM_KEEP_NMI_WATCHDOG | yes (tries to disable)| src/cpucounters.cpp:disableNMIWatchdog() | |
| msr | RW: /dev/cpu/X/msr + privileged or CAP_ADMIN/CAP_RAWIO | PCM_NO_MSR | msr is disabled | msr.cpp:MsrHandle() | privileged or some method to access /dev/cpu |
| | RW: /dev/mem | ? | msr is disabled | cpucounters.cpp:initUncoreObjects, pci.cpp:PCIHandleM() | privileged or some method to access /dev/cpu |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ PCM_NO_RDT: 0 # Collect RDT data
PCM_USE_RESCTRL: 0 # using MSR (no resctrl)

# RDT metrics will be used by direct msr programming
resctrlHostMount: false
resctrlMount: false
resctrlInsideMount: false

# sys and pci mounts are required for uncore PMU devices discovery
Expand Down
4 changes: 2 additions & 2 deletions deployment/pcm/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ spec:
name: nmi-watchdog
readOnly: true # RW? # TODO
{{- end }}
{{- if .Values.resctrlHostMount }}
{{- if .Values.resctrlMount }}
- mountPath: /sys/fs/resctrl
name: sysfs-resctrl
{{- end }}
Expand Down Expand Up @@ -193,7 +193,7 @@ spec:
# hostPath:
# path: /sys/firmware/acpi/tables/MCFG
# {{- end }}
{{- if .Values.resctrlHostMount }}
{{- if .Values.resctrlMount }}
- name: sysfs-resctrl
hostPath:
path: /sys/fs/resctrl
Expand Down
2 changes: 1 addition & 1 deletion deployment/pcm/values-direct-privileged.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ PCM_NO_RDT: 0 # Enable RDT metrics ...
PCM_USE_RESCTRL: 0 # but using MSR (no resctrl filesystem)

# with privileged container addtional mounts aren't required
resctrlHostMount: false # with MSR resctrl mount is not needed
resctrlMount: false # with MSR resctrl mount is not needed
resctrlInsideMount: false
sysMount: false
pciMount: false
Expand Down
2 changes: 1 addition & 1 deletion deployment/pcm/values-vm.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ nmiWatchdogMount: true

# Disable RDT because is not avaiable for VM instances
PCM_NO_RDT: 1
resctrlHostMount: false
resctrlMount: false
2 changes: 1 addition & 1 deletion deployment/pcm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ PCM_NO_RDT: 0 # 0 - try to collect RDT data, enables local/remot
PCM_USE_RESCTRL: 1 # use Linux Perf instead of MSR access (more reliable)
# required for indirect RDT access, not available for VM only in baremetal
# do not mount by default RDT can be also accessed through direct MSR programming
resctrlHostMount: true # mount from external host
resctrlMount: true # mount from external host
resctrlInsideMount: false # TODO: mount inside with extra call to mount, requires image with mount installed - doesn't require

### -------------- Other (NMI handling and/or on VM/AWS)
Expand Down

0 comments on commit 7c6a03c

Please sign in to comment.