From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59892) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ws9T0-0006fq-34 for qemu-devel@nongnu.org; Wed, 04 Jun 2014 07:34:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ws9Sr-0003Jo-Kc for qemu-devel@nongnu.org; Wed, 04 Jun 2014 07:34:50 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:63368) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ws9Sr-0003JY-El for qemu-devel@nongnu.org; Wed, 04 Jun 2014 07:34:41 -0400 Received: by mail-pb0-f46.google.com with SMTP id rq2so6903655pbb.33 for ; Wed, 04 Jun 2014 04:34:40 -0700 (PDT) Message-ID: <538F044A.70308@ozlabs.ru> Date: Wed, 04 Jun 2014 21:34:34 +1000 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1401869330-32449-1-git-send-email-aik@ozlabs.ru> <538EE73D.4080408@ozlabs.ru> <538EE960.2040108@redhat.com> <538EEBFD.8090606@suse.de> In-Reply-To: <538EEBFD.8090606@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/4] cpus: Add generic NMI support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf , Peter Maydell , Paolo Bonzini Cc: Alex Bligh , Markus Armbruster , QEMU Developers , Luiz Capitulino , "qemu-ppc@nongnu.org" , Stefan Hajnoczi , Cornelia Huck , =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= , Richard Henderson On 06/04/2014 07:50 PM, Alexander Graf wrote: > > On 04.06.14 11:47, Peter Maydell wrote: >> On 4 June 2014 10:39, Paolo Bonzini wrote: >>> It's a kind of "emergency button" on real machines. On PCs it sends an NMI >>> and this results in some kind of crash dump if the OS is configured >>> appropriately. The command may be ill-named for historical reasons, but >>> the >>> effect is not x86-specific. >> OK, so our callback function name should be sensibly named >> to match what the effect is supposed to be, and we should have >> a sensibly named command, and we should make "nmi" be >> a historical-legacy-backwards-compatibility command alias >> (possibly only exposed for x86). > > I honestly don't have a better name :). Worse name? Any name? If we come up with some name, "nmi" will still do "restart" on s390, won't it (backward compatibility blablabla)? So there is no point adding it for anyone but PPC64 and therefore it can be something like "machinecheck" and we'll have yet another #ifdef in cpus.c. Which would be very unusual and very awesome as I am normally not allowed to do ppc-hacks in generic code :-D -- Alexey