From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60340)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <pbonzini@redhat.com>) id 1XLsOR-0001pO-FR
	for qemu-devel@nongnu.org; Mon, 25 Aug 2014 07:25:05 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <pbonzini@redhat.com>) id 1XLsOK-0005YC-E9
	for qemu-devel@nongnu.org; Mon, 25 Aug 2014 07:24:59 -0400
Message-ID: <53FB1CFA.4000309@redhat.com>
Date: Mon, 25 Aug 2014 13:24:42 +0200
From: Paolo Bonzini <pbonzini@redhat.com>
MIME-Version: 1.0
References: <1408536997-2075-1-git-send-email-aik@ozlabs.ru>
In-Reply-To: <1408536997-2075-1-git-send-email-aik@ozlabs.ru>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH v8 0/5] cpus: Add generic "nmi" monitor
	command support
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexey Kardashevskiy <aik@ozlabs.ru>, qemu-devel@nongnu.org
Cc: Cornelia Huck <cornelia.huck@de.ibm.com>, qemu-ppc@nongnu.org, Alexander Graf <agraf@suse.de>

Il 20/08/2014 14:16, Alexey Kardashevskiy ha scritto:
> This adds an "nmi" monitor command handler per CPUs.
> x86, s390 and ppc CPUS are supported.
> 
> Please comment. Thanks.
> 
> Changes:
> v8:
> * adjusted commit log for s390 patch
> * fixed missing "static" in 1/4
> 
> v7:
> * fixed typenames, function names, copyrights
> * s390x QOM'ed for later addition of the NMI interface
> * Since none of x86 machines is QOM'ed, postpone migration to new interface for x86
> 
> v6:
> * back 5 steps and make it an interface again
> 
> v5:
> * added Error** to the callback
> * fixed some comments
> 
> v4:
> * now it is not nmi() but nmi_monitor_handler() to avoid confusion
> 
> v3:
> * patches reorganized
> * comments from v2 addressed, more details are in individual commit logs
> 
> v2:
> * moved from machine interface to CPUClass callback
> * s390 and x86 moved to target-s390/target-i386
> * x86 handler delivers to the current CPU only now
> 
> 
> 
> 
> Alexey Kardashevskiy (5):
>   cpus: Define callback for QEMU "nmi" command
>   s390x: Convert QEMUMachine to MachineClass
>   s390x: Migrate to new NMI interface
>   spapr: Add support for new NMI interface
>   pc_piix: Migrate to new NMI interface
> 
>  cpus.c                     | 31 ++---------------
>  hmp-commands.hx            |  6 ++--
>  hw/core/Makefile.objs      |  1 +
>  hw/core/nmi.c              | 84 ++++++++++++++++++++++++++++++++++++++++++++++
>  hw/i386/pc_piix.c          | 42 +++++++++++++++++++++++
>  hw/ppc/spapr.c             | 21 ++++++++++++
>  hw/s390x/s390-virtio-ccw.c | 49 ++++++++++++++++++---------
>  hw/s390x/s390-virtio.c     | 59 ++++++++++++++++++++++----------
>  hw/s390x/s390-virtio.h     |  3 ++
>  include/hw/nmi.h           | 49 +++++++++++++++++++++++++++
>  qapi-schema.json           |  4 +--
>  qmp-commands.hx            |  3 +-
>  target-ppc/cpu-qom.h       |  1 +
>  target-ppc/excp_helper.c   |  8 +++++
>  14 files changed, 291 insertions(+), 70 deletions(-)
>  create mode 100644 hw/core/nmi.c
>  create mode 100644 include/hw/nmi.h
> 

Applying patches 1-4 to uq/master.  PC will wait for the QOM machine
conversion.

Paolo