Re: [tracecompass-dev] Thread Priority in KernelStateProvider

[Matthew Khouzam <matthew.khouzam@xxxxxxxxxxxx>]

Shooting an idea, if the priority is an aspect(tm) then we can have a
column in the events table show this.

Would that work out?

Matt

On 15-02-13 02:29 PM, Alexandre Montplaisir wrote:
> Hi Chris,
>
> In fact we're trying to move away from having columns in views like
> these, so that we can eventually have a kind of "unified time axis",
> as described at [1]. If we want to display extra information for each
> entry, we could do so in a dedicated view, for example the Properties
> view.
>
> What do you think of adding the priority information to the tooltip,
> when the user mouses-over a state in the Control Flow View? Would it
> be sufficient?
>
>
> Cheers,
> Alexandre
>
> [1]
> https://wiki.eclipse.org/Trace_Compass/Design_Documents/Aligned_x_axis
>
>
> On 02/13/2015 02:06 PM, christian mansky wrote:
>> Hi Matthew,
>> thanks for the reply. To be honest up until now I have not given the
>> data output much thought, but now that you mention it ... I want a
>> quick overview of the current thread when stepping through a kernel
>> trace. Since the Control Flow View already highlights the current
>> thread, what about a new column "Prio" next to the process name or
>> after the ptid column?
>> Regards Chris
>>
>>
>>
>>> Date: Fri, 13 Feb 2015 10:47:46 -0500
>>> From: Matthew Khouzam <matthew.khouzam@xxxxxxxxxxxx>
>>> To: tracecompass developer discussions <tracecompass-dev@xxxxxxxxxxx>
>>> Subject: Re: [tracecompass-dev] Thread Priority in KernelStateProvider
>>> Message-ID: <54DE1CA2.4040109@xxxxxxxxxxxx>
>>> Content-Type: text/plain; charset="windows-1252"
>>>
>>> Looks great as an idea! How do you plan on displaying this? I
>>> personally
>>> encourage writing patches, they are very welcome. I just want to make
>>> sure before you embark on that journey that you already know how to
>>> output the data to the user.
>>> If you have any questions, don't hesitate to ask.
>>> BR
>>> Matthew
>>> On 15-02-13 04:15 AM, Mansky Christian wrote:
>>>> Hi everyone,
>>>> I'm new to Trace Compass and I looked into your great tool for some
>>>> time now. I'd like to propose a change (or an enhancement) to the
>>>> KernelStateProvider
>>>> (org.eclipse.tracecompass.analysis.os.linux.core.kernelanalysis). I
>>>> miss the current priority of a thread. In case of a sched_switch event
>>>> the next_prio field contains the current thread priority. Of course
>>>> the sched_pi_setprio event (only applied from a thread on itself) and
>>>> a change of priority from the usermode (most likely a syscall, and
>>>> again applied on itself) must also be taken into account.
>>>> In your opinion, is this a worthy enhancemet? If so, I'll look into it
>>>> a bit deeper trying to provide a patch?
>>>>
>>>>   Thanks in advance and best regards,
>>>> Chris
>>
>>
>> ------------------------------
>>
>> _______________________________________________
>> tracecompass-dev mailing list
>> tracecompass-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or
>> unsubscribe from this list, visit
>> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev[https://dev.eclipse.org/mailman/listinfo/tracecompass-dev]
>>
>>
>> _______________________________________________
>> tracecompass-dev mailing list
>> tracecompass-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or
>> unsubscribe from this list, visit
>> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
>
> _______________________________________________
> tracecompass-dev mailing list
> tracecompass-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or
> unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev