Re: [PATCH 0/9] x86: Cleanup low-level arch code

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

 



On Wed, Apr 20, 2016 at 03:18:46PM +0200, Alexander Gordeev wrote:
> Hi all,
> 
> This is an attempt to make mainly x86 IO code more consistent
> with other architectures. The series paves the way for the
> PCI update I sent earlier, but it also may be of interest
> by itself.
> 
> Thanks!
> 
> Cc: Andrew Jones <drjones@xxxxxxxxxx>
> Cc: Thomas Huth <thuth@xxxxxxxxxx>
> Cc: Radim Krčmář <rkrcmar@xxxxxxxxxx>
> 
> Alexander Gordeev (9):
>   Remove unused and unnecessary PHYS32 macro
>   Move phys_addr_t type definition to lib/libcflat.h
>   x86: Introduce lib/x86/asm/page.h
>   x86: Introduce lib/x86/asm/io.h
>   x86: Introduce lib/x86/asm/barrier.h
>   x86: Optimize virt_to_phys() and phys_to_virt()
>   io: Introduce generic IO accessors
>   io: Make ioremap() prototype conform to Linux one
>   io/x86: Factor out ioremap()

All the patches look pretty good to me (minus a few comments I had).
Let's see what x86 people say.

drew


> 
>  arm/selftest.c          |  1 -
>  lib/alloc.h             |  7 -------
>  lib/arm/asm/page.h      |  2 --
>  lib/arm/asm/setup.h     |  1 -
>  lib/arm/setup.c         |  1 -
>  lib/asm-generic/io.h    | 48 ++++++++++++++++++++++++++++++++++++++++--
>  lib/libcflat.h          |  3 +++
>  lib/powerpc/asm/setup.h |  1 -
>  lib/x86/asm/barrier.h   |  8 +++++++
>  lib/x86/asm/io.h        | 56 +++++++++++++++++++++++++++++++++++++++++++++++++
>  lib/x86/asm/page.h      | 30 ++++++++++++++++++++++++++
>  lib/x86/asm/pci.h       |  2 +-
>  lib/x86/io.c            | 16 +++++++++++++-
>  lib/x86/io.h            | 40 -----------------------------------
>  lib/x86/smp.h           |  4 ----
>  lib/x86/vm.c            | 17 ---------------
>  lib/x86/vm.h            | 24 +--------------------
>  x86/eventinj.c          |  7 +------
>  x86/hyperv.c            |  1 +
>  x86/hyperv.h            |  1 -
>  x86/hyperv_stimer.c     |  1 -
>  x86/hyperv_synic.c      |  1 -
>  x86/init.c              |  2 +-
>  x86/kvmclock.c          |  1 +
>  x86/svm.c               |  1 -
>  x86/vmexit.c            | 11 ++--------
>  x86/vmx.c               |  1 -
>  x86/vmx_tests.c         |  1 -
>  28 files changed, 166 insertions(+), 123 deletions(-)
>  create mode 100644 lib/x86/asm/barrier.h
>  create mode 100644 lib/x86/asm/io.h
>  create mode 100644 lib/x86/asm/page.h
>  delete mode 100644 lib/x86/io.h
> 
> -- 
> 1.8.3.1
> 
> --
> 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
--
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