-----Original Message-----
>From: Jan Kiszka <[email protected]> 
>Sent: Wednesday, January 26, 2022 12:37 AM
>To: Chen, Hongzhan <[email protected]>; [email protected]
>Subject: Re: [PATCH] kernel-shark: Add plugin for handling Xenomai 
>cobalt_context_switch
>
>On 18.01.22 04:45, Hongzhan Chen wrote:
>> For Xenomai-cobalt enabled system, cobalt_switch_context means
>> that there is schedule and context switch in companion core(realtime
>> core), which we may need to do special treatment and take correct
>> action as main kernel sched_switch to visualize out-of-band state
>> of realtime tasks running in cobalt core. To achive our target,
>> we implement following:
>> 
>>    1. store corresponding cobalt_switch_context events into
>>       container data.
>>    2. modify pid stored in entry to be equal to next_pid to
>>       show correct color in cpu bar when cobalt_switch_context
>>       event happen.
>>    3. show cobalt blue hollow box to mark out-of-band state according
>>       to cobalt_switch_context events.
>>    4. clickable cobalt_switch_context plugin shapes.
>> 
>> Signed-off-by: Hongzhan Chen <[email protected]>
>> 
>> diff --git a/src/plugins/CMakeLists.txt b/src/plugins/CMakeLists.txt
>> index 3e170fa..16f080b 100644
>> --- a/src/plugins/CMakeLists.txt
>> +++ b/src/plugins/CMakeLists.txt
>
>This is targeting the kernel-shark repo, right? But isn't the plan 
>rather to have this as an out-of-tree (/wrt kernel-shark) plugin, hosted 
>inside the Xenomai repo? Then we need a different structure and a hook

Yes. As one of plugin in kernelshark, it should be finial patch based on 
updated 
upstream kernel-shark source because there is some modification to
avoid duplication compared to last one that I submitted to our xenomai
community. 

If there is no more comments to modify or improve the patch itself as 
kernel-shark plugin, 
I would go forward and make a new patch hosted inside xenomai repo as we 
suggested. 

Regards

Hongzhan Chen


 
>into the Xenomai build system - or at least a README how to build that 
>manually.
>
Jan

-- 
Siemens AG, Technology
Competence Center Embedded Linux

Reply via email to