From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:39610)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <hutao@cn.fujitsu.com>) id 1UcW8R-0007W8-2d
	for qemu-devel@nongnu.org; Wed, 15 May 2013 03:28:29 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <hutao@cn.fujitsu.com>) id 1UcW8P-00050M-3K
	for qemu-devel@nongnu.org; Wed, 15 May 2013 03:28:26 -0400
Received: from [222.73.24.84] (port=51568 helo=song.cn.fujitsu.com)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <hutao@cn.fujitsu.com>) id 1UcW8N-0004rI-Ny
	for qemu-devel@nongnu.org; Wed, 15 May 2013 03:28:25 -0400
Date: Wed, 15 May 2013 15:27:39 +0800
From: Hu Tao <hutao@cn.fujitsu.com>
Message-ID: <20130515072739.GA30450@localhost.localdomain>
References: <cover.1365143012.git.hutao@cn.fujitsu.com>
	<1365146271-6220-1-git-send-email-hutao@cn.fujitsu.com>
	<51933792.7030700@redhat.com>
MIME-Version: 1.0
In-Reply-To: <51933792.7030700@redhat.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Subject: Re: [Qemu-devel] [SeaBIOS] [PATCH v17] Add pvpanic device driver
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: Laszlo Ersek <lersek@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>, Gleb Natapov <gleb@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Jan Kiszka <jan.kiszka@siemens.com>, seabios@seabios.org, qemu-devel <qemu-devel@nongnu.org>, Markus Armbruster <armbru@redhat.com>, Blue Swirl <blauwirbel@gmail.com>, Orit Wasserman <owasserm@redhat.com>, Juan Quintela <quintela@redhat.com>, Alexander Graf <agraf@suse.de>, Christian Borntraeger <borntraeger@de.ibm.com>, Andrew Jones <drjones@redhat.com>, Alex Williamson <alex.williamson@redhat.com>, Kevin O'Connor <kevin@koconnor.net>, Sasha Levin <levinsasha928@gmail.com>, Stefan Hajnoczi <stefanha@redhat.com>, Luiz Capitulino <lcapitulino@redhat.com>, KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>, Anthony Liguori <aliguori@us.ibm.com>, Marcelo Tosatti <mtosatti@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>

On Wed, May 15, 2013 at 09:21:54AM +0200, Laszlo Ersek wrote:
> On 04/05/13 09:17, Hu Tao wrote:
> 
> > +            Method(RDPT, 0, NotSerialized) {
> > +                Store(PEPT, Local0)
> > +                Return (Local0)
> > +            }
> > +
> > +            Method(WRPT, 1, NotSerialized) {
> > +                Store(Arg0, PEPT)
> > +            }
> 
> Please excuse my asking, I haven't been following this -- who's supposed
> to call these methods? The latest guest kernel patch I managed to find
> ([PATCH v7] kvm: notify host when the guest is panicked -- probably
> obsolete by now) appears to do a direct outl().

An old version calls WRPT but is buggy, so you see the outl version.
This patch is in seabios before the outl version. If you think these
methods are needless, feel free to send a patch.