linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Fengguang Wu <wfg@mail.ustc.edu.cn>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Michael Rubin <mrubin@google.com>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [patch] Converting writeback linked lists to a tree based data structure
Date: Wed, 16 Jan 2008 12:55:07 +0800	[thread overview]
Message-ID: <400459376.04290@ustc.edu.cn> (raw)
Message-ID: <E1JF0It-0000yD-Mi@localhost.localdomain> (raw)
In-Reply-To: <20080115204236.6349ac48.akpm@linux-foundation.org>

On Tue, Jan 15, 2008 at 08:42:36PM -0800, Andrew Morton wrote:
> On Wed, 16 Jan 2008 12:25:53 +0800 Fengguang Wu <wfg@mail.ustc.edu.cn> wrote:
> 
> > list_heads are OK if we use them for one and only function.
> 
> Not really.  They're inappropriate when you wish to remember your
> position in the list while you dropped the lock (as we must do in
> writeback).
> 
> A data structure which permits us to interate across the search key rather
> than across the actual storage locations is more appropriate.

I totally agree with you. What I mean is to first do the split of
functions - into three: ordering, starvation prevention, and blockade
waiting. Then to do better ordering by adopting radix tree(or rbtree
if radix tree is not enough), and lastly get rid of the list_heads to
avoid locking. Does it sound like a good path?

--
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:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2008-01-16  4:55 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-15  8:09 [patch] Converting writeback linked lists to a tree based data structure Michael Rubin, Michael Rubin
2008-01-15  8:46 ` Peter Zijlstra
2008-01-15 17:53   ` Michael Rubin
     [not found]     ` <E1JEyWa-0001Ys-F9@localhost.localdomain>
2008-01-16  3:01       ` Fengguang Wu
2008-01-16  3:44         ` Andrew Morton
     [not found]           ` <E1JEzqb-0003YX-Rg@localhost.localdomain>
2008-01-16  4:25             ` Fengguang Wu
2008-01-16  4:42               ` Andrew Morton
     [not found]                 ` <E1JF0It-0000yD-Mi@localhost.localdomain>
2008-01-16  4:55                   ` Fengguang Wu [this message]
2008-01-16  5:51                     ` Andrew Morton
     [not found]                       ` <E1JF4Ey-0000x4-5p@localhost.localdomain>
2008-01-16  9:07                         ` Fengguang Wu
2008-01-18  7:36                           ` Mike Waychison
2008-01-16 22:35                         ` David Chinner
     [not found]                           ` <E1JFLEW-0002oE-G1@localhost.localdomain>
2008-01-17  3:16                             ` Fengguang Wu
2008-01-17  5:21                             ` David Chinner
2008-01-16  7:55           ` David Chinner
2008-01-16  8:13             ` Andrew Morton
     [not found]               ` <E1JF7yp-0006l8-5P@localhost.localdomain>
2008-01-16 13:06                 ` Fengguang Wu
2008-01-16 18:55         ` Michael Rubin
     [not found]           ` <E1JFLTR-0002pn-4Y@localhost.localdomain>
2008-01-17  3:31             ` Fengguang Wu
     [not found] ` <E1JFRFm-00011Q-0q@localhost.localdomain>
2008-01-17  9:41   ` Fengguang Wu
2008-01-17 21:07     ` Michael Rubin
     [not found]       ` <E1JFjGz-0001eU-3O@localhost.localdomain>
2008-01-18  4:56         ` Fengguang Wu
2008-01-18  5:41           ` Andi Kleen
     [not found]             ` <E1JFkHy-0001jR-VD@localhost.localdomain>
2008-01-18  6:01               ` Fengguang Wu
2008-01-18  7:48             ` Mike Waychison
2008-01-18  6:43           ` Michael Rubin
     [not found]             ` <E1JFnZz-00015z-Vq@localhost.localdomain>
2008-01-18  9:32               ` Fengguang Wu
2008-01-18  5:01       ` David Chinner
2008-01-18  5:38         ` Michael Rubin
2008-01-18  8:54           ` David Chinner
2008-01-18  9:26             ` Michael Rubin
     [not found]         ` <E1JFjyv-0001hU-FA@localhost.localdomain>
2008-01-18  5:41           ` Fengguang Wu
2008-01-19  2:50           ` David Chinner
  -- strict thread matches above, loose matches on Subject: below --
2007-12-13  0:32 Michael Rubin, Michael Rubin

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=400459376.04290@ustc.edu.cn \
    --to=wfg@mail.ustc.edu.cn \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mrubin@google.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).