On Tue, Feb 28, 2012 at 07:38:34PM +0200, Avi Kivity wrote: > On 02/28/2012 07:36 PM, David Ahern wrote: > > I was to suggest the reverse: since this patch addesses an AMD bug, > > why not push those functions into perf_event_amd.c and make them > > dependent on CONFIG_CPU_SUP_AMD as well. > > It depends on which direction you expect the code to grow. These hooks > seem reasonable, so I think they should be generic. Okay, I am fine with both directions. But since this patch is a fix in the first place I make it more AMD specific for now. If we need this as a generic feature it can easily be changed back. Joerg -- AMD Operating System Research Center Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach General Managers: Alberto Bozzo Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html