All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Xen-devel <xen-devel@lists.xenproject.org>,
	Wei Liu <wei.liu2@citrix.com>
Subject: Re: [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device
Date: Fri, 20 Oct 2017 15:56:22 +0100	[thread overview]
Message-ID: <20171020145622.nszfbikwdhsnaotc@citrix.com> (raw)
In-Reply-To: <23017.64861.529293.658705@mariner.uk.xensource.com>

On Fri, Oct 20, 2017 at 02:42:53PM +0100, Ian Jackson wrote:
> Wei Liu writes ("Re: [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device"):
> > On Fri, Oct 20, 2017 at 02:20:29PM +0100, Ian Jackson wrote:
> > > I don't think the guest booting "eventually" after some timeout is
> > > correct.
> > 
> > The OS is honoring whatever is written in the resume file. That's the
> > correct behaviour to me.
> > 
> > Booting after trying is better than not booting at all.  That parameter
> > isn't vital to a functional system. This has always been the case since
> > forever. It is only the "timeout" in the past is shorter so osstest
> > never noticed it.
> > 
> > What is not correct is the content of the file. We can't blame the guest
> > OS for that.
> 
> I agree with all of this.  But, the overall behaviour is not correct.
> Even though all of the things that osstest specified to
> xen-create-image were correct.  Therefore there is a bug.
> 
> > > > There is no option in xen-create-image to manipulate extra=. The only
> > > > viable solution is to provide a new template. That seems overkill and
> > > > would require more code.
> > > 
> > > What you're saying is that this infelicity in xen-tools is not
> > > particularly easy to fix.  That doesn't mean it shouldn't be reported
> > > as a bug.
> > 
> > There is nothing to fix. System administrator should provide a proper
> > template -- that's what xen-tools developers most likely to say.
> 
> I'm not sure what you mean here.  We are using the config template
> provided by xen-tools.  The reuse of the host's initrd is suggested by
> the xen-create-image documentation.
> 

TBH I don't know what to write in a report. What is the expected
behaviour?

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  reply	other threads:[~2017-10-20 14:56 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-20 10:38 [OSSTEST PATCH 00/16] Upgrade to Stretch Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 01/16] XXX add a stretch config based on production-config Wei Liu
2017-10-20 10:54   ` Ian Jackson
2017-10-20 11:17     ` Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 02/16] gitignore: ignore vim swap file Wei Liu
2017-10-20 10:53   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 03/16] ts-xen-build-prep: don't install w3c-dtd-xhtml for Stretch onward Wei Liu
2017-10-20 10:50   ` Ian Jackson
2017-10-20 11:37     ` Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 04/16] ts-xen-build-prep: install packages for stretch Wei Liu
2017-10-20 10:50   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 05/16] mg-debian-installer-update-all: put quotes around arguments Wei Liu
2017-10-20 10:56   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 06/16] ts-xen-install: install some packages on stretch Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 07/16] Debian.pm: use sysvinit-core " Wei Liu
2017-10-20 10:57   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 08/16] ts-leak-check: suppress systemd-shim, which leaks in stretch Wei Liu
2017-10-20 10:58   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 09/16] ts-host-install: don't use the new nic naming scheme Wei Liu
2017-10-20 10:58   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 10/16] ts-debian-fixup: remove extra= before appending our own Wei Liu
2017-10-20 11:03   ` Ian Jackson
2017-10-20 11:15     ` Wei Liu
2017-10-20 12:52       ` Ian Jackson
2017-10-20 13:09         ` Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume device Wei Liu
2017-10-20 11:05   ` Ian Jackson
2017-10-20 11:14     ` Wei Liu
2017-10-20 12:35       ` Ian Jackson
2017-10-20 13:17         ` Wei Liu
2017-10-20 13:20           ` Ian Jackson
2017-10-20 13:36             ` Wei Liu
2017-10-20 13:42               ` Ian Jackson
2017-10-20 14:56                 ` Wei Liu [this message]
2017-10-20 15:10                   ` Ian Jackson
2017-10-20 15:36                     ` Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 12/16] make-flight: don't test pvgrub for Xen XXX Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 13/16] ts-debian-hvm-install: disable new nic naming scheme Wei Liu
2017-10-20 11:01   ` Ian Jackson
2017-10-20 10:38 ` [OSSTEST PATCH 14/16] ts-xen-build-prep: install e2fslibs-dev Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 15/16] TestSupport: add dpkg option when installing packages Wei Liu
2017-10-20 10:38 ` [OSSTEST PATCH 16/16] ts-guests-nbd-mirror: make it work with stretch Wei Liu
2017-10-20 11:09   ` Ian Jackson
2017-10-20 11:24     ` Wei Liu
2017-10-20 11:31       ` Wei Liu
2017-10-25 13:41 ` [OSSTEST PATCH 00/16] Upgrade to Stretch Wei Liu
2017-10-25 16:36 ` Wei Liu

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=20171020145622.nszfbikwdhsnaotc@citrix.com \
    --to=wei.liu2@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.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.