From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ian Pratt <Ian.Pratt@eu.citrix.com>
Subject: RE: [PATCH 2/3] remove saving/restoring method in Xend.
Date: Wed, 22 Apr 2009 11:38:13 +0100
Message-ID: <4FA716B1526C7C4DB0375C6DADBC4EA34172EC1DA5@LONPMAILBOX01.citrite.net>
References: <20090422105816.32A4.27C06F64@necst.nec.co.jp>
	<20090422111600.32AA.27C06F64@necst.nec.co.jp>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Return-path: <xen-devel-bounces@lists.xensource.com>
In-Reply-To: <20090422111600.32AA.27C06F64@necst.nec.co.jp>
Content-Language: en-US
List-Unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xensource.com>
List-Help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-Subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
Sender: xen-devel-bounces@lists.xensource.com
Errors-To: xen-devel-bounces@lists.xensource.com
To: Yuji Shimada <shimada-yxb@necst.nec.co.jp>, Keir Fraser <Keir.Fraser@eu.citrix.com>, Kouya Shimura <kouya@jp.fujitsu.com>, "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Cc: Ian Pratt <Ian.Pratt@eu.citrix.com>, Ross Philipson <Ross.Philipson@citrix.com>
List-Id: xen-devel@lists.xenproject.org

> This patch removes Xend method which saves/restores PCI configuration
> space.
> And this patch modifies the timing of saving/restoring configuration
> space like below.
>=20
>   When pciback is bound to devices.
>    - Pciback saves configuration space.
>=20
>   When pciback is unbound to devices.
>    - Pciback restores configuration space.
>=20
>   When guest OS boots or a device is hotadded.
>    - Pciback restores configuration space.
>    - Pciback changes state of backend device to
> Initialised/Reconfigured.
>    - Xend waits for the transition to Initialised/Reconfigured.
>=20
>   When guest OS shutdowns or a device is hotremoved.
>    - Pciback restores configuration space.
>    - Xend resets devices.
>      * If D-state of the device is not D0, the state is changed to D0
>        before resetting the device.
>    - Xend deassigns devices.

Does it not make sense to have pciback do the reset too? I think I've seen =
draft patches from Ross Philipson to move the reset into pciback -- they ma=
y even be checked in to the xenbits.xen.org/xenclient tree.

Thanks,
Ian