From: Jes Sorensen <jes@sgi.com>
To: "Zhang, Xiantao" <xiantao.zhang@intel.com>
Cc: Avi Kivity <avi@qumranet.com>, "Luck, Tony" <tony.luck@intel.com>,
"Xu, Anthony" <anthony.xu@intel.com>,
Akio Takebe <takebe_akio@jp.fujitsu.com>,
kvm-devel@lists.sourceforge.net,
kvm-ia64-devel@lists.sourceforge.net, linux-ia64@vger.kernel.org,
virtualization@lists.linux-foundation.org,
Carsten Otte <cotte@de.ibm.com>
Subject: Re: [02/17][PATCH] Implement smp_call_function_mask for ia64 - V8
Date: Mon, 31 Mar 2008 09:12:56 +0000 [thread overview]
Message-ID: <47F0AB18.2010707@sgi.com> (raw)
In-Reply-To: <42DFA526FC41B1429CE7279EF83C6BDC01048240@pdsmsx415.ccr.corp.intel.com>
Zhang, Xiantao wrote:
>>From 697d50286088e98da5ac8653c80aaa96c81abf87 Mon Sep 17 00:00:00 2001
> From: Xiantao Zhang <xiantao.zhang@intel.com>
> Date: Mon, 31 Mar 2008 09:50:24 +0800
> Subject: [PATCH] KVM:IA64: Implement smp_call_function_mask for ia64
>
> This function provides more flexible interface for smp
> infrastructure.
> Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
Hi Xiantao,
I'm a little wary of the performance impact of this change. Doing a
cpumask compare on all smp_call_function calls seems a little expensive.
Maybe it's just noise in the big picture compared to the actual cost of
the IPIs, but I thought I'd bring it up.
Keep in mind that a cpumask can be fairly big these days, max NR_CPUS
is currently 4096. For those booting a kernel with NR_CPUS at 4096 on
a dual CPU machine, it would be a bit expensive.
Why not keep smp_call_function() the way it was before, rather than
implementing it via the call to smp_call_function_mask()?
Cheers,
Jes
WARNING: multiple messages have this Message-ID (diff)
From: Jes Sorensen <jes@sgi.com>
To: "Zhang, Xiantao" <xiantao.zhang@intel.com>
Cc: Avi Kivity <avi@qumranet.com>, "Luck, Tony" <tony.luck@intel.com>,
"Xu, Anthony" <anthony.xu@intel.com>,
Akio Takebe <takebe_akio@jp.fujitsu.com>,
kvm-devel@lists.sourceforge.net,
kvm-ia64-devel@lists.sourceforge.net, linux-ia64@vger.kernel.org,
virtualization@lists.linux-foundation.org,
Carsten Otte <cotte@de.ibm.com>
Subject: Re: [02/17][PATCH] Implement smp_call_function_mask for ia64 - V8
Date: Mon, 31 Mar 2008 11:12:56 +0200 [thread overview]
Message-ID: <47F0AB18.2010707@sgi.com> (raw)
In-Reply-To: <42DFA526FC41B1429CE7279EF83C6BDC01048240@pdsmsx415.ccr.corp.intel.com>
Zhang, Xiantao wrote:
>>From 697d50286088e98da5ac8653c80aaa96c81abf87 Mon Sep 17 00:00:00 2001
> From: Xiantao Zhang <xiantao.zhang@intel.com>
> Date: Mon, 31 Mar 2008 09:50:24 +0800
> Subject: [PATCH] KVM:IA64: Implement smp_call_function_mask for ia64
>
> This function provides more flexible interface for smp
> infrastructure.
> Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
Hi Xiantao,
I'm a little wary of the performance impact of this change. Doing a
cpumask compare on all smp_call_function calls seems a little expensive.
Maybe it's just noise in the big picture compared to the actual cost of
the IPIs, but I thought I'd bring it up.
Keep in mind that a cpumask can be fairly big these days, max NR_CPUS
is currently 4096. For those booting a kernel with NR_CPUS at 4096 on
a dual CPU machine, it would be a bit expensive.
Why not keep smp_call_function() the way it was before, rather than
implementing it via the call to smp_call_function_mask()?
Cheers,
Jes
next prev parent reply other threads:[~2008-03-31 9:12 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-28 9:59 [02/17][PATCH] Implement smp_call_function_mask for ia64 Zhang, Xiantao
2008-03-30 11:42 ` [kvm-ia64-devel] [02/17][PATCH] Implement smp_call_function_mask Avi Kivity
2008-03-30 11:42 ` [kvm-ia64-devel] [02/17][PATCH] Implement smp_call_function_mask for ia64 Avi Kivity
2008-03-31 1:40 ` Zhang, Xiantao
2008-03-31 1:40 ` Zhang, Xiantao
2008-03-30 11:42 ` Avi Kivity
2008-03-31 8:25 ` [02/17][PATCH] Implement smp_call_function_mask for ia64 - V8 Zhang, Xiantao
2008-03-31 9:12 ` Jes Sorensen [this message]
2008-03-31 9:12 ` Jes Sorensen
2008-03-31 10:17 ` [kvm-devel] [02/17][PATCH] Implement smp_call_function_mask foria64 " Zhang, Xiantao
2008-03-31 10:17 ` Zhang, Xiantao
2008-03-31 15:02 ` [02/17][PATCH] Implement smp_call_function_mask for ia64 " Jeremy Fitzhardinge
2008-03-31 15:02 ` Jeremy Fitzhardinge
2008-04-01 8:34 ` Jes Sorensen
2008-04-01 8:34 ` Jes Sorensen
2008-04-01 8:34 ` Jes Sorensen
2008-04-01 16:06 ` Jeremy Fitzhardinge
2008-04-01 16:06 ` Jeremy Fitzhardinge
2008-04-02 7:30 ` Jes Sorensen
2008-04-02 7:30 ` Jes Sorensen
2008-04-02 23:48 ` Luck, Tony
2008-04-02 23:48 ` Luck, Tony
2008-04-02 23:48 ` Luck, Tony
2008-04-02 7:30 ` Jes Sorensen
2008-04-01 16:06 ` Jeremy Fitzhardinge
2008-03-31 15:02 ` Jeremy Fitzhardinge
2008-03-31 9:12 ` Jes Sorensen
-- strict thread matches above, loose matches on Subject: below --
2008-03-31 8:25 Zhang, Xiantao
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=47F0AB18.2010707@sgi.com \
--to=jes@sgi.com \
--cc=anthony.xu@intel.com \
--cc=avi@qumranet.com \
--cc=cotte@de.ibm.com \
--cc=kvm-devel@lists.sourceforge.net \
--cc=kvm-ia64-devel@lists.sourceforge.net \
--cc=linux-ia64@vger.kernel.org \
--cc=takebe_akio@jp.fujitsu.com \
--cc=tony.luck@intel.com \
--cc=virtualization@lists.linux-foundation.org \
--cc=xiantao.zhang@intel.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.