All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dimitri Sivanich <sivanich@sgi.com>
To: Jesse Barnes <jbarnes@engr.sgi.com>
Cc: Christoph Hellwig <hch@infradead.org>,
	Andrew Morton <akpm@osdl.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH]: Option to run cache reap in thread mode
Date: Wed, 16 Jun 2004 11:51:55 -0500	[thread overview]
Message-ID: <20040616165155.GA6069@sgi.com> (raw)
In-Reply-To: <200406161225.11946.jbarnes@engr.sgi.com>

On Wed, Jun 16, 2004 at 12:25:11PM -0400, Jesse Barnes wrote:
> On Wednesday, June 16, 2004 12:07 pm, Christoph Hellwig wrote:
> > Well, if you want deterministic interrupt latencies you should go for a
> > realtime OS.
> 
> Although I don't want to see another kernel thread added as much as the next 
> guy, I think that minimizing the amount of time that irqs are turned off is 
> probably a good thing in general.  For example, the patch to allow interrupts 
> in spin_lock_irq if the lock is already taken is generally a really good 
> thing, because even though reducing lock contention should be a goal, locks 
> by their very nature are taken sometimes, and allowing other CPUs to get 
> useful work done while they're waiting for it is obviously desirable.
> 
> > I know Linux is the big thing in the industry, but you're 
> > really better off looking for a small Hard RT OS. 
> 
> Sure, for some applications, an RTOS is necessary.  But it seems like keeping 
> latencies down in Linux is a good thing to do nonetheless.
> 
> Can you think of other ways to reduce the length of time that interrupts are 
> disabled during cache reaping?  It seems like the cache_reap loop might be a 
> candidate for reorganization (though that would probably imply other 
> changes).

I have another patch forthcoming that does some reorganizing of the locking.
With the two patches I see substantial improvement.

> 
> Thanks,
> Jesse

WARNING: multiple messages have this Message-ID (diff)
From: Dimitri Sivanich <sivanich@sgi.com>
To: Jesse Barnes <jbarnes@engr.sgi.com>
Cc: Christoph Hellwig <hch@infradead.org>,
	Andrew Morton <akpm@osdl.org>,
	linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH]: Option to run cache reap in thread mode
Date: Wed, 16 Jun 2004 11:51:55 -0500	[thread overview]
Message-ID: <20040616165155.GA6069@sgi.com> (raw)
In-Reply-To: <200406161225.11946.jbarnes@engr.sgi.com>

On Wed, Jun 16, 2004 at 12:25:11PM -0400, Jesse Barnes wrote:
> On Wednesday, June 16, 2004 12:07 pm, Christoph Hellwig wrote:
> > Well, if you want deterministic interrupt latencies you should go for a
> > realtime OS.
> 
> Although I don't want to see another kernel thread added as much as the next 
> guy, I think that minimizing the amount of time that irqs are turned off is 
> probably a good thing in general.  For example, the patch to allow interrupts 
> in spin_lock_irq if the lock is already taken is generally a really good 
> thing, because even though reducing lock contention should be a goal, locks 
> by their very nature are taken sometimes, and allowing other CPUs to get 
> useful work done while they're waiting for it is obviously desirable.
> 
> > I know Linux is the big thing in the industry, but you're 
> > really better off looking for a small Hard RT OS. 
> 
> Sure, for some applications, an RTOS is necessary.  But it seems like keeping 
> latencies down in Linux is a good thing to do nonetheless.
> 
> Can you think of other ways to reduce the length of time that interrupts are 
> disabled during cache reaping?  It seems like the cache_reap loop might be a 
> candidate for reorganization (though that would probably imply other 
> changes).

I have another patch forthcoming that does some reorganizing of the locking.
With the two patches I see substantial improvement.

> 
> Thanks,
> Jesse
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>

  reply	other threads:[~2004-06-16 16:52 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-16 14:24 [PATCH]: Option to run cache reap in thread mode Dimitri Sivanich
2004-06-16 14:24 ` Dimitri Sivanich
2004-06-16 15:29 ` Christoph Hellwig
2004-06-16 15:29   ` Christoph Hellwig
2004-06-16 16:03   ` Dimitri Sivanich
2004-06-16 16:03     ` Dimitri Sivanich
2004-06-16 16:07     ` Christoph Hellwig
2004-06-16 16:07       ` Christoph Hellwig
2004-06-16 16:25       ` Jesse Barnes
2004-06-16 16:25         ` Jesse Barnes
2004-06-16 16:51         ` Dimitri Sivanich [this message]
2004-06-16 16:51           ` Dimitri Sivanich
2004-06-16 16:46 ` Lori Gilbertson
2004-06-16 16:46   ` Lori Gilbertson
2004-06-16 16:53   ` Christoph Hellwig
2004-06-16 16:53     ` Christoph Hellwig
2004-06-16 21:30 ` Andrew Morton
2004-06-16 21:30   ` Andrew Morton
     [not found] <27JKg-4ht-11@gated-at.bofh.it>
2004-06-16 16:22 ` Andi Kleen
2004-06-16 18:16   ` Dimitri Sivanich
2004-06-16 18:16     ` Dimitri Sivanich
  -- strict thread matches above, loose matches on Subject: below --
2004-06-16 16:43 Mark_H_Johnson
2004-06-16 16:43 ` Mark_H_Johnson
2004-06-16 17:23 Manfred Spraul
2004-06-16 18:02 ` Dimitri Sivanich
2004-06-16 18:02   ` Dimitri Sivanich
2004-06-16 18:58   ` Manfred Spraul
2004-06-16 18:58     ` Manfred Spraul
2004-06-17 13:10     ` Dimitri Sivanich
2004-06-17 13:10       ` Dimitri Sivanich
2004-06-18  4:40       ` Andrew Morton
2004-06-18  4:40         ` Andrew Morton
2004-06-18 14:33         ` Dimitri Sivanich
2004-06-18 14:33           ` Dimitri Sivanich
2004-06-18 20:40           ` Andrew Morton
2004-06-18 20:40             ` Andrew Morton
2004-06-18 21:04             ` Manfred Spraul
2004-06-18 21:04               ` Manfred Spraul
2004-06-18 21:44               ` Dimitri Sivanich
2004-06-18 21:44                 ` Dimitri Sivanich
2004-06-18 22:03               ` Andrew Morton
2004-06-18 22:03                 ` Andrew Morton

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040616165155.GA6069@sgi.com \
    --to=sivanich@sgi.com \
    --cc=akpm@osdl.org \
    --cc=hch@infradead.org \
    --cc=jbarnes@engr.sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.