Re: [RFC PATCH 0/2] ARM: KVM: Moving GIC/timer out of arch/arm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/05/13 16:31, Anup Patel wrote:
> On Fri, May 3, 2013 at 7:32 PM, Marc Zyngier <marc.zyngier@xxxxxxx> wrote:
>> As KVM/arm64 is looming on the horizon, it makes sense to move some
>> of the common code to a single location in order to reduce duplication.
>>
>> The code could live anywhere. Actually, most of KVM is already built
>> with a bunch of ugly ../../.. hacks in the various Makefiles, so we're
>> not exactly talking about style here. But maybe it is time to start
>> moving into a less ugly direction.
>>
>> The include files must be in a "public" location, as they are accessed
>> from non-KVM files (arch/arm/kernel/asm-offsets.c).
>>
>> For this purpose, introduce two new locations:
>> - virt/kvm/arm/ : x86 and ia64 already share the ioapic code in
>>   virt/kvm, so this could be seen as a (very ugly) precedent.
>> - include/kvm/  : there is already an include/xen, and while the
>>   intent is slightly different, this seems as good a location as
>>   any
>>
>> Once the code has been moved, it becomes easy to build it in a
>> less hackish way, which makes the code easily reusable by KVM/arm64.
>>
>> Marc Zyngier (2):
>>   ARM: KVM: move GIC/timer code to a common location
>>   ARM: KVM: standalone Makefile for vgic and timers
>>
>>  Makefile                                               | 2 +-
>>  arch/arm/include/asm/kvm_host.h                        | 4 ++--
>>  arch/arm/kvm/Makefile                                  | 5 ++---
>>  {arch/arm/include/asm => include/kvm}/kvm_arch_timer.h | 0
>>  {arch/arm/include/asm => include/kvm}/kvm_vgic.h       | 0
>>  virt/Makefile                                          | 1 +
>>  virt/kvm/Makefile                                      | 1 +
>>  virt/kvm/arm/Makefile                                  | 2 ++
>>  {arch/arm/kvm => virt/kvm/arm}/arch_timer.c            | 4 ++--
>>  {arch/arm/kvm => virt/kvm/arm}/vgic.c                  | 0
>>  10 files changed, 11 insertions(+), 8 deletions(-)
>>  rename {arch/arm/include/asm => include/kvm}/kvm_arch_timer.h (100%)
>>  rename {arch/arm/include/asm => include/kvm}/kvm_vgic.h (100%)
>>  create mode 100644 virt/Makefile
>>  create mode 100644 virt/kvm/Makefile
>>  create mode 100644 virt/kvm/arm/Makefile
>>  rename {arch/arm/kvm => virt/kvm/arm}/arch_timer.c (99%)
>>  rename {arch/arm/kvm => virt/kvm/arm}/vgic.c (100%)
>>
>> --
>> 1.8.2.1
>>
>>
>>
>> _______________________________________________
>> kvmarm mailing list
>> kvmarm@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm
> 
> The source files arch/arm/kvm/arm.c and arch/arm/kvm/mmu.c are also
> shared between KVM ARM and KVM ARM64.
> 
> Can we move these files in virt/arm ?

I suggest we start by finding out if there is an agreement on the
location, method and overall usefulness of this particular patch.

Moving core ARM code around is quite different from sharing what is
basically device emulation stuff.

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...

--
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




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux