On Thu, 2008-12-11 at 16:54 -0600, Hollis Blanchard wrote: > > > I don't know that kvm_ppc.c is a very information name for this sort > of > > stuff. Since this is really host specific, not target specific, why > not > > move it out of target-ppc. > > I could combine kvm_ppc.c into target-ppc/kvm.c. However, they're > really > two different things, and I thought it would cause the least confusion > if they were logically separate. Most of it is hooks required by > common > code, and then some of it isn't. (I'm thinking about e.g. IA64 doing a > copy/paste, and then wondering which functions they actually need to > implement.) Regardless, I will still need a kvm_ppc.h, so kvm_ppc.c > seemed like a good place to match. Any further thoughts on this issue? It's the only issue I still have unresolved from the initial reviews. -- Hollis Blanchard IBM Linux Technology Center -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html