Rialto Server profiling by smudri85 · Pull Request #441 · rdkcentral/rialto

smudri85 · 2026-02-06T14:37:24Z

Summary: Add profiling to Rialto Server.
Type: Feature
Test Plan: UT/CT, Fullstack
Jira: RDKEMW-11764

github-actions · 2026-02-06T14:37:36Z

Pull request must be merged with a description containing the required fields,

Summary:
Type: Feature/Fix/Cleanup
Test Plan:
Jira:

If there is no jira releated to this change, please put 'Jira: NO-JIRA'.

Description can be changed by editing the top comment on your pull request and making a new commit.

github-actions · 2026-02-06T14:38:08Z

common/source/Profiler.cpp:98:0: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm]
if (record.stage == stage)
^
common/source/Profiler.cpp:111:0: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm]
if (record.stage == stage && record.info == info)
^
common/source/Profiler.cpp:184:0: style: Consider using std::find_if algorithm instead of a raw loop. [useStlAlgorithm]
if (record.id == id)
^
nofile:0:0: information: Active checkers: 161/592 (use --checkers-report= to see details) [checkersReport]

rdkcmf-jenkins · 2026-02-06T14:41:11Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/441/rdkcentral/rialto
Commit: 282cf54

Report detail: gist'

Copilot

Pull request overview

This pull request adds profiling functionality to the Rialto Server to track performance metrics and pipeline events. The implementation includes a generic profiling interface in the common module and a GStreamer-specific profiler that integrates with the media player pipeline. The profiler records timestamped events at various stages of media playback and can be enabled via environment variable.

Changes:

Added core profiling infrastructure with IProfiler interface and Profiler implementation
Integrated GstProfiler into the media player pipeline to track key events
Added unit tests and mocks for the profiling functionality

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 25 comments.

Show a summary per file

File	Description
common/interface/IProfiler.h	Defines the profiler interface with methods for recording, finding, logging, and dumping profiling records
common/include/Profiler.h	Declares the concrete Profiler class implementation with internal Record structure
common/source/Profiler.cpp	Implements the profiler with thread-safe record storage, environment-based enabling, and file dumping
common/CMakeLists.txt	Adds Profiler.cpp to the build configuration
media/server/gstplayer/include/GstProfiler.h	Defines GStreamer-specific profiler wrapper for pipeline profiling
media/server/gstplayer/include/GenericPlayerContext.h	Adds GstProfiler instance to player context
media/server/gstplayer/source/GstProfiler.cpp	Implements GStreamer profiler with pad probe callbacks and element filtering
media/server/gstplayer/source/GstGenericPlayer.cpp	Integrates profiler into pipeline lifecycle (creation, termination, segment handling)
media/server/gstplayer/source/tasks/generic/HandleBusMessage.cpp	Records pipeline state changes
media/server/gstplayer/source/tasks/generic/FinishSetupSource.cpp	Records when all sources are attached
media/server/gstplayer/source/tasks/generic/AttachSource.cpp	Records AppSrc element creation
media/server/gstplayer/source/tasks/generic/DeepElementAdded.cpp	Schedules profiling for dynamically added GStreamer elements
media/server/gstplayer/CMakeLists.txt	Adds GstProfiler.cpp to build and includes common headers
tests/unittests/common/unittests/ProfilerTests.cpp	Unit tests for core profiler functionality
tests/unittests/common/unittests/CMakeLists.txt	Adds ProfilerTests.cpp to test build
tests/unittests/common/mocks/ProfilerMock.h	Mock implementation of IProfiler for testing
tests/unittests/common/mocks/ProfilerFactoryMock.h	Mock implementation of IProfilerFactory for testing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

media/server/gstplayer/source/tasks/generic/AttachSource.cpp

common/interface/IProfiler.h

media/server/gstplayer/source/GstProfiler.cpp

tests/unittests/common/unittests/ProfilerTests.cpp

common/source/Profiler.cpp

media/server/gstplayer/source/GstGenericPlayer.cpp

media/server/gstplayer/source/GstProfiler.cpp

tests/unittests/common/mocks/ProfilerFactoryMock.h

media/server/gstplayer/include/GstProfiler.h

rdkcmf-jenkins · 2026-02-06T14:59:35Z

Coverity Issue - Uncaught exception

