All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Whitcroft <apw@shadowen.org>
To: Andi Kleen <ak@suse.de>
Cc: Dave Hansen <hansendc@us.ibm.com>,
	Christoph Lameter <clameter@sgi.com>,
	linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
	Martin Bligh <mbligh@google.com>,
	linux-mm@kvack.org,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Subject: Re: [PATCH 1/4] x86_64: Switch to SPARSE_VIRTUAL
Date: Thu, 05 Apr 2007 13:07:25 +0100	[thread overview]
Message-ID: <4614E67D.7000006@shadowen.org> (raw)
In-Reply-To: <200704030031.24898.ak@suse.de>

Andi Kleen wrote:
> On Monday 02 April 2007 23:56:08 Dave Hansen wrote:
>> On Mon, 2007-04-02 at 14:28 -0700, Christoph Lameter wrote:
>>> I do not care what its called as long as it 
>>> covers all the bases and is not a glaring performance regresssion (like 
>>> SPARSEMEM so far). 
>> I honestly don't doubt that there are regressions, somewhere.  Could you
>> elaborate, and perhaps actually show us some numbers on this?  Perhaps
>> instead of adding a completely new model, we can adapt the existing ones
>> somehow.
> 
> If it works I would be inclined to replaced old sparsemem with Christoph's
> new one on x86-64. Perhaps that could cut down the bewildering sparsemem
> ifdef jungle that is there currently.
> 
> But I presume it won't work on 32bit because of the limited address space?

Right.  But we might be able to do switch SPARSEMEM_EXTREME users here
if performance is better and no other regressions are detected.

There seems to be a theme, we need to get some numbers.  I will try and
get what I can with the hardware I have and see whats missing.

> 
>> But, without some cold, hard, data, we mere mortals without the 1024-way
>> machines can only guess. ;)


-apw

WARNING: multiple messages have this Message-ID (diff)
From: Andy Whitcroft <apw@shadowen.org>
To: Andi Kleen <ak@suse.de>
Cc: Dave Hansen <hansendc@us.ibm.com>,
	Christoph Lameter <clameter@sgi.com>,
	linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
	Martin Bligh <mbligh@google.com>,
	linux-mm@kvack.org,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Subject: Re: [PATCH 1/4] x86_64: Switch to SPARSE_VIRTUAL
Date: Thu, 05 Apr 2007 13:07:25 +0100	[thread overview]
Message-ID: <4614E67D.7000006@shadowen.org> (raw)
In-Reply-To: <200704030031.24898.ak@suse.de>

Andi Kleen wrote:
> On Monday 02 April 2007 23:56:08 Dave Hansen wrote:
>> On Mon, 2007-04-02 at 14:28 -0700, Christoph Lameter wrote:
>>> I do not care what its called as long as it 
>>> covers all the bases and is not a glaring performance regresssion (like 
>>> SPARSEMEM so far). 
>> I honestly don't doubt that there are regressions, somewhere.  Could you
>> elaborate, and perhaps actually show us some numbers on this?  Perhaps
>> instead of adding a completely new model, we can adapt the existing ones
>> somehow.
> 
> If it works I would be inclined to replaced old sparsemem with Christoph's
> new one on x86-64. Perhaps that could cut down the bewildering sparsemem
> ifdef jungle that is there currently.
> 
> But I presume it won't work on 32bit because of the limited address space?

Right.  But we might be able to do switch SPARSEMEM_EXTREME users here
if performance is better and no other regressions are detected.

There seems to be a theme, we need to get some numbers.  I will try and
get what I can with the hardware I have and see whats missing.

> 
>> But, without some cold, hard, data, we mere mortals without the 1024-way
>> machines can only guess. ;)


-apw

