We are experiencing intermittent failures right now with the ARM
credit2 tests. I suspect the failure probability is low.
CC: Julien Grall
Signed-off-by: Ian Jackson
---
sg-run-job |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sg-run-job b/sg-run-job
index aa97ee6
George Dunlap writes ("Re: [PATCH 10/16] SUPPORT.md: Add Debugging, analysis,
crash post-portem"):
> gdbsx security support: Someone may want to debug an untrusted guest,
> so I think we should say 'yes' here.
I think running gdb on an potentially hostile program is foolish.
> I don't have a str
Jan Beulich writes ("Re: [PATCH 03/16] SUPPORT.md: Add some x86 features"):
> Much depends on whether you think "guest" == "DomU". To me
> Dom0 is a guest, too.
Not to me. I'm with George. (As far as I can make out his message,
which I think was sent with HTML-style quoting which some Citrix thi
osstest service owner writes ("[xen-4.6-testing test] 116250: regressions -
FAIL"):
> flight 116250 xen-4.6-testing real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/116250/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be ru
osstest service owner writes ("[xen-4.5-testing test] 116245: regressions -
FAIL"):
> flight 116245 xen-4.5-testing real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/116245/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be ru
osstest service owner writes ("[xen-4.6-testing test] 116222: regressions -
FAIL"):
> flight 116222 xen-4.6-testing real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/116222/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be ru
osstest service owner writes ("[xen-4.8-testing test] 116221: regressions -
FAIL"):
> flight 116221 xen-4.8-testing real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/116221/
>
> Regressions :-(
...
> version targeted for testing:
> xen 9ba6783e47db71379c5120039b878f
George Dunlap writes ("Re: [BUG] Error applying XSA240 update 5 on 4.8 and 4.9
(patch 3 references CONFIG_PV_LINEAR_PT, 3285e75dea89, x86/mm: Make PV linear
pagetables optional)"):
> These are two different things. Steve's reluctance to backport a
> potentially arbitrary number of non-security-r
Juergen Gross writes ("Re: [OSSTEST PATCH] ts-xen-build-prep: Install
libelf-dev for benefit of linux.git"):
> The kernel now is using objtool to create unwind information. This needs
> libelf to work. Advantage is that this approach no longer depends on
> assembler sources being heavily annotated
Paul Durrant
CC: Wei Liu
Signed-off-by: Ian Jackson
---
ts-xen-build-prep | 1 +
1 file changed, 1 insertion(+)
diff --git a/ts-xen-build-prep b/ts-xen-build-prep
index 3e98364..3309216 100755
--- a/ts-xen-build-prep
+++ b/ts-xen-build-prep
@@ -207,6 +207,7 @@ sub prep () {
Ross Lagerwall writes ("Re: [PATCH] tools: xentoolcore_restrict_all: Do
deregistration before close"):
> On 11/14/2017 12:15 PM, Ian Jackson wrote:
> > + * Note for multi-threaded programs: If xentoolcore_restrict_all is
> > + * called concurrently with a function whi
Julien Grall writes ("Re: [PATCH] tools: xentoolcore_restrict_all: Do
deregistration before close"):
> I think this is 4.10 material, xentoolcore was introduced in this
> release and it would be good to have it right from now. I want to
> confirm that you are both happy with that?
Yes, absolute
/null", thing->fd);
pthread_mutex_lock(&handles_lock);
thing->fd = -1;
pthread_mutex_unlock(&handles_lock);
close(fd);
which would be terribly tedious.
Signed-off-by: Ian Jackson
---
tools/libs/call/core.c | 4 ++--
tools
Ross Lagerwall writes ("Re: [PATCH for-4.10] libs/evtchn: Remove active handler
on clean-up or failure"):
> Now that I look at it, a similar scenario can happen during open. Since
> the handle is registered before it is actually opened, a concurrent
> xentoolcore_restrict_all() will try to restr
sh the application later one.
Sorry for not spotting this during review.
The fix is correct as far as it goes, so:
Acked-by: Ian Jackson
> > The call to xentoolcore_deregister_active_handle is done at the same
> > place as for the grants. But I am not convinced this is thread saf
Olaf Hering writes ("Re: [Xen-devel] [PATCH v1] tools/hotplug: convert
proc-xen.mount to proc-xen.service"):
> On Wed, Nov 08, Wei Liu wrote:
> > But is there really no way to ask nicely to see if systemd would accept
> > a change in behaviour? That is, to make proc-xen.mount (or any attempt
> > t
Julien Grall writes ("Re: Bringing up OSS test framework on moonshot(aarch64)
systems"):
> On 08/11/17 11:39, Ian Jackson wrote:
> > I'm not familiar with the referent of "moonshot" in this context. IME
> > "moonshot" is a project name chosen mul
osstest service owner writes ("[linux-linus test] 115643: regressions - FAIL"):
> flight 115643 linux-linus real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/115643/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
> tes
Roger Pau Monné writes ("Re: [Xen-devel] [linux-4.9 test] 115504: regressions -
FAIL"):
> On Fri, Nov 03, 2017 at 08:21:31PM +, osstest service owner wrote:
> > flight 115504 linux-4.9 real [real]
> > http://logs.test-lab.xenproject.org/osstest/logs/115504/
> >
> > Regressions :-(
> >
> > Te
Bhupinder Thakur writes ("Bringing up OSS test framework on moonshot(aarch64)
systems"):
> While going through [1], I have some queries/doubts on the configuration.
>
> 1. The following configuration:
>
> DnsDomain uk.xensource.com
> NetNameservers 10.80.248.2 10.80.16.28 10.80.16.67
> HostProp_
Markus Armbruster writes ("Re: [Qemu-devel] [PATCH 7/8] os-posix: Provide new
-runasid option"):
> Ian Jackson writes:
> > qemu_strtoul fails (returns an error) if the delimiter (that is, the
> > first character which is not processed as digit by strtoul) is not
> >
This does some safety checks and reduces the risk of c&p mistakes.
It has to be run as osst...@osstest.test-lab (or equivalent).
Signed-off-by: Ian Jackson
---
mg-force-push | 121 ++
1 file changed, 121 insertions(+)
create mode 10
This makes the output of mg-force-push quite unpleasant, amongst other
things.
Signed-off-by: Ian Jackson
---
ap-push | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ap-push b/ap-push
index a27ccc2..6c95b1f 100755
--- a/ap-push
+++ b/ap-push
@@ -17,7 +17,7 @@
# along with
osstest service owner writes ("[linux-4.9 test] 115538: regressions - FAIL"):
> flight 115538 linux-4.9 real [real]
> http://logs.test-lab.xenproject.org/osstest/logs/115538/
>
> Regressions :-(
>
> Tests which did not succeed and are blocking,
> including tests which could not be run:
> test-am
Hi. Thanks for the (re)-review.
Markus Armbruster writes ("Re: [Qemu-devel] [PATCH 7/8] os-posix: Provide new
-runasid option"):
> Ian Jackson writes:
> > +case QEMU_OPTION_runasid:
> > +errno = 0;
> > +lv = strtoul(optarg, &ep,
Jan Beulich writes ("Re: [PATCH for-next 1/9] gcov: return ENOSYS for
unimplemented gcov domctl"):
> On 26.10.17 at 11:19, wrote:
> > --- a/xen/common/gcov/gcov.c
> > +++ b/xen/common/gcov/gcov.c
> > @@ -239,7 +239,7 @@ int sysctl_gcov_op(struct xen_sysctl_gcov_op *op)
> > break;
> >
>
Wei Liu writes ("[OSSTEST PATCH 2/2] make-flight: guest should use jessie to
test pvgrub"):
> Stretch has 64bit feature enabled for ext4, which pvgrub can't cope.
> We want to continue to test pvgrub, so specify jessie in the guest
> suite field.
I'm not entirely comfortable with the hardcoding h
George Dunlap writes ("Re: [Xen-devel] Commit moratorium to staging"):
> Well, with a looping xen-build going on in the guest, I've done 40 local
> migrates with no problems yet.
>
> But Roger -- is this on emulated devices only, no PV drivers?
Yes. None of our Windows tests have PV drivers.
Ia
Roger Pau Monne writes ("[PATCH v2] osstest: fix rm to use '-f' in
ts-freebsd-host-install"):
> It's perfectly valid for the .tmp file to not exists, and the script
> shouldn't fail in that case.
Acked-by: Ian Jackson
__
Roger Pau Monne writes ("[PATCH] osstest: remove unneeded rm in
ts-freebsd-host-install"):
> The usage of `rm` here is wrong for two reasons:
>
> - It will fail if $sharedpath.tmp doesn't exist and report and error
>(ie: -f should be used).
> - It's not needed because dd will truncate $shar
Wei Liu writes ("[OSSTEST PATCH 1/2] ts-debian-di-install: use gho to pick
d-i"):
> The original code used ho which gave us the host suite, but we wanted
> the guest suite.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-dev
We want to keep the old testid or some new failures will be "never
pass".
Roger reports that this change makes the existing host-specific
Windows migration failures fail everywhere, so so things may need
force pushing.
CC: Roger Pau Monné
Signed-off-by: Ian Jackson
---
sg-run-job
Julien Grall writes ("Re: Commit moratorium to staging"):
> Thank you for the explanation. I agree with the force push to unblock
> master (and other tree I mentioned).
I will force push all the affected trees, but in a reactive way
because I base each force push on a test report - so it won't be
Roger Pau Monné writes ("Re: [Xen-devel] Commit moratorium to staging"):
> Is there anyway to get that from windows in an automatic way? If not I
> could test that with a Debian guest. In fact it might even be a good
> thing for Linux based guest to be added to the regular migration tests
> in orde
Julien Grall writes ("Re: Commit moratorium to staging"):
> Hi Ian,
>
> Thank you for the detailed e-mail.
>
> On 11/01/2017 02:07 PM, Ian Jackson wrote:
> > Furthermore, the test is not intermittent, so a force push will be
> > effective in the following
So, investigations (mostly by Roger, and also a bit of archaeology in
the osstest db by me) have determined:
* This bug is 100% reproducible on affected hosts. The repro is
to boot the Windows guest, save/restore it, then migrate it,
then shut down. (This is from an IRL conversation with Rog
Wei Liu writes ("[PATCH v3 for-4.10] scripts: introduce a script for build
test"):
> Signed-off-by: Ian Jackson
> Signed-off-by: Wei Liu
...
...
> +trap "echo Restoring original HEAD ; git checkout $ORIG_BRANCH" EXIT
This will smash the whole script's exit
gt;
> > Signed-off-by: Andrii Anisov
> > Reviewed-by: Volodymyr Babchuk
> > Reviewed-by: Oleksandr Andrushchenko
>
> Acked-by: Wei Liu
>
> Julien I think we should apply this for 4.10.
I agree. Subject line tag added.
Acked-by: Ian Jackson
Ian.
_
Roger Pau Monne writes ("[PATCH 12/16] osstest: add script to install build
dependencies on FreeBSD"):
> Since at the moment osstest only builds FreeBSD on FreeBSD, there are
> no dependencies to install. Just mark the host as ready to share.
Acked
Roger Pau Monne writes ("[PATCH 14/16] osstest: add support for FreeBSD
buildjobs to sg-run-job"):
> Add support and introduce a FreeBSD build job to sg-run-job.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.
- s/resource_shared_mark_ready/host_shared_mark_ready/.
> - First argument of jobdb_resource_shared_mark_ready must be 'host'.
Although it would be good to mention that you are fixing a bug here,
to wit the $ho->{Ident} to 'host' change.
Acked-by: Ian Jackson
Thanks,
Ian.
, of course.
Also `$resource' here is the wrong variable name. This is actually
the $sharetype (as is evident from jobdb_resource_shared_mark_ready
and what is now executive_resource_shared_mark_ready).
CC: Roger Pau Monné
Signed-off-by: Ian Jackson
---
Osstest/TestSupport.pm | 6 +++---
1 fil
This function only works on resource of restype `host'. Ie, hosts.
Signed-off-by: Ian Jackson
CC: Roger Pau Monné
---
Osstest/TestSupport.pm | 4 ++--
ts-freebsd-host-install | 4 ++--
ts-xen-build-prep | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/Os
gered from within dom0 (there
> is a check in do_control() rejecting all non-dom0 requests with
> an EACCES).
Acked-by: Ian Jackson
(Added the for-4.10 tag to the Subject.)
Ian.
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Roger Pau Monne writes ("[PATCH v6 07/20] osstest: introduce
resource_shared_mark_ready"):
> That allows marking a host as ready to be shared. Replace the current
> caller that open-codes it.
...
> -$mjobdb->jobdb_resource_shared_mark_ready
> - ($ho->{Ident}, $ho->{Name}, "build-".$ho->{Suite}."
Roger Pau Monne writes ("[PATCH for-next 1/9] gcov: return ENOSYS for
unimplemented gcov domctl"):
> Signed-off-by: Roger Pau Monné
...
> default:
> -ret = -EINVAL;
> +ret = -ENOSYS;
> break;
> }
Reviewed-by: Ian Jackson
I think t
Stefano Stabellini writes ("Re: [PATCH v5.1 8/8] configure: do_compiler: Dump
some extra info under bash"):
> CC'ing the maintainers for this.
Thanks, but scripts/get_maintainer.pl seems to print different
information for me... (see my other mail)
Ian.
__
found, printing recent contributors.
get_maintainer.pl: Do not blindly cc: them on patches! Use common sense.
Anthony PERARD (commit_signer:1/2=50%)
Paolo Bonzini
(commit_signer:1/2=50%,commit_signer:11/57=19%)
Ian Jackson (commit_signer:1/2=50%)
Michael Tokarev (commit_signer:
Stefano Stabellini writes ("Re: [PATCH v5.1 2/8] xen: restrict: use
xentoolcore_restrict_all"):
> On Fri, 20 Oct 2017, Ian Jackson wrote:
...
> > Drop individual use of xendevicemodel_restrict and
> > xenforeignmemory_restrict. These are not actually effective in
Stefano Stabellini writes ("Re: [PATCH v5.1 6/8] xen: destroy_hvm_domain: Try
xendevicemodel_shutdown"):
> On Fri, 20 Oct 2017, Ian Jackson wrote:
> > xc_interface_open etc. is not going to work if we have dropped
> > privilege, but xendevicemodel_shutdown will if eve
Stefano Stabellini writes ("Re: [PATCH v5.1 1/8] xen: link against
xentoolcore"):
> On Fri, 20 Oct 2017, Ian Jackson wrote:
> >then
> > - xen_stable_libs="-lxendevicemodel $xen_stable_libs"
> > + xen_stable_libs="-lxendevicemodel
Wei Liu writes ("Re: [PATCH v2] scripts: introduce a script for build test"):
> On Wed, Oct 25, 2017 at 04:25:21PM +0100, Ian Jackson wrote:
> > If you are worried about this you should check that there are no
> > uncommitted files before starting.
>
> This is alrea
Wei Liu writes ("Re: [PATCH v2] scripts: introduce a script for build test"):
> On Tue, Oct 24, 2017 at 02:38:39PM +0100, Ian Jackson wrote:
> > Anthony PERARD writes ("Re: [PATCH v2] scripts: introduce a script for
> > build test"):
> > > That feels w
d our mind,
or because of changed circumstances such as the struct growing
significantly) we can just change it. Of course it's better to get it
right first time so if there is a good reason.
> On 19/10/17 16:17, Ian Jackson wrote:
> > Andrew Cooper writes ("Re: [PATCH for-4.10
Anthony PERARD writes ("Re: [PATCH v5.1 7/8] os-posix: Provide new -runas
: facility"):
> On Fri, Oct 20, 2017 at 02:38:21PM +0100, Ian Jackson wrote:
> > +static bool os_parse_runas_uid_gid(const char *optarg)
...
> > +errno = 0;
> > +lv = strtoul(optarg,
Anthony PERARD writes ("Re: [PATCH v2] scripts: introduce a script for build
test"):
> That feels wrong. How do I run the same exact command at the default
> one, but with -j8 instead of -j4?
.../build-test sh -ec make -j4 distclean && ./configure && make -j4
But I think Anthony has a point. T
Roger Pau Monné writes ("Re: [PATCH v12 00/33] osstest: FreeBSD host support"):
> Sorry for the delay, had to cherry-pick some commits from the FreeBSD
> host install series in order for the examine one to work. I've pushed
> this to the following branch:
>
> git://xenbits.xen.org/people/royger/os
17 00:00:00 2001
From: Anthony PERARD
Date: Tue, 24 Oct 2017 11:37:20 +0100
Subject: [OSSTEST PATCH] mg-hosts: Fix of showprops doc comment
./mg-hosts showprops description and implementation didn't match.
Fix description.
Signed-off-by: Anthony PERARD
Signed-off-by: Ian Jackson
---
mg-host
Wei Liu writes ("[PATCH v2] scripts: introduce a script for build test"):
...
> +if git branch | grep -q '^\*.\+detached at'; then
You mean some rune involving git-symbolic-ref.
git-symbolic-ref -q HEAD exits with status 1 if HEAD is detached, 0 if
HEAD is a branch, or some other status in case o
Wei Liu writes ("Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for
build testing"):
> On Mon, Oct 23, 2017 at 03:50:31PM +0100, Anthony PERARD wrote:
> > FIY, I do like to put script and other files in my checkouts, the git
> > clean will remove them.
>
> I changed that to make distclean
Julien Grall writes ("Re: [Xen-devel] [xen-unstable test] 115037: regressions -
FAIL"):
> Would it be possible of a platform specific bug? The last two flights
> are failing on merlot1.
The merlots are a highly unusual AMD machines which have NUMA nodes
with no memory and seem to sometimes have
Wei Liu writes ("Re: [Xen-devel] [PATCH for-4.10] libxl: annotate s to be
nonnull in libxl__enum_from_string"):
> On Mon, Oct 23, 2017 at 01:32:50PM +0100, Julien Grall wrote:
> > I would be ok with that. Wei do you have any opinion?
>
> Sure this is a simple enough patch. We should preferably tu
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"):
> On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote:
> > On 20.10.17 at 19:32, wrote:
> > > +git rebase $BASE $TIP -x "$CMD"
> >
> > Is this quoting on $CMD really going to work right no matter what
> > the
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"):
> On Mon, Oct 23, 2017 at 01:02:00PM +0100, Ian Jackson wrote:
> > In particular, if you:
> > * check that the tree is not dirty
> > * detach HEAD
>
> I think these two che
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"):
> On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote:
> > What is this startup delay intended for?
>
> To give user a chance to check the command -- git-rebase can be
> destructive after all.
I can't resist
Ross Lagerwall writes ("[PATCH v2 4/5] tools: libxendevicemodel: Provide
xendevicemodel_add_to_physmap"):
> Signed-off-by: Ross Lagerwall
Assuming the hypervisor parts go in:
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@
Ian Jackson writes ("[PATCH v5.1 8/8] configure: do_compiler: Dump some extra
info under bash"):
> This makes it much easier to find a particular thing in config.log.
>
> The information may be lacking in other shells, resulting in harmless
> empty output. (This is why w
Stefano Stabellini writes ("Re: [Xen-devel] linux-arm-xen branch, commit
access, etc."):
> On Fri, 20 Oct 2017, Konrad Rzeszutek Wilk wrote:
> > 3. Use upstream released kernels. Follow them when they are released.
>
> I agree with Konrad. The reason why that branch is there is that
> initially w
Wei Liu writes ("[PATCH for-4.10] docs: update coverage.markdown"):
> The coverage support in hypervisor is redone. Update the document.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
We have decided:
We will push the anoint and examine parts of this series to osstest
pretest. (You're going to give me a suitable branch on Monday.)
This should work because we have anointed FreeBSD builds already.
If this works (passes pretest) we will then run a special invocation
of the
Wei Liu writes ("Re: [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume
device"):
> 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"):
> >
Roger Pau Monne writes ("[PATCH v12 31/33] ts-examine-hostprops-save: introduce
a script to save properties"):
> This script turns the properties stored in the runvars using the
> format hostprop/$ident/$prop=$val into host properties stored in the
> database.
Acked-by: Ian
Do not pass a host ident to ts-examine-hostprops-save.
> - Use .- for ts-memdisk-try-append so that the rest of the job will
>run even if this step fails.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Roger Pau Monne writes ("[PATCH v12 29/33] ts-freebsd-host-install: add
arguments to test memdisk append options"):
> This is needed in order to figure out which memdisk options should be
> used to boot the images on each specific box.
>
> Note that when passed the --recordappend argument upon su
ng" in the commit message
and error message, since there is also a current blessing etc.
With that changed,
Acked-by: Ian Jackson
Ian.
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Currently we are running our ARM tests in osstest off a branch in
Stefano's personal Linux tree. This is a bit unsatisfactory.
We would like to switch to a branch that Julien can push to too, and
that is in a more official place.
There are two options:
1. Create an ARM-specific Xen tree. Curr
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
From: Anthony PERARD
Xen libraries in 4.10 include a new xentoolcore library. This
contains the xentoolcore_restrict_all function which we are about to
want to use.
Signed-off-by: Ian Jackson
---
v5: More truthful commit message.
---
configure | 8 +---
1 file changed, 5 insertions(+), 3
is is not a regression in this patch, because
previously the -xen-restrict-domid call was, in fact, simply
ineffective!) We will revisit this in the Xen 4.11 release cycle.
Signed-off-by: Ian Jackson
Reviewed-by: Anthony PERARD
---
v5: Discuss problems with migration startup notification
i
ers would be at the bottom of the
file, so that they can naturally benefit from the compatibility layers
for earlier version. But that's rather too much for this series.)
No functional change.
Signed-off-by: Ian Jackson
Acked-by: Anthony PERARD
---
v2: New patch in v2 of the series
---
in
We are going to want to reuse this.
No functional change.
Signed-off-by: Ian Jackson
Reviewed-by: Anthony PERARD
---
hw/i386/xen/xen-hvm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c
index 7b60ec6..83420cd 100644
--- a
ed if configure is run with bash. The
something), it is necessary to say bash ./configure to get the extra
debug info in the log.
Signed-off-by: Ian Jackson
---
v4: No longer tag this patch RFC.
---
configure | 4
1 file changed, 4 insertions(+)
diff --git a/configure b/configure
index 6f
, because the
restriction needs to be done very late - after qemu has opened all of
its control fds.
xentoolcore_restrict_all and xentoolcore.h are available in Xen 4.10
and later, only. Provide a compatibility stub. And drop the
compatibility stubs for the old functions.
Signed-off-by: Ian Jackson
xc_interface_open etc. is not going to work if we have dropped
privilege, but xendevicemodel_shutdown will if everything is new
enough.
xendevicemodel_shutdown is only availabe in Xen 4.10 and later, so
provide a stub for earlier versions.
Signed-off-by: Ian Jackson
Reviewed-by: Anthony PERARD
x27;t
know what gid we ought to use (since uids may eppear in multiple
passwd file entries with different gids).
Signed-off-by: Ian Jackson
---
v5: Use : rather than . to separate uid from gid
v4: Changed to reuse option -runas
v3: Error messages fixed. Thanks to Peter Maydell and Ross Lag
Anthony PERARD writes ("Re: [PATCH v5 0/8] xen: xen-domid-restrict
improvements"):
> The patches in this v5 appear to be the same the one from the patch
> series v4.
Erk, so they are.
I'll post a v5.1 in reply to this email.
Ian.
___
Xen-devel mailin
Wei Liu writes ("Re: [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume
device"):
> I wouldn't call this behaviour a bug. Most people won't notice it
> because the guest will boot eventually. It is not like this will cause
> the guest to crash.
I don't think the guest booting "eventually"
Wei Liu writes ("Re: [OSSTEST PATCH 10/16] ts-debian-fixup: remove extra=
before appending our own"):
> On Fri, Oct 20, 2017 at 12:03:24PM +0100, Ian Jackson wrote:
> > Wei Liu writes ("[OSSTEST PATCH 10/16] ts-debian-fixup: remove extra=
> > before appending our o
Wei Liu writes ("Re: [OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume
device"):
> On Fri, Oct 20, 2017 at 12:05:55PM +0100, Ian Jackson wrote:
> > Wei Liu writes ("[OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume
> > device"):
>
Wei Liu writes ("[OSSTEST PATCH 16/16] ts-guests-nbd-mirror: make it work with
stretch"):
> On the server side, only add oldstyle= and port= on jessie. Stretch
> doesn't support or need those anymore.
See my earlier comments about old vs new Debian suite names.
> Prune check for older versions
Wei Liu writes ("[OSSTEST PATCH 11/16] ts-debian-fixup: use correct resume
device"):
> See code comment for explanation.
...
> +# There might be stale entries in /etc/initramfs-tools/conf.d/resume
> +# which get stored in the initramfs. That introduces delay in guest
> booting
> +# wh
Wei Liu writes ("[OSSTEST PATCH 10/16] ts-debian-fixup: remove extra= before
appending our own"):
> The original extra= was not removed, so there were two extra= in the
> resulting config file.
What is the original extra= ? Why should we not combine them ?
> It wasn't a problem for xl because t
Wei Liu writes ("[OSSTEST PATCH 13/16] ts-debian-hvm-install: disable new nic
naming scheme"):
> This is required to fix nested hvm test. The L1 host is installed by
> this script. We want the L1 host to not use the new nic naming scheme.
The principle is fine.
> +# Do not use "Predictable N
Wei Liu writes ("[OSSTEST PATCH 09/16] ts-host-install: don't use the new nic
naming scheme"):
> Signed-off-by: Wei Liu
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Wei Liu writes ("[OSSTEST PATCH 07/16] Debian.pm: use sysvinit-core on
stretch"):
...
> diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm
> index 845027a..24bc260 100644
> --- a/Osstest/Debian.pm
> +++ b/Osstest/Debian.pm
> @@ -827,7 +827,7 @@ sub preseed_base ($$$;@) {
>
> # Systemd doesn
Wei Liu writes ("[OSSTEST PATCH 08/16] ts-leak-check: suppress systemd-shim,
which leaks in stretch"):
> Signed-off-by: Wei Liu
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Wei Liu writes ("[OSSTEST PATCH 05/16] mg-debian-installer-update-all: put
quotes around arguments"):
> Signed-off-by: Wei Liu
...
> suite=$1
> arch=$2
> -packages="$3"
> +packages=$3
Not sure why this needless style change, but if you did it
deliberately I don't really mind...
> site=http:/
Wei Liu writes ("[OSSTEST PATCH 01/16] XXX add a stretch config based on
production-config"):
> diff -ub production-config production-config-stretch
The changes LGTM but obviously this ought to go straight into
`production-config'.
Ian.
___
Xen-devel
Wei Liu writes ("[OSSTEST PATCH 02/16] gitignore: ignore vim swap file"):
> Signed-off-by: Wei Liu
Acked-by: Ian Jackson
Although, you may find your life improved by putting this in your
~/.config/git/ignore.
Ian.
___
Xen-devel mai
Wei Liu writes ("[OSSTEST PATCH 04/16] ts-xen-build-prep: install packages for
stretch"):
> Stubdom build needs texinfo.
Same comment as my previous patch. You should only mention old
release names in these kind of tests, unless you know that the requirement
is specific to only stretch and not f
1 - 100 of 4993 matches
Mail list logo