Home » Mailing list archives » linux-kernel » 2008 » February » 11

Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that support self-snoop

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Andi Kleen
Subject: Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that support self-snoop
Date: Monday, February 11, 2008 - 10:58 am

On Monday 11 February 2008 18:36:06 Siddha, Suresh B wrote:

Ok that would imply that my patch is ok for all current in tree users at least
(none of them ever set WC currently, only UC). It might not be safe for
the out of tree WC users though.

So it should be ok to apply according to standard policy :)


You mean the PAT patch should do something about it? Yes probably, but what?
 

Not assuming owner ship was the main reason why the intent change was originally
submitted. I personally never quite understood the rationale for that,
but the submitter and the maintainers apparently thought it a valuable
thing to have, otherwise the patch would not have been written and included. 

You'll need to ask them if that has changed now.

At least if that requirement was dropped it would be possible to remove
quite a lot of code from pageattr.c

-Andi
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH] [0/8] Misc CPA patchkit, Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [1/8] CPA: Remove my copyright notice, Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [2/8] CPA: Remove inline from static_protections, Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [3/8] CPA: Fix some incorrect comments in the debu ..., Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [4/8] CPA: Ifdef the cpa pool for DEBUG_PAGEALLOC, Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [5/8] CPA: Move pool allocation/free into separate ..., Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [6/8] CPA: Remove BUG_ON for LRU/Compound pages, Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [7/8] CPA: Don't flush caches on CPUs that support ..., Andi Kleen, (Mon Feb 11, 2:50 am)
[PATCH] [8/8] CPA: Add statistics about state of direct ma ..., Andi Kleen, (Mon Feb 11, 2:50 am)
Re: [PATCH] [2/8] CPA: Remove inline from static_protections, Ingo Molnar, (Mon Feb 11, 7:04 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Arjan van de Ven, (Mon Feb 11, 8:04 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Andi Kleen, (Mon Feb 11, 8:12 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Arjan van de Ven, (Mon Feb 11, 8:21 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Andi Kleen, (Mon Feb 11, 8:27 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Siddha, Suresh B, (Mon Feb 11, 10:36 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Andi Kleen, (Mon Feb 11, 10:58 am)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Siddha, Suresh B, (Mon Feb 11, 12:47 pm)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Arjan van de Ven, (Mon Feb 11, 1:36 pm)
Re: [PATCH] [1/8] CPA: Remove my copyright notice, Thomas Gleixner, (Mon Feb 11, 5:42 pm)
Re: [PATCH] [7/8] CPA: Don't flush caches on CPUs that sup ..., Andi Kleen, (Tue Feb 12, 1:45 am)
Re: [PATCH] [6/8] CPA: Remove BUG_ON for LRU/Compound pages, Thomas Gleixner, (Sun Feb 17, 7:07 am)