All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: "Pang, LongtaoX" <longtaox.pang@intel.com>
Cc: "wei.liu2@citrix.com" <wei.liu2@citrix.com>,
	"Hu, Robert" <robert.hu@intel.com>,
	"Ian.Jackson@eu.citrix.com" <Ian.Jackson@eu.citrix.com>,
	Ian Campbell <ian.campbell@citrix.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs
Date: Mon, 23 Mar 2015 17:19:30 +0000	[thread overview]
Message-ID: <20150323171930.GA19412@zion.uk.xensource.com> (raw)
In-Reply-To: <86C3224E41A7434B904EC364302132D80E4790B3@SHSMSX101.ccr.corp.intel.com>

On Mon, Mar 23, 2015 at 04:20:08PM +0000, Pang, LongtaoX wrote:
> > 
> > > > > The editconfig_cd thing -- yet another thing which Ian questioned
> > > > > and which it was agreed you would change but you haven't.
> > > > >
> > > > For this question, I have sent a mail about it.(2015-03-04) After
> > > > finishing L1 guest VM installation, we need to change L1 guest boot
> > > > sequence from ISO image to hard disk, we need modify the "boot=cd" ,
> > >
> > > Do you? As Ian asked before, why is guest_editconfig_nocd  not
> > > sufficient? It removes the CD from the virtual drive, meaning that
> > > "boot=dc" will fail to boot from d and fallthru to c.
> > >
> > > >  also need to enable 'nestedhvm' feature in hvm configure file,
> > >
> > > This certainly doesn't belong in a function called
> > > guest_editconfig_cd, since it has nothing to do with cds at all.
> > >
> > > Anyway, it's not clear why you need to edit this into the nestedhvm
> > > configuration, instead of adding it when the configuration is created
> > > via more_prepareguest_hvm. What harm is there in enabling this during
> > > guest install?
> > >
> > I will try it.
> > 
> Re-use 'guest_ediconfig_nocd', after finishing L1 installation, it could boot into L1 OS, but failed to install packages( such as lvm2, rsync, bridge-utils ....) via Debian repo in L1, as below msg:
> root@nested:~# apt-get install lvm2 --no-install-recommends -y
> Reading package lists... Done
> Building dependency tree
> Reading state information... Done
> The following extra packages will be installed:
>   libdevmapper-event1.02.1 libreadline5
> The following NEW packages will be installed:
>   libdevmapper-event1.02.1 libreadline5 lvm2
> 0 upgraded, 3 newly installed, 0 to remove and 0 not upgraded.
> Need to get 0 B/767 kB of archives.
> After this operation, 1,521 kB of additional disk space will be used.
> Media change: please insert the disc labeled
>  'Debian GNU/Linux 7.6.0 _Wheezy_ - Official amd64 DVD Binary-1 20140712-14:11'
> in the drive '/media/cdrom/' and press enter
> 
> I checked the 'sources.list' file in L1, because L1 Debian OS assume that ISO Image as repo, but the ISO image does not exist actually.
> Since '$gho->{Rimage}' is replaced as ' $emptyiso' in ' guest_ediconfig_nocd', maybe it is not sufficient for nested test.
> If keep to re-use ' guest_ediconfig_nocd' , is there any approach to setting guest's repo in osstest system? 
> 
> 

Here are my 2 cents. Might be an alternative solution to your problem.

The reason that we call guest_editconfig_nocd is to prevent the guest
from booting from CDROM again.

You can probably invent a function like guest_editconfig_boot_from_hd
(name subject to improvement) to switch "boot=dc" (which is the default
value in more_prepareguest_hvm) to "boot=cd".

That way you can boot from hard drive while at the same time keep the
CDROM filled.

Wei.

  parent reply	other threads:[~2015-03-23 17:19 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-20 12:09 [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs Pang, LongtaoX
2015-03-20 12:20 ` Ian Campbell
2015-03-20 12:59   ` Pang, LongtaoX
2015-03-20 13:37     ` Ian Campbell
2015-03-23  6:31   ` Hu, Robert
2015-03-23  9:49     ` Ian Campbell
2015-03-24  3:25       ` Hu, Robert
2015-03-24  9:43         ` Ian Campbell
2015-03-24  3:34       ` Hu, Robert
2015-03-24  9:45         ` Ian Campbell
2015-03-24 11:41           ` Ian Jackson
2015-03-23 16:20   ` Pang, LongtaoX
2015-03-23 16:45     ` Ian Campbell
2015-03-23 17:29       ` Wei Liu
2015-03-23 17:36         ` Ian Campbell
2015-03-24  5:13           ` Pang, LongtaoX
2015-03-24  8:50             ` Wei Liu
2015-03-24  9:36             ` Ian Campbell
2015-03-23 17:19     ` Wei Liu [this message]
2015-03-23 17:32       ` Ian Campbell
  -- strict thread matches above, loose matches on Subject: below --
2015-03-17 18:16 [OSSTEST Nested PATCH 0/6] Introduction of netsted HVM test job longtao.pang
2015-03-17 18:16 ` [OSSTEST Nested PATCH 2/6] Add and expose some testsupport APIs longtao.pang
2015-03-19 16:27   ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150323171930.GA19412@zion.uk.xensource.com \
    --to=wei.liu2@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=ian.campbell@citrix.com \
    --cc=longtaox.pang@intel.com \
    --cc=robert.hu@intel.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.