qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eric Auger <eric.auger@linaro.org>
To: eric.auger@st.com, christoffer.dall@linaro.org,
	qemu-devel@nongnu.org, agraf@suse.de, pbonzini@redhat.com,
	mst@redhat.com, b.reynal@virtualopensystems.com
Cc: peter.maydell@linaro.org, alex.williamson@redhat.com,
	patches@linaro.org, kvmarm@lists.cs.columbia.edu,
	eric.auger@linaro.org
Subject: [Qemu-devel] [PATCH v4 0/3] explicit VGIC initialization in finalize function
Date: Wed,  4 Mar 2015 14:16:14 +0000	[thread overview]
Message-ID: <1425478577-32556-1-git-send-email-eric.auger@linaro.org> (raw)

With current implementation the vgic is initialized implicitly,
on-demand. This patch forces the init in the vgic finalize function.
At that point all the dimensioning parameters are known.

A new group/attribute in VGIC KVM device is used for that:
KVM_DEV_ARM_VGIC_GRP_CTRL/KVM_DEV_ARM_VGIC_CTRL_INIT.

Among other things, this earlier VGIC initialization allows to
have a simpler handling of VFIO. VFIO platform signaling can be
setup on reset handler, making sure the VGIC is ready, at that time,
to accept injected virtual IRQs.

Tested on Calxeda Midway with VFIO platform use case

Can be found on
http://git.linaro.org/people/eric.auger/qemu.git,
branch early_vgic_init_v4

v3 -> v4:
- now integrates all the header updates of update-linux-headers.sh applied
  on top of 4.0-rc2 kernel (and not a partial export as in v3).
- VIRTIO_TRANSPORT_F_END value needed to be changed to 33 in
  include/hw/virtio/virtio.h to fix a duplicate definition compilation error
  (also defined in virtio_config.h)
- integrates a new patch file that add export of virtio_types.h, now
  included by virtio_ring.h.

v2 -> v3:
- changed to a patch since the kernel dependency was pulled

v1 -> v2:
- The init is not mandated to be done in a machine init done notifier
  anymore since only the number of vcpus and number of IRQs must be known
  at init time.

Eric Auger (3):
  update-linux-headers.sh: export virtio_types.h
  linux-headers: update to 4.0-rc2
  hw/intc/arm_gic: Initialize the vgic in the realize function

 hw/intc/arm_gic_kvm.c               |  6 +++++
 include/hw/virtio/virtio.h          |  4 +--
 linux-headers/asm-arm/kvm.h         |  2 ++
 linux-headers/asm-arm64/kvm.h       |  9 +++++++
 linux-headers/asm-s390/kvm.h        | 37 ++++++++++++++++++++++++++
 linux-headers/asm-x86/hyperv.h      | 11 ++++++++
 linux-headers/linux/kvm.h           | 20 +++++++-------
 linux-headers/linux/vfio.h          |  1 +
 linux-headers/linux/virtio_config.h | 11 ++++++--
 linux-headers/linux/virtio_ring.h   | 52 +++++++++++++++++++++----------------
 linux-headers/linux/virtio_types.h  | 46 ++++++++++++++++++++++++++++++++
 scripts/update-linux-headers.sh     |  2 +-
 12 files changed, 163 insertions(+), 38 deletions(-)
 create mode 100644 linux-headers/linux/virtio_types.h

-- 
1.8.3.2

             reply	other threads:[~2015-03-04 14:16 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-04 14:16 Eric Auger [this message]
2015-03-04 14:16 ` [Qemu-devel] [PATCH v4 1/3] update-linux-headers.sh: export virtio_types.h Eric Auger
2015-03-04 15:37   ` Michael S. Tsirkin
2015-03-04 15:40     ` Eric Auger
2015-03-09  9:26     ` Christian Borntraeger
2015-03-04 14:16 ` [Qemu-devel] [PATCH v4 2/3] linux-headers: update to 4.0-rc2 Eric Auger
2015-03-04 14:16 ` [Qemu-devel] [PATCH v4 3/3] hw/intc/arm_gic: Initialize the vgic in the realize function Eric Auger
2015-03-11 14:56   ` Peter Maydell

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=1425478577-32556-1-git-send-email-eric.auger@linaro.org \
    --to=eric.auger@linaro.org \
    --cc=agraf@suse.de \
    --cc=alex.williamson@redhat.com \
    --cc=b.reynal@virtualopensystems.com \
    --cc=christoffer.dall@linaro.org \
    --cc=eric.auger@st.com \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=mst@redhat.com \
    --cc=patches@linaro.org \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).