From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:35797)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1StGSZ-0008Ui-L3
	for qemu-devel@nongnu.org; Mon, 23 Jul 2012 07:06:01 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1StGST-00052l-Om
	for qemu-devel@nongnu.org; Mon, 23 Jul 2012 07:05:55 -0400
Received: from mail-bk0-f45.google.com ([209.85.214.45]:48959)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1StGST-00051D-Hz
	for qemu-devel@nongnu.org; Mon, 23 Jul 2012 07:05:49 -0400
Received: by bkcji1 with SMTP id ji1so4285960bkc.4
	for <qemu-devel@nongnu.org>; Mon, 23 Jul 2012 04:05:48 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <500D2F16.5000209@samsung.com>
References: <1342112068-23345-1-git-send-email-m.kozlov@samsung.com>
	<1342112068-23345-5-git-send-email-m.kozlov@samsung.com>
	<CAFEAcA9+3+Dhn24yctLAqhrKZfz6Qf2SaHa3XmH7bnQT1qfXUg@mail.gmail.com>
	<500D2F16.5000209@samsung.com>
Date: Mon, 23 Jul 2012 12:05:48 +0100
Message-ID: <CAFEAcA-q1vr9p6AwYD26UerUp-53NbdGHdVV4FTbdr63zrCRsQ@mail.gmail.com>
From: Peter Maydell <peter.maydell@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH v2 4/4] ARM: exynos4210_pmu: Add software
	reset 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: Maksim Kozlov <m.kozlov@samsung.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>, Evgeny Voevodin <e.voevodin@samsung.com>, qemu-devel@nongnu.org, kyungmin.park@samsung.com, Paolo Bonzini <pbonzini@redhat.com>, =?UTF-8?Q?Andreas_F=C3=A4rber?= <afaerber@suse.de>

On 23 July 2012 12:01, Maksim Kozlov <m.kozlov@samsung.com> wrote:
> 20.07.2012 18:32, Peter Maydell =D0=BF=D0=B8=D1=88=D0=B5=D1=82:
>> This patch seems to be trying to make a distinction that QEMU doesn't
>> support, ie between system wide "software reset" and system wide
>> "hard reset". I'm not convinced about the wisdom of trying to paper
>> over this lack with a single-device workaround.
>
> Peter, if we add callback (*swreset)() into DeviceClass, whether it will =
be
> an acceptable solution? And devices which have different behavior during
> swreset can register different function for that. What do you think about
> this?

I think reset is a key part of device lifecycle and adding support
for 'warm reset' or whatever needs general discussion so we can get
the semantics right... cc'ing a couple of people who might have opinions.

-- PMM