* [PATCH] x86: add support for domain initiated global cache flush
@ 2010-04-29 15:59 Jan Beulich
2010-05-03 17:37 ` Konrad Rzeszutek Wilk
0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2010-04-29 15:59 UTC (permalink / raw)
To: xen-devel
[-- Attachment #1: Type: text/plain, Size: 2857 bytes --]
Newer Linux' AGP code wants to flush caches on all CPUs under certain
circumstances. Since doing this on all vCPU-s of the domain in
question doesn't yield the intended effect, this needs to be done in
the hypervisor. Add a new MMUEXT operation for this.
While looking at the pre-existing flush implementation, I also noticed
an off-by-one error in the cache flush portion of flush_area_local().
Signed-off-by: Jan Beulich <jbeulich@novell.com>
--- 2010-04-22.orig/xen/arch/x86/flushtlb.c 2008-10-14 19:44:12.000000000 +0200
+++ 2010-04-22/xen/arch/x86/flushtlb.c 2010-04-29 17:47:22.000000000 +0200
@@ -149,7 +149,7 @@ void flush_area_local(const void *va, un
{
unsigned long i, sz = 0;
- if ( order < (BITS_PER_LONG - PAGE_SHIFT - 1) )
+ if ( order < (BITS_PER_LONG - PAGE_SHIFT) )
sz = 1UL << (order + PAGE_SHIFT);
if ( c->x86_clflush_size && c->x86_cache_size && sz &&
--- 2010-04-22.orig/xen/arch/x86/mm.c 2010-04-29 17:35:20.000000000 +0200
+++ 2010-04-22/xen/arch/x86/mm.c 2010-04-29 17:51:08.000000000 +0200
@@ -2889,6 +2889,27 @@ int do_mmuext_op(
}
break;
+ case MMUEXT_FLUSH_CACHE_GLOBAL:
+ if ( unlikely(foreigndom != DOMID_SELF) )
+ okay = 0;
+ else if ( likely(cache_flush_permitted(d)) )
+ {
+ unsigned int cpu;
+ cpumask_t mask = CPU_MASK_NONE;
+
+ for_each_online_cpu(cpu)
+ if ( !cpus_intersects(mask,
+ per_cpu(cpu_sibling_map, cpu)) )
+ cpu_set(cpu, mask);
+ flush_mask(&mask, FLUSH_CACHE);
+ }
+ else
+ {
+ MEM_LOG("Non-physdev domain tried to FLUSH_CACHE_GLOBAL");
+ okay = 0;
+ }
+ break;
+
case MMUEXT_SET_LDT:
{
unsigned long ptr = op.arg1.linear_addr;
--- 2010-04-22.orig/xen/include/public/xen.h 2010-04-29 17:35:20.000000000 +0200
+++ 2010-04-22/xen/include/public/xen.h 2010-04-29 17:09:30.000000000 +0200
@@ -239,6 +239,10 @@ DEFINE_XEN_GUEST_HANDLE(xen_pfn_t);
*
* cmd: MMUEXT_FLUSH_CACHE
* No additional arguments. Writes back and flushes cache contents.
+ *
+ * cmd: MMUEXT_FLUSH_CACHE_GLOBAL
+ * No additional arguments. Writes back and flushes cache contents
+ * on all CPUs in the system.
*
* cmd: MMUEXT_SET_LDT
* linear_addr: Linear address of LDT base (NB. must be page-aligned).
@@ -268,6 +272,7 @@ DEFINE_XEN_GUEST_HANDLE(xen_pfn_t);
#define MMUEXT_NEW_USER_BASEPTR 15
#define MMUEXT_CLEAR_PAGE 16
#define MMUEXT_COPY_PAGE 17
+#define MMUEXT_FLUSH_CACHE_GLOBAL 18
#ifndef __ASSEMBLY__
struct mmuext_op {
[-- Attachment #2: x86-cache-flush-global.patch --]
[-- Type: text/plain, Size: 2851 bytes --]
Newer Linux' AGP code wants to flush caches on all CPUs under certain
circumstances. Since doing this on all vCPU-s of the domain in
question doesn't yield the intended effect, this needs to be done in
the hypervisor. Add a new MMUEXT operation for this.
While looking at the pre-existing flush implementation, I also noticed
an off-by-one error in the cache flush portion of flush_area_local().
Signed-off-by: Jan Beulich <jbeulich@novell.com>
--- 2010-04-22.orig/xen/arch/x86/flushtlb.c 2008-10-14 19:44:12.000000000 +0200
+++ 2010-04-22/xen/arch/x86/flushtlb.c 2010-04-29 17:47:22.000000000 +0200
@@ -149,7 +149,7 @@ void flush_area_local(const void *va, un
{
unsigned long i, sz = 0;
- if ( order < (BITS_PER_LONG - PAGE_SHIFT - 1) )
+ if ( order < (BITS_PER_LONG - PAGE_SHIFT) )
sz = 1UL << (order + PAGE_SHIFT);
if ( c->x86_clflush_size && c->x86_cache_size && sz &&
--- 2010-04-22.orig/xen/arch/x86/mm.c 2010-04-29 17:35:20.000000000 +0200
+++ 2010-04-22/xen/arch/x86/mm.c 2010-04-29 17:51:08.000000000 +0200
@@ -2889,6 +2889,27 @@ int do_mmuext_op(
}
break;
+ case MMUEXT_FLUSH_CACHE_GLOBAL:
+ if ( unlikely(foreigndom != DOMID_SELF) )
+ okay = 0;
+ else if ( likely(cache_flush_permitted(d)) )
+ {
+ unsigned int cpu;
+ cpumask_t mask = CPU_MASK_NONE;
+
+ for_each_online_cpu(cpu)
+ if ( !cpus_intersects(mask,
+ per_cpu(cpu_sibling_map, cpu)) )
+ cpu_set(cpu, mask);
+ flush_mask(&mask, FLUSH_CACHE);
+ }
+ else
+ {
+ MEM_LOG("Non-physdev domain tried to FLUSH_CACHE_GLOBAL");
+ okay = 0;
+ }
+ break;
+
case MMUEXT_SET_LDT:
{
unsigned long ptr = op.arg1.linear_addr;
--- 2010-04-22.orig/xen/include/public/xen.h 2010-04-29 17:35:20.000000000 +0200
+++ 2010-04-22/xen/include/public/xen.h 2010-04-29 17:09:30.000000000 +0200
@@ -239,6 +239,10 @@ DEFINE_XEN_GUEST_HANDLE(xen_pfn_t);
*
* cmd: MMUEXT_FLUSH_CACHE
* No additional arguments. Writes back and flushes cache contents.
+ *
+ * cmd: MMUEXT_FLUSH_CACHE_GLOBAL
+ * No additional arguments. Writes back and flushes cache contents
+ * on all CPUs in the system.
*
* cmd: MMUEXT_SET_LDT
* linear_addr: Linear address of LDT base (NB. must be page-aligned).
@@ -268,6 +272,7 @@ DEFINE_XEN_GUEST_HANDLE(xen_pfn_t);
#define MMUEXT_NEW_USER_BASEPTR 15
#define MMUEXT_CLEAR_PAGE 16
#define MMUEXT_COPY_PAGE 17
+#define MMUEXT_FLUSH_CACHE_GLOBAL 18
#ifndef __ASSEMBLY__
struct mmuext_op {
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH] x86: add support for domain initiated global cache flush
2010-04-29 15:59 [PATCH] x86: add support for domain initiated global cache flush Jan Beulich
@ 2010-05-03 17:37 ` Konrad Rzeszutek Wilk
0 siblings, 0 replies; 2+ messages in thread
From: Konrad Rzeszutek Wilk @ 2010-05-03 17:37 UTC (permalink / raw)
To: Jan Beulich; +Cc: xen-devel
On Thu, Apr 29, 2010 at 04:59:49PM +0100, Jan Beulich wrote:
> Newer Linux' AGP code wants to flush caches on all CPUs under certain
> circumstances. Since doing this on all vCPU-s of the domain in
> question doesn't yield the intended effect, this needs to be done in
> the hypervisor. Add a new MMUEXT operation for this.
Is there a corresponding patch for this new hypercall API?
>
> While looking at the pre-existing flush implementation, I also noticed
> an off-by-one error in the cache flush portion of flush_area_local().
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-05-03 17:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-29 15:59 [PATCH] x86: add support for domain initiated global cache flush Jan Beulich
2010-05-03 17:37 ` Konrad Rzeszutek Wilk
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).