From: Dario Faggioli Subject: Re: [PATCH 22/27] ts-bench-hostcmp-host-prep: new script Date: Thu, 11 Dec 2014 14:23:38 +0100 Message-ID: <1418304218.31647.104.camel@Abyss.station> References: <20141210180651.26400.13356.stgit@Abyss.station> <20141210181152.26400.21301.stgit@Abyss.station> <20141211123258.GJ21659@zion.uk.xensource.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5060965656687854609==" Return-path: In-Reply-To: <20141211123258.GJ21659@zion.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Wei Liu Cc: Ian Jackson , Ian Campbell , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org --===============5060965656687854609== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-cNXjO09P35v7y3TW/jBT" --=-cNXjO09P35v7y3TW/jBT Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, 2014-12-11 at 12:32 +0000, Wei Liu wrote: > On Wed, Dec 10, 2014 at 07:11:52PM +0100, Dario Faggioli wrote: > > + logm("Will run the benchmark on host with: $bcpus vcpus and $bmem MB= RAM"); > > + > > + my $kernver=3D get_runvar('kernel_ver',$r{'kernbuildjob'}); > > + my $kopt=3D "maxcpus=3D$bcpus mem=3D$bmem" . "M"; > > + > > + if ($ho->{Flags}{'need-uboot-bootscr'}) { > > + $bl=3D setupboot_uboot($ho,$kernver,undef,$kopt); > > + } elsif ($ho->{Suite} =3D~ m/lenny/) { > > + $bl=3D setupboot_grub1($ho,$kernver,undef,$kopt); > > + } else { > > + $bl=3D setupboot_grub2($ho,$kernver,undef,$kopt); > > + } > > + > > + $bootkern=3D $bl->{PreFinalUpdate}(); > > + $bl->{UpdateConfig}($ho); > > + > > + $bootkern=3D $bl->{GetBootKern}(); > > + logm("$ho->{Name} will reboot on kernel $bootkern with '$kopt' as op= tions"); >=20 > Can you refactor debian_setup_boot to achieve your goal? >=20 This happens in patch 23, at least to some extent. :-) BTW, that patch came out without a changelog, so sorry again for that. I can see if I can merge the parts of these patches that play with debian_setup_boot into something that makes sense for both this and the other patch's purposes. Thanks and Regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-cNXjO09P35v7y3TW/jBT Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEABECAAYFAlSJmtoACgkQk4XaBE3IOsQ5EgCgqCqwsdSSu0U1B5wX6T+jZjkf 0FoAoK9fYuS7kKqZvhfXzPnVO+rG/uhU =+AiI -----END PGP SIGNATURE----- --=-cNXjO09P35v7y3TW/jBT-- --===============5060965656687854609== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============5060965656687854609==--