lkml.org 
[lkml]   [1998]   [Sep]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: [PATCH] Faster scheduler and separate RT run queue
    Linus Torvalds writes:
    >
    > On Wed, 30 Sep 1998, Richard Gooch wrote:
    > >
    > > Linus: I'd like to get your opinion of this patch.
    >
    > I certainly like the implementation. It's of the "this is how I
    > would have wanted it done" category, and essentially looks like
    > something I could be very comfortable with.

    Thanks. I spent a couple of hours thinking hard about how to make it
    both elegant and efficient.

    > If it wasn't for me not wanting to do this before the code-freeze,
    > it would already be in my kernel. As it is, I'll look at the patch a
    > bit more.

    Understood. I'm far more interested in getting devfs included prior to
    2.2. The last you said anything about this was that it would "probably
    go in". Is this still your feeling? Any comments about the timescale?

    If you decide you don't want this change for 2.1, can you at least
    please include the bugfix for RT processes yielding to lower priority
    processes? What we have now is contrary to POSIX.4.

    The hunk in question is:

    @@ -1488,7 +1512,8 @@
    {
    spin_lock(&scheduler_lock);
    spin_lock_irq(&runqueue_lock);
    - current->policy |= SCHED_YIELD;
    + if (current->policy == SCHED_OTHER)
    + current->policy |= SCHED_YIELD;
    current->need_resched = 1;
    move_last_runqueue(current);
    spin_unlock_irq(&runqueue_lock);

    although looking again I'd rather have:

    @@ -1488,7 +1512,8 @@
    {
    spin_lock(&scheduler_lock);
    spin_lock_irq(&runqueue_lock);
    - current->policy |= SCHED_YIELD;
    + if (current->policy == SCHED_OTHER)
    + current->policy = SCHED_OTHER | SCHED_YIELD;
    current->need_resched = 1;
    move_last_runqueue(current);
    spin_unlock_irq(&runqueue_lock);

    Regards,

    Richard....
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.rutgers.edu
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2005-03-22 13:44    [W:0.021 / U:29.564 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site