[lkml]   [2007]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Mark timer_stats as incompatible with multiple pid namespaces
    Ingo Molnar <> writes:

    > * Eric W. Biederman <> wrote:
    >> > the problem is, this interface stores historic PIDs too - i.e. PIDs
    >> > of tasks that might have exited already.
    >> Well struct pid * works in that case if you grab the reference to it.
    > but the display of the stats might happen much later. The point of this
    > API is to save pid+comm, which gives users a good idea about what caused
    > the events in the past - without having to pin any resource of that
    > task.

    Likewise struct pid is designed not to be a problem if pinned. It is a
    little heavier then it used to be with the addition of pid namespace
    support but not much. And if it is to heavy struct pid needs to be

    Holding the struct pid very much does not pin the task struct, and it
    shouldn't pin any other resources. I agree 64bytes or so is a bit
    more to pin then 4 bytes but it really isn't a lot.

    >> {
    >> timer->start_site = NULL;
    >> + pit_pid(timer->start_pid);
    >> + timer->start_pid = NULL;
    > s/pit/put, right?

    Yes. Clearly I haven't tested it yet.


     \ /
      Last update: 2007-12-13 22:33    [W:0.020 / U:5.524 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site