An exception of type "std::bad_optional_access" is thrown but the exception specification "/implicit/noexcept" doesn't allow it to be thrown. This will result in a call to terminate().

Medium Impact, CWE-248
UNCAUGHT_EXCEPT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
media/server/gstplayer/source/GstGenericPlayer.cpp:210

media/server/gstplayer/source/GstProfiler.cpp

mhughesacn · 2026-02-06T17:58:18Z

Hi @smudri85 : The blackduck issue is not a problem but all new code files should have proper headers please.

MartinH, RDK CMF Compliance Team

rdkcmf-jenkins · 2026-02-06T23:22:15Z

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

Upvote reason: ok
Commit: 282cf54
'

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 10 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

media/server/gstplayer/source/GstProfiler.cpp

common/source/Profiler.cpp

media/server/gstplayer/source/tasks/generic/AttachSource.cpp

media/server/gstplayer/source/GstProfiler.cpp

media/server/gstplayer/include/GenericPlayerContext.h

media/server/gstplayer/CMakeLists.txt

tests/unittests/common/unittests/ProfilerTests.cpp

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 8 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tests/unittests/common/unittests/ProfilerTests.cpp

media/server/gstplayer/source/tasks/generic/FinishSetupSource.cpp

media/server/gstplayer/source/tasks/generic/DeepElementAdded.cpp

media/server/gstplayer/source/tasks/generic/HandleBusMessage.cpp

media/server/gstplayer/source/tasks/generic/AttachSource.cpp

media/server/gstplayer/source/GstProfiler.cpp

media/server/gstplayer/include/GenericPlayerContext.h

tests/unittests/common/unittests/ProfilerTests.cpp

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 7 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

media/server/gstplayer/source/GstProfiler.cpp

media/server/gstplayer/include/GenericPlayerContext.h

media/server/gstplayer/source/tasks/generic/AttachSource.cpp

common/source/Profiler.cpp

tests/unittests/common/unittests/ProfilerTests.cpp

media/server/gstplayer/include/GstProfiler.h

github-actions · 2026-02-12T13:49:38Z

Coverage statistics of your commit:
WARNING: Lines coverage decreased from: 85.2% to 84.6%
WARNING: Functions coverage decreased from: 92.5% to 91.9%

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-117641

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

rdkcmf-jenkins · 2026-04-03T19:45:00Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/441/rdkcentral/rialto
Commit: 024c46d

Report detail: gist'

Copilot

Pull request overview

Copilot reviewed 23 out of 23 changed files in this pull request and generated 11 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

media/server/gstplayer/source/GstProfiler.cpp

media/server/gstplayer/source/GstGenericPlayer.cpp

media/server/gstplayer/include/GenericPlayerContext.h

common/source/Profiler.cpp

common/interface/IProfiler.h

tests/unittests/media/server/gstplayer/profiler/GstProfilerTests.cpp

common/include/Profiler.h

github-actions · 2026-04-03T19:52:41Z

Coverage statistics of your commit:
WARNING: Lines coverage decreased from: 84.4% to 83.6%
WARNING: Functions coverage decreased from: 92.5% to 91.9%

rdkcmf-jenkins · 2026-04-04T17:40:06Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/441/rdkcentral/rialto
Commit: 0220ef0

Report detail: gist'

github-actions · 2026-04-04T17:49:11Z

Coverage statistics of your commit:
WARNING: Lines coverage decreased from: 84.4% to 84.0%
WARNING: Functions coverage decreased from: 92.5% to 92.4%

rdkcmf-jenkins · 2026-04-04T19:13:10Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/441/rdkcentral/rialto
Commit: d79ac19

Report detail: gist'

Copilot

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

common/source/Profiler.cpp

github-actions · 2026-04-04T19:20:52Z

Coverage statistics of your commit:
WARNING: Lines coverage decreased from: 84.4% to 84.3%
Congratulations, your commit improved functions coverage from: 92.5% to 92.6%

rdkcmf-jenkins · 2026-04-04T23:12:06Z

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

Upvote reason: OK
Commit: d79ac19
'

github-actions · 2026-04-05T19:44:01Z

Coverage statistics of your commit:
Lines coverage stays unchanged and is: 84.4%
Congratulations, your commit improved functions coverage from: 92.5% to 92.6%

Copilot AI review requested due to automatic review settings February 6, 2026 14:37

Copilot started reviewing on behalf of smudri85 February 6, 2026 14:37 View session

Copilot AI reviewed Feb 6, 2026

View reviewed changes

rdkcmf-jenkins reviewed Feb 6, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings February 11, 2026 12:19

smudri85 force-pushed the feature/RDKEMW-11764 branch from 95cc046 to 43eb771 Compare February 11, 2026 12:19

Copilot started reviewing on behalf of smudri85 February 11, 2026 12:20 View session

Copilot AI reviewed Feb 11, 2026

View reviewed changes

smudri85 force-pushed the feature/RDKEMW-11764 branch from 43eb771 to e476ba7 Compare February 11, 2026 13:45

Copilot AI review requested due to automatic review settings February 11, 2026 14:06

smudri85 force-pushed the feature/RDKEMW-11764 branch from e476ba7 to 5862d6c Compare February 11, 2026 14:06

Copilot started reviewing on behalf of smudri85 February 11, 2026 14:06 View session

Copilot AI reviewed Feb 11, 2026

View reviewed changes

smudri85 force-pushed the feature/RDKEMW-11764 branch 2 times, most recently from 3934f05 to 761b259 Compare February 12, 2026 09:18

Copilot AI review requested due to automatic review settings February 12, 2026 09:18

Copilot started reviewing on behalf of smudri85 February 12, 2026 09:18 View session

Copilot AI reviewed Feb 12, 2026

View reviewed changes

smudri85 force-pushed the feature/RDKEMW-11764 branch from 761b259 to 3d607f2 Compare February 12, 2026 13:28

Sasa Mudri added 4 commits April 3, 2026 12:57

Added Profiler and GstProfiler classes.

b65cc65

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Updated Profiler and GstProfiler classes.

fec5d93

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-117641

Added IProfiler interface, dump() method and additional changes.

b91ef13

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Added Profiler configuration (enable/disable) via environment variable.

ef91344

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Sasa Mudri added 8 commits April 3, 2026 12:57

Added descriptions for IProfiler and some small fixes.

94c40c1

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Added mocks and unit tests for IProfiler.

48a21f8

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Tiny change for deep element record stage name.

9e141fc

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Added 'github actions' compliance changes.

69224c8

Summary: Add profiling to Rialto Server. Type: Feature Test Plan: UT/CT, Fullstack Jira: RDKEMW-11764

Handle rialto decryptor element in GstProfiler.

0e5af34

Changing to using system clock for profiler timestamp values.

18f7196

Create tune data and log in one line.

7be9902

Added UTs for GstProfiler and modified existing ones for Profiler.

024c46d

Copilot AI review requested due to automatic review settings April 3, 2026 19:42

smudri85 force-pushed the feature/RDKEMW-11764 branch from 3d607f2 to 024c46d Compare April 3, 2026 19:42

Copilot started reviewing on behalf of smudri85 April 3, 2026 19:42 View session

Copilot AI reviewed Apr 3, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings April 4, 2026 19:10

smudri85 force-pushed the feature/RDKEMW-11764 branch from 0220ef0 to d79ac19 Compare April 4, 2026 19:10

Copilot started reviewing on behalf of smudri85 April 4, 2026 19:10 View session

Copilot AI reviewed Apr 4, 2026

View reviewed changes

common/source/Profiler.cpp Show resolved Hide resolved

Resolve issues reported by Copilot.

ff86fca

smudri85 force-pushed the feature/RDKEMW-11764 branch from d79ac19 to ff86fca Compare April 5, 2026 19:34

smudri85 requested review from DouglasAdler and skywojciechowskim April 6, 2026 02:16

Conversation

smudri85 commented Feb 6, 2026

Uh oh!

github-actions bot commented Feb 6, 2026

Uh oh!

github-actions bot commented Feb 6, 2026

Uh oh!

rdkcmf-jenkins commented Feb 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rdkcmf-jenkins commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverity Issue - Uncaught exception

Issue location

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mhughesacn commented Feb 6, 2026

Uh oh!

rdkcmf-jenkins commented Feb 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Feb 12, 2026

Uh oh!

rdkcmf-jenkins commented Apr 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

rdkcmf-jenkins commented Feb 6, 2026 •

edited

Loading