[DRAFT] Transformation memory profiler

[mryzhov]

Details:

• item1
• ...

Tickets:

• ticket-id

[Copilot]

Pull Request Overview

This PR adds a memory profiler to track memory usage during transformation passes. It implements functionality to monitor and report memory consumption at the process level, providing insights into which passes consume the most memory.

• Adds memory information tracking with RSS and virtual memory monitoring
• Implements profiling infrastructure to measure memory usage before and after each pass
• Adds summary reporting to identify passes with highest memory consumption

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File	Description
src/core/src/pass/manager.cpp	Implements memory profiling logic, adds getProcessMemoryInfo function and memory tracking in pass execution
src/core/include/openvino/pass/manager.hpp	Declares MemoryInfo struct and getProcessMemoryInfo function for memory profiling interface

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Corrected spelling of 'Mbytes' to 'MB' for standard unit abbreviation.

Suggested change

	info.vm_rss_bytes /= 1024; // KB → Mbytes
	info.vm_rss_bytes /= 1024; // KB → MB

[Copilot]

The division by 1024 converts from KB to bytes, but the comment says 'KB → Mbytes'. The variable name suggests bytes, so the conversion should be *= 1024 to convert KB to bytes, or rename the variable to indicate MB units.

Suggested change

	info.vm_rss_bytes /= 1024; // KB → Mbytes
	} else if (line.rfind("VmSize:", 0) == 0) {
	std::sscanf(line.c_str(), "VmSize: %zu", &info.vm_size_bytes);
	info.vm_size_bytes /= 1024;
	info.vm_rss_bytes *= 1024; // KB → bytes
	} else if (line.rfind("VmSize:", 0) == 0) {
	std::sscanf(line.c_str(), "VmSize: %zu", &info.vm_size_bytes);
	info.vm_size_bytes *= 1024; // KB → bytes

[Copilot]

Same issue as with vm_rss_bytes - the division converts KB to a smaller unit but the variable name suggests bytes. Either multiply by 1024 to get bytes or rename to indicate the actual unit.

Suggested change

	info.vm_rss_bytes /= 1024; // KB → Mbytes
	} else if (line.rfind("VmSize:", 0) == 0) {
	std::sscanf(line.c_str(), "VmSize: %zu", &info.vm_size_bytes);
	info.vm_size_bytes /= 1024;
	info.vm_rss_bytes *= 1024; // KB → bytes
	} else if (line.rfind("VmSize:", 0) == 0) {
	std::sscanf(line.c_str(), "VmSize: %zu", &info.vm_size_bytes);
	info.vm_size_bytes *= 1024;

[Copilot]

Both conditions check the same value (vm_rss_bytes). The first condition should check vm_size_bytes for VM memory tracking, and the second should check vm_rss_bytes for RSS tracking.

Suggested change

	if(mem_info_after.vm_rss_bytes > max_vm_consumer.vm_rss) {
	if(mem_info_after.vm_size_bytes > max_vm_consumer.vm) {

[Copilot]

Both conditions check the same value (vm_rss_bytes). The first condition should check vm_size_bytes for VM memory tracking, and the second should check vm_rss_bytes for RSS tracking.

[Copilot]

Static member variables should be declared in the private section of the class, not in the middle of the public section. Move this declaration to the private section with other member variables.