--
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>

  parent reply	other threads:[~2007-04-05 12:07 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-01  7:10 [PATCH 1/2] Generic Virtual Memmap suport for SPARSEMEM Christoph Lameter
2007-04-01  7:10 ` Christoph Lameter
2007-04-01  7:10 ` [PATCH 1/4] x86_64: Switch to SPARSE_VIRTUAL Christoph Lameter
2007-04-01  7:10   ` Christoph Lameter
2007-04-01  7:11   ` Christoph Lameter
2007-04-01  7:11     ` Christoph Lameter
2007-04-01 10:46   ` Andi Kleen
2007-04-01 10:46     ` Andi Kleen
2007-04-02 15:37     ` Christoph Lameter
2007-04-02 15:37       ` Christoph Lameter
2007-04-02 15:44       ` Andi Kleen
2007-04-02 15:44         ` Andi Kleen
2007-04-02 15:51         ` Martin J. Bligh
2007-04-02 15:51           ` Martin J. Bligh
2007-04-02 15:54         ` Christoph Lameter
2007-04-02 15:54           ` Christoph Lameter
2007-04-02 17:14           ` Andi Kleen
2007-04-02 17:14             ` Andi Kleen
2007-04-02 17:34             ` Christoph Lameter
2007-04-02 17:34               ` Christoph Lameter
2007-04-02 19:54           ` Dave Hansen
2007-04-02 19:54             ` Dave Hansen
2007-04-02 20:11             ` Christoph Lameter
2007-04-02 20:11               ` Christoph Lameter
2007-04-02 20:13       ` Dave Hansen
2007-04-02 20:13         ` Dave Hansen
2007-04-02 20:30         ` Christoph Lameter
2007-04-02 20:30           ` Christoph Lameter
2007-04-02 20:38           ` Martin Bligh
2007-04-02 20:38             ` Martin Bligh
2007-04-02 20:51             ` Christoph Lameter
2007-04-02 20:51               ` Christoph Lameter
2007-04-05 11:50               ` Andy Whitcroft
2007-04-05 11:50                 ` Andy Whitcroft
2007-04-05 18:27                 ` Christoph Lameter
2007-04-05 18:27                   ` Christoph Lameter
2007-04-07 22:06                   ` [PATCH 1/4] x86_64: (SPARSE_VIRTUAL doubles sparsemem speed) Christoph Lameter
2007-04-07 22:06                     ` Christoph Lameter
2007-04-07 22:18                     ` Andi Kleen
2007-04-07 22:18                       ` Andi Kleen
2007-04-09 16:40                     ` William Lee Irwin III
2007-04-09 16:40                       ` William Lee Irwin III
2007-04-09 17:16                       ` Christoph Lameter
2007-04-09 17:16                         ` Christoph Lameter
2007-04-09 18:20                         ` William Lee Irwin III
2007-04-09 18:20                           ` William Lee Irwin III
2007-04-02 21:08           ` [PATCH 1/4] x86_64: Switch to SPARSE_VIRTUAL Dave Hansen
2007-04-02 21:08             ` Dave Hansen
2007-04-02 21:28             ` Christoph Lameter
2007-04-02 21:28               ` Christoph Lameter
2007-04-02 21:43               ` Martin Bligh
2007-04-02 21:43                 ` Martin Bligh
2007-04-02 21:56               ` Dave Hansen
2007-04-02 21:56                 ` Dave Hansen
2007-04-02 22:29                 ` Christoph Lameter
2007-04-02 22:29                   ` Christoph Lameter
2007-04-02 22:31                 ` Andi Kleen
2007-04-02 22:31                   ` Andi Kleen
2007-04-02 22:37                   ` Christoph Lameter
2007-04-02 22:37                     ` Christoph Lameter
2007-04-02 22:41                     ` Martin Bligh
2007-04-02 22:41                       ` Martin Bligh
2007-04-02 22:49                       ` Christoph Lameter
2007-04-02 22:49                         ` Christoph Lameter
2007-04-02 22:52                         ` Martin Bligh
2007-04-02 22:52                           ` Martin Bligh
2007-04-05 12:07                   ` Andy Whitcroft [this message]
2007-04-05 12:07                     ` Andy Whitcroft
2007-04-04 21:27               ` Bob Picco
2007-04-04 21:27                 ` Bob Picco
2007-04-04 22:38                 ` Christoph Lameter
2007-04-04 22:38                   ` Christoph Lameter
2007-04-02 23:03       ` Bjorn Helgaas
2007-04-02 23:03         ` Bjorn Helgaas
2007-04-02 15:44     ` Christoph Lameter
2007-04-02 15:44       ` Christoph Lameter
2007-04-02 16:18     ` Christoph Lameter
2007-04-02 16:18       ` Christoph Lameter
2007-04-02 20:50 ` [PATCH 1/2] Generic Virtual Memmap suport for SPARSEMEM Dave Hansen
2007-04-02 20:50   ` Dave Hansen
2007-04-02 21:00   ` Christoph Lameter
2007-04-02 21:00     ` Christoph Lameter
2007-04-02 21:22     ` Dave Hansen
2007-04-02 21:22       ` Dave Hansen
2007-04-02 21:31       ` Christoph Lameter
2007-04-02 21:31         ` Christoph Lameter
2007-04-02 21:42         ` Dave Hansen
2007-04-02 21:42           ` Dave Hansen
2007-04-02 21:53           ` Christoph Lameter
2007-04-02 21:53             ` Christoph Lameter
2007-04-02 22:05             ` Dave Hansen
2007-04-02 22:05               ` Dave Hansen

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=4614E67D.7000006@shadowen.org \
    --to=apw@shadowen.org \
    --cc=ak@suse.de \
    --cc=clameter@sgi.com \
    --cc=hansendc@us.ibm.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mbligh@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 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.