From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43100) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ag06d-00059m-Bq for qemu-devel@nongnu.org; Tue, 15 Mar 2016 21:18:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ag06b-000729-VP for qemu-devel@nongnu.org; Tue, 15 Mar 2016 21:18:35 -0400 Date: Wed, 16 Mar 2016 12:19:14 +1100 From: David Gibson Message-ID: <20160316011914.GA9032@voom> References: <1457974600-13828-1-git-send-email-clg@fr.ibm.com> <20160315003942.GE15272@voom.fritz.box> <56E7C3B3.3030809@fr.ibm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="LQFJYnjHKDAbJRTQ" Content-Disposition: inline In-Reply-To: <56E7C3B3.3030809@fr.ibm.com> Subject: Re: [Qemu-devel] [PATCH 00/17] ppc: preparing pnv landing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?C=E9dric?= Le Goater Cc: Thomas Huth , qemu-ppc@nongnu.org, qemu-devel@nongnu.org --LQFJYnjHKDAbJRTQ Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Mar 15, 2016 at 09:11:31AM +0100, C=E9dric Le Goater wrote: > On 03/15/2016 01:39 AM, David Gibson wrote: > > On Mon, Mar 14, 2016 at 05:56:23PM +0100, C=E9dric Le Goater wrote: > >> Hello, > >> > >> This is a first mini-serie of patches adding support for new ppc SPRs. > >> They were taken from Ben's larger patchset adding the ppc powernv > >> platform and they should already be useful for the pseries guest > >> migration. > >> > >> Initial patches come from : > >> > >> https://github.com/ozbenh/qemu/commits/powernv > >> > >> The changes are mostly due to the rebase on Dave's 2.6 branch: > >> > >> https://github.com/dgibson/qemu/commits/ppc-for-2.6 > >> > >> A couple more are bisect and checkpatch fixes and finally some patches > >> were merge to reduce the noise. > >> > >> =20 > >> > >> The patchset is also available here:=20 > >> > >> https://github.com/legoater/qemu/commits/for-2.6 > >> > >> It was quickly tested with a pseries guest using KVM and TCG. > >=20 > > Hmm.. do these all fix bugs with migration, or only some of them? >=20 > Probably only some.=20 >=20 > Initially, Thomas gave a shorter list which I expanded to a larger one=20 > because of dependencies between patches and I didn't want to change too > much what Ben had sent. You had also reviewed a few. >=20 > > The relevance is that things to fix migration should go into 2.6, but > > preparation work for powernv that doesn't fix bug shouldn't really be > > going in now, after the soft freeze and will need to wait for 2.7. >=20 > OK. I will rework and keep the rest for 2.7.=20 So, I'm ok with including (low risk) patches that aren't directly relevant to 2.6 if they're prereqs for patches that are relevant to 2.6. After all, reworking the patches isn't risk free either. Please mention why these patches are being included in the commit messages though. > Thomas, thanks for the review. I have identified a few things I need=20 > to work on but may be, the patchset is still too large for 2.6 ? It's not really a question of being too large, it's that I'm nervous about applying patches which touch the core translation code (e.g. fixes to HV mode tests) during soft freeze if they're not addressing a bug that's relevant to 2.6. --=20 David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson --LQFJYnjHKDAbJRTQ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJW6LSSAAoJEGw4ysog2bOSp4EQAMoLR+0b4Gh+FmGElJ3uDCBj ZppBvvO9oBxADxCRQpsZEwKMwcMd928npsMaqxUrODGFWpm7MUptiKxEflD5jmhI JbWuugTOxsPFcRBkzjtoLAKBCeskjaRFIP9vmbSVSfgILMOKmZADs4rxYL2rAu/T oEgE5vzRdfk936VCwJ9IMMTLWyyQbdnXRSlpvC0LDuFiKcugUmjQaotjsKBq5qPZ e2Ar/uPShc/ExBYaw9f1iyjjQ9P2ZNv5KEef+3m3th3GzgyN6ktIjOTdzWx06ukt mr/U3xoiBFU5vXraWU54nRY4lyKVBHDoelJbPzjH+lrCib77WNDFCE0+ZKe8xiV1 lIMIOnawSTSNRkAlhogljpJ3dwtZL/9b7WkzeKkj5MROlLwXcGkM2HxQd7AIW2Xd qpi66H67Ar6EedymLN96Xr2xImq7Cu2ZnwnJXWkkJEr40eJQV+DNDs/udE7g0TSZ ZFaTap0dlJE32+4unVBnA70H3ScAZD/dMrgzgZOd0fS2svpGY6bBk0vDcL9NH63d xlFEc7IEcpPa5XJ2JUf+RkiGDweIDaA7lmXX6djnOfrv3QAP2qytvdu6lrF3p+xp 1dQWOT3fTF5JgwoxvSdOfpZg8QPD8MZ0D/IZwA8HHMN1OIywTqpXCCidDhTV8Ha0 LN3taM0Ov/7gw1P7y7Ke =po9t -----END PGP SIGNATURE----- --LQFJYnjHKDAbJRTQ--