From: Andi Kleen <andi@firstfloor.org>
To: holt@sgi.com
Cc: Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
Thomas Gleixner <tglx@linutronix.de>,
Venkatesh Pallipadi <venkatesh.pallipadi@gmail.com>,
Suresh Siddha <suresh.b.siddha@intel.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
x86@kernel.org, Rafael Wysocki <rjw@novell.com>
Subject: Re: [patch 0/2] x86,pat: Reduce contention on the memtype_lock -V4
Date: Wed, 24 Mar 2010 03:16:14 +0100 [thread overview]
Message-ID: <87wrx2for5.fsf@basil.nowhere.org> (raw)
In-Reply-To: <20100324003608.811051277@gulag1.americas.sgi.com> (holt@sgi.com's message of "Tue, 23 Mar 2010 19:36:08 -0500")
holt@sgi.com writes:
> Tracking memtype on x86 uses a single global spin_lock for either reading
> or changing the memory type. This includes changes made to page flags
> which is perfectly parallel.
>
> Part one of the patchset makes the page-based tracking use cmpxchg
> without a need for a lock.
>
> Part two of the patchset converts the spin_lock into a read/write lock.
I'm curious: in what workloads did you see contention?
For any scalability patches it would be always good to have a description
of the workload.
-Andi
--
ak@linux.intel.com -- Speaking for myself only.
next prev parent reply other threads:[~2010-03-24 2:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-24 0:36 [patch 0/2] x86,pat: Reduce contention on the memtype_lock -V4 holt
2010-03-24 0:36 ` [patch 1/2] x86,pat Update the page flags for memtype atomically instead of using memtype_lock. -V4 holt
2010-03-24 0:36 ` [patch 2/2] x86,pat Convert memtype_lock into an rw_lock holt
2010-03-24 2:16 ` Andi Kleen [this message]
2010-03-24 8:55 ` [patch 0/2] x86,pat: Reduce contention on the memtype_lock -V4 Robin Holt
2010-03-24 11:15 ` Thomas Gleixner
2010-03-24 16:22 ` Suresh Siddha
2010-03-24 20:32 ` Rafael J. Wysocki
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=87wrx2for5.fsf@basil.nowhere.org \
--to=andi@firstfloor.org \
--cc=holt@sgi.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=rjw@novell.com \
--cc=suresh.b.siddha@intel.com \
--cc=tglx@linutronix.de \
--cc=venkatesh.pallipadi@gmail.com \
--cc=x86@kernel.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.