All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Gordeev <agordeev@redhat.com>
To: kvm@vger.kernel.org
Cc: "Alexander Gordeev" <agordeev@redhat.com>,
	"Andrew Jones" <drjones@redhat.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Radim Krčmář" <rkrcmar@redhat.com>
Subject: [kvm-unit-tests PATCH v7 00/11] Cleanup low-level arch code
Date: Thu,  5 May 2016 16:26:33 +0200	[thread overview]
Message-ID: <cover.1462457467.git.agordeev@redhat.com> (raw)

Andrew et al,

This is 7th attempt to make mainly x86 arch code consistent
with other architectures.

I withdrew your Reviewed-by from "x86: Cleanup PT_*_MASK flags"
patch 09/11 due to couple of more than just nits changes.

PCD-ed ioremap() could be a follow-up update on top of this series.

Changes since v6:
  - ioremap() PCD comment updated;
  - PT_PSE_MASK renamed to PT_PAGE_SIZE_MASK;
  - GENMASK_ULL used for PT_ADDR_MASK definition;
  - missed x86/taskswitch2.c update added;

Changes since v5:
  - ioremap() PCD comment added, the Reviewed-by preserved;
  - page table flag names changed from PTE_* to PT_*_MASK;

Changes since v4:
  - single instead of multiple pages allocation bug fixed in x86 ioremap();
  - style fixes squashed into code movements for lib/x86/asm/page.h and
    lib/x86/asm/barrier.h;
  - #ifndef __ASSEMBLY__ added to lib/x86/asm/page.h;

Changes since v3:
  - x86 PAGE_* constants and PTE_* flags cleaned up;
  - missing Reviewed-bys restored;
  - missing headers to lib/x86/asm/page.h and lib/x86/asm/barriers.h added
    (I did not remove already given Reviewed-bys from these);

Changes since v2:
  - x86 ioremap() address arithmetics nonsense fixed;
  - x86 virt_to_phys() and phys_to_virt() implementations left intact;
  - separate overrides of virt_to_phys() and phys_to_virt() added;
  - unsigned long size specifier removed from PAGE_SIZE constant;

Changes since v1:
  - arm compilation error fixed;
  - disabling memory re-ordering for generic memory barriers added;


Cc: Andrew Jones <drjones@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>
Cc: Radim Krčmář <rkrcmar@redhat.com>


Alexander Gordeev (11):
  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
  io: Separate overrides of virt_to_phys() and phys_to_virt()
  io: Disallow memory re-ordering for generic memory barriers
  x86: Cleanup PAGE_* constants
  x86: Cleanup PT_*_MASK flags
  io: Make ioremap() prototype conform to Linux one
  x86: io: Factor out ioremap()

 arm/selftest.c          |  1 -
 lib/alloc.h             |  5 -----
 lib/arm/asm/page.h      |  2 --
 lib/asm-generic/io.h    | 13 +++++++++----
 lib/libcflat.h          |  2 ++
 lib/powerpc/asm/setup.h |  1 -
 lib/ppc64/asm/io.h      |  2 ++
 lib/x86/asm/barrier.h   | 13 +++++++++++++
 lib/x86/{ => asm}/io.h  | 18 ++++++++++++++++--
 lib/x86/asm/page.h      | 45 +++++++++++++++++++++++++++++++++++++++++++++
 lib/x86/asm/pci.h       |  2 +-
 lib/x86/io.c            | 18 +++++++++++++++++-
 lib/x86/smp.h           |  4 ----
 lib/x86/vm.c            | 37 ++++++++++---------------------------
 lib/x86/vm.h            | 25 ++-----------------------
 x86/access.c            | 14 ++------------
 x86/asyncpf.c           |  2 +-
 x86/eventinj.c          | 17 ++++++-----------
 x86/hypercall.c         |  2 +-
 x86/hyperv.c            |  1 +
 x86/hyperv.h            |  1 -
 x86/hyperv_stimer.c     |  2 +-
 x86/hyperv_synic.c      |  1 -
 x86/init.c              |  2 +-
 x86/kvmclock.c          |  1 +
 x86/pku.c               |  2 +-
 x86/rmap_chain.c        |  2 +-
 x86/smap.c              |  6 +++---
 x86/svm.c               |  1 -
 x86/taskswitch2.c       |  2 +-
 x86/vmexit.c            | 11 ++---------
 x86/vmx.c               |  1 -
 x86/vmx.h               |  2 +-
 x86/vmx_tests.c         |  1 -
 34 files changed, 140 insertions(+), 119 deletions(-)
 create mode 100644 lib/x86/asm/barrier.h
 rename lib/x86/{ => asm}/io.h (74%)
 create mode 100644 lib/x86/asm/page.h

-- 
1.8.3.1


             reply	other threads:[~2016-05-05 14:26 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-05 14:26 Alexander Gordeev [this message]
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 01/11] Remove unused and unnecessary PHYS32 macro Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 02/11] Move phys_addr_t type definition to lib/libcflat.h Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 03/11] x86: Introduce lib/x86/asm/page.h Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 04/11] x86: Introduce lib/x86/asm/io.h Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 05/11] x86: Introduce lib/x86/asm/barrier.h Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 06/11] io: Separate overrides of virt_to_phys() and phys_to_virt() Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 07/11] io: Disallow memory re-ordering for generic memory barriers Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 08/11] x86: Cleanup PAGE_* constants Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 09/11] x86: Cleanup PT_*_MASK flags Alexander Gordeev
2016-05-06  9:04   ` Andrew Jones
2016-05-06 11:22     ` [kvm-unit-tests PATCH v8 " Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 10/11] io: Make ioremap() prototype conform to Linux one Alexander Gordeev
2016-05-05 14:26 ` [kvm-unit-tests PATCH v7 11/11] x86: io: Factor out ioremap() Alexander Gordeev
2016-05-06  9:06   ` Andrew Jones
2016-05-06  9:08 ` [kvm-unit-tests PATCH v7 00/11] Cleanup low-level arch code Andrew Jones
2016-05-06 11:29   ` Alexander Gordeev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1462457467.git.agordeev@redhat.com \
    --to=agordeev@redhat.com \
    --cc=drjones@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=rkrcmar@redhat.com \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.