lkml.org 
[lkml]   [2006]   [May]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: sched_clock() uses are broken
    On Sun, May 07, 2006 at 11:30:15PM +1000, Nick Piggin wrote:
    > Russell King wrote:
    > >On Sun, May 07, 2006 at 11:00:29PM +1000, Nick Piggin wrote:
    > >
    > >>Nick Piggin wrote:
    > >>
    > >>
    > >>>I stand by my first reply to your comment WRT the API.
    > >>
    > >>Actually, on rereading, it seems like I was a bit confused about
    > >>your proposal. I don't think you specified anyway the units
    > >>returned by your new sched_clock(). So it is identical to my
    > >>"corrected" interface :\
    > >
    > >
    > >Okay, so that presumably means we have to either stick with what we
    > >currently have, or go the whole hog and re-implement the sched_clock()
    > >support?
    > >
    > >IOW, my patch on 2nd May isn't of any use as it currently stands?
    >
    > IMO it would probably be best to try to re implement it in one go.
    > It shouldn't have spread too far out of kernel/sched.c, and the arch
    > code should mostly be implementable in terms of their sched_clock().
    > Mundane but not difficult.

    Having looked at this several times over the last couple of days, I've
    come to the conclusion that I'm not the right person to fix this problem.
    I've tried several methods of converting the code, but every time I
    remain unconvinced that the changes are provably correct as far as not
    missing something, so I end up throwing the changes away and starting
    again.

    Yes, I admit defeat.

    Maybe someone who cares about this stuff[1] (or who sees the problem)
    should look into it.

    [1] - eg, the original poster on linux-arm-kernel who indirectly pointed
    out the sched_clock() issue.

    --
    Russell King
    Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
    maintainer of: 2.6 Serial core
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-05-07 18:56    [from the cache]
    ©2003-2014 Jasper Spaans. Advertise on this site