Hi,
here a new version of my proposal for fuzzing the hypervisor. The original
can be found here: [1].
==
1. Motivation and Description
==
Fuzzing is a recent trend for systematic testing of interfaces by trying
more or less random i
flight 110984 linux-linus real [real]
http://logs.test-lab.xenproject.org/osstest/logs/110984/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemuu-ovmf-amd64 14 guest-saverestore.2 fail REGR. vs.
110515
test-amd64-i386
On 2017-06-23 10:03 AM, Boris Ostrovsky wrote:
On 06/23/2017 12:54 PM, Olaf Hering wrote:
On Thu, Jun 22, Boris Ostrovsky wrote:
They are queued for 4.13.
git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.13
This works for me. Thanks.
I assume there is no ready-to-pull
Remove unnecessary variable mfn in function xen_foreach_remap_area() and,
refactor the code.
Variable mfn at line 518:mfn = xen_remap_buf.mfns[i];
is only being used to store a value to be passed as
an argument to the xen_update_mem_tables() function.
This value can be passed directly, which makes
flight 110978 xen-4.9-testing real [real]
http://logs.test-lab.xenproject.org/osstest/logs/110978/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-armhf-armhf-xl 6 xen-boot fail in 110942 REGR. vs. 110550
Tests which are
flight 111020 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/111020/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a
test-amd64-amd64-libvirt 12 mig
> -Original Message-
> From: Stefano Stabellini [mailto:sstabell...@kernel.org]
> Sent: Friday, June 23, 2017 4:09 PM
> To: Jarvis Roach
> Cc: Stefano Stabellini ; Julien Grall
> ; Zhongze Liu ; xen-
> de...@lists.xenproject.org; Wei Liu ; Ian Jackson
> ; edg...@xilinx.com; Edgar E. Igle
On 06/23/2017 08:09 PM, Sergej Proskurin wrote:
Hi Julien,
Hi Sergej,
[...]
Looking at the code, I see very limited point of having the offsets
array as you don't use a loop and also use each offset in a single place.
+((paddr_t)(gva >> 20) & ((1ULL << (12 - n)) - 1)),
Don'
Hi Stefano,
On 06/23/2017 07:21 PM, Stefano Stabellini wrote:
On Fri, 23 Jun 2017, Julien Grall wrote:
Hi,
On 22/06/17 22:05, Stefano Stabellini wrote:
When we encounter an id IDx during "xl create":
+ If it’s not under /local/shared_mem:
+ If the corresponding entry has a "master" t
On Fri, 23 Jun 2017, Jarvis Roach wrote:
> > -Original Message-
> > From: Stefano Stabellini [mailto:sstabell...@kernel.org]
> > Sent: Friday, June 23, 2017 2:21 PM
> > To: Julien Grall
> > Cc: Stefano Stabellini ; Zhongze Liu
> > ; xen-de...@lists.xenproject.org; Wei Liu
> > ; Ian Jackson
Hi,
On 06/23/2017 07:28 PM, Stefano Stabellini wrote:
On Fri, 23 Jun 2017, Julien Grall wrote:
Hi Stefano,
On 22/06/17 23:53, Stefano Stabellini wrote:
On Thu, 22 Jun 2017, Bhupinder Thakur wrote:
+static void vpl011_write_data(struct domain *d, uint8_t data)
+{
+unsigned long flags;
+
On 2017-06-23 10:03 AM, Boris Ostrovsky wrote:
On 06/23/2017 12:54 PM, Olaf Hering wrote:
On Thu, Jun 22, Boris Ostrovsky wrote:
They are queued for 4.13.
git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.13
This works for me. Thanks.
I assume there is no ready-to-pull
> -Original Message-
> From: Stefano Stabellini [mailto:sstabell...@kernel.org]
> Sent: Friday, June 23, 2017 2:21 PM
> To: Julien Grall
> Cc: Stefano Stabellini ; Zhongze Liu
> ; xen-de...@lists.xenproject.org; Wei Liu
> ; Ian Jackson ; Jarvis Roach
> ; edg...@xilinx.com; Edgar E. Igles
Hi Julien,
[...]
>
> Looking at the code, I see very limited point of having the offsets
> array as you don't use a loop and also use each offset in a single place.
>
>> +((paddr_t)(gva >> 20) & ((1ULL << (12 - n)) - 1)),
>
Don't you think it is more readable to have the GVA offsets at
On Fri, 23 Jun 2017, Jan Beulich wrote:
> >>> On 22.06.17 at 20:52, wrote:
> > On Thu, 22 Jun 2017, Jan Beulich wrote:
> >> >>> On 21.06.17 at 20:46, wrote:
> >> > On Wed, 21 Jun 2017, Jan Beulich wrote:
> >> >> >>> On 20.06.17 at 23:48, wrote:
> >> >> > On Tue, 20 Jun 2017, Jan Beulich wrote:
>
On Fri, 23 Jun 2017, Julien Grall wrote:
> Hi Stefano,
>
> On 22/06/17 23:53, Stefano Stabellini wrote:
> > On Thu, 22 Jun 2017, Bhupinder Thakur wrote:
> > > +static void vpl011_write_data(struct domain *d, uint8_t data)
> > > +{
> > > +unsigned long flags;
> > > +struct vpl011 *vpl011 =
On Fri, 23 Jun 2017, Julien Grall wrote:
> Hi,
>
> On 21/06/17 14:10, Julien Grall wrote:
> > Hi Stefano.
> >
> > On 21/06/17 00:04, Stefano Stabellini wrote:
> > > Add a warning: use passthrough with care.
> > >
> > > Add a pointer to the gic device tree bindings. Add an explanation on how
> >
On Fri, 23 Jun 2017, Julien Grall wrote:
> Hi,
>
> On 22/06/17 22:05, Stefano Stabellini wrote:
> > > When we encounter an id IDx during "xl create":
> > >
> > > + If it’s not under /local/shared_mem:
> > > + If the corresponding entry has a "master" tag, create the
> > > correspondin
flight 111009 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/111009/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a
test-amd64-amd64-libvirt 12 mig
On Fri, Jun 23, Wei Liu wrote:
> We support >=4.1. Please check those as well.
Yes, 4.1.2 works as well.
Olaf
signature.asc
Description: PGP signature
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
On Fri, 23 Jun 2017, Julien Grall wrote:
> Hi Bhupinder,
>
> On 22/06/17 08:38, Bhupinder Thakur wrote:
> > There are some TBD items which need to be looked at in the future:
> >
> > 1. Currently UEFI firmware logs the output to hvc console only. How can
> >UEFI firmware be made aware of pl01
On Fri, 23 Jun 2017, Julien Grall wrote:
> On 23/06/17 14:17, Julien Grall wrote:
> >
> >
> > On 23/06/17 00:04, Stefano Stabellini wrote:
> > > > diff --git a/tools/libxc/include/xenctrl.h
> > > > b/tools/libxc/include/xenctrl.h
> > > > index 1629f41..26f3d1e 100644
> > > > --- a/tools/libxc/inc
On Fri, Jun 23, Wei Liu wrote:
> We support >=4.1. Please check those as well.
According to the PDF manuals at https://gcc.gnu.org/onlinedocs/ a
"-fno-foo" is mentioned, so I think -fno-pic is recognized. I will see
if I find a copy of SLE10 to verify with 4.1.2.
Olaf
signature.asc
Descriptio
On 6/23/2017 5:00 AM, Borislav Petkov wrote:
On Fri, Jun 16, 2017 at 01:56:19PM -0500, Tom Lendacky wrote:
Add the support to encrypt the kernel in-place. This is done by creating
new page mappings for the kernel - a decrypted write-protected mapping
and an encrypted mapping. The kernel is encry
flight 110988 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/110988/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf c01f13d52a85f097e1cc6b194df1316a3ed24710
baseline version:
ovmf 95b5c32fb3d6bf677d4cb
On Fri, Jun 23, 2017 at 07:40:36PM +0200, Olaf Hering wrote:
> On Fri, Jun 23, Wei Liu wrote:
>
> > Do you need to check if the compiler supports -fno-pic?
>
> In my testing gcc-4.3 and 4.5 know about this option.
>
We support >=4.1. Please check those as well.
On Fri, Jun 23, Wei Liu wrote:
> Do you need to check if the compiler supports -fno-pic?
In my testing gcc-4.3 and 4.5 know about this option.
Olaf
signature.asc
Description: PGP signature
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://li
On Fri, Jun 16, 2017 at 01:56:39PM -0500, Tom Lendacky wrote:
> Add support to check if SME has been enabled and if memory encryption
> should be activated (checking of command line option based on the
> configuration of the default state). If memory encryption is to be
> activated, then the encry
gcc7 is more strict with functions marked as inline. They are not
automatically inlined. Instead a function call is generated, but the
actual code is not visible by the linker.
Do a mechanical change and mark every 'inline' as 'static inline'. For
simpler review the static goes into an extra line.
On Fri, Jun 23, 2017 at 07:26:01PM +0200, Olaf Hering wrote:
> If the default compiler silently defaults to to -fPIC/-fPIE building
> rombios fails:
>
> ld -melf_i386 -s -r 32bitbios.o tcgbios/tcgbiosext.o util.o pmm.o -o
> 32bitbios_all.o
> There are undefined symbols in the BIOS:
>
If the default compiler silently defaults to to -fPIC/-fPIE building
rombios fails:
ld -melf_i386 -s -r 32bitbios.o tcgbios/tcgbiosext.o util.o pmm.o -o
32bitbios_all.o
There are undefined symbols in the BIOS:
U _GLOBAL_OFFSET_TABLE_
make[10]: *** [Makefile:26: 32bitbios_all.o] Error
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 11/20] ts-openstack-deploy:
Increase fd and memory limits for rabbitmq"):
> On Fri, Jun 23, 2017 at 04:34:41PM +0100, Ian Jackson wrote:
> > And if one isn't using systemd ?
>
> I guess it would have to fix devstack. Also, the default is 1k, devstack
On 06/23/2017 12:54 PM, Olaf Hering wrote:
On Thu, Jun 22, Boris Ostrovsky wrote:
They are queued for 4.13.
git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.13
This works for me. Thanks.
I assume there is no ready-to-pull variant for linux-4.4.x?
nm.
Not that I am a
Anthony PERARD writes ("[OSSTEST PATCH v11 20/20] Introduce flight for stable
branches of OpenStack"):
> OpenStack have many different repo which should be in sync, so this
> patch should grab the revisions of the stable branch of every OpenStack
> tree. Tempest does not have stable branch and sho
flight 110975 qemu-upstream-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/110975/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-armhf-armhf-xl-credit2 15 guest-start/debian.repeat fail REGR. vs. 106833
Tests wh
On Thu, Jun 22, Boris Ostrovsky wrote:
> They are queued for 4.13.
> git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.13
This works for me. Thanks.
I assume there is no ready-to-pull variant for linux-4.4.x?
nm.
Olaf
signature.asc
Description: PGP signature
__
On Fri, Jun 23, 2017 at 04:34:41PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("Re: [OSSTEST PATCH v11 11/20] ts-openstack-deploy:
> Increase fd and memory limits for rabbitmq"):
> > On Fri, Jun 23, 2017 at 02:41:59PM +0100, Ian Jackson wrote:
> > > Anthony PERARD writes ("[OSSTEST PATCH v1
On 06/23/2017 11:00 AM, Jan Beulich wrote:
So far callers of the libxc interface passed in a domain ID which was
then ignored in the hypervisor. Instead, make the hypervisor honor it
(accepting DOMID_INVALID to obtain original behavior), allowing to
query whether a device can be assigned to a par
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 20/20] Introduce flight for
stable branches of OpenStack"):
> We decided to track only nova.git, and let osstest clone other trees and
> checkout the HEAD (by having REVISION_* empty). That is fine if we track
> "master" of nova.git.
Right.
> The is
On Fri, Jun 23, 2017 at 02:58:25PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 20/20] Introduce flight for stable
> branches of OpenStack"):
> > OpenStack have many different repo which should be in sync, so this
> > patch should grab the revisions of the stable branch o
On Thu, 2017-06-22 at 17:55 +0100, George Dunlap wrote:
> On 08/06/17 13:08, Dario Faggioli wrote:
> > This commit implements the Xen part of the cap mechanism for
> > Credit2.
> >
> > A cap is how much, in terms of % of physical CPU time, a domain
> > can execute at most.
> >
> > For instance, a
flight 111006 xen-unstable-smoke real [real]
http://logs.test-lab.xenproject.org/osstest/logs/111006/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a
test-amd64-amd64-libvirt 12 mig
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 18/20] ts-logs-capture: Capture
OpenStack logs"):
> On Fri, Jun 23, 2017 at 02:49:11PM +0100, Ian Jackson wrote:
> > This is not fine. If a build host is shared, it will collect all the
> > logs from all of the builds.
>
> This file should not exist
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 16/20] ts-openstack-tempest:
Update list of skipped tests"):
> On Fri, Jun 23, 2017 at 02:47:37PM +0100, Ian Jackson wrote:
> > Again, does this not mean we're going to suffer a maintenance burden
> > as tempest grows new inapplicable tests ?
>
> Tha
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 12/20] make-flight: Increase
dom0_mem for openstack flight"):
> On Fri, Jun 23, 2017 at 02:42:53PM +0100, Ian Jackson wrote:
> > Anthony PERARD writes ("[OSSTEST PATCH v11 12/20] make-flight: Increase
> > dom0_mem for openstack flight"):
> > > With 4
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 11/20] ts-openstack-deploy:
Increase fd and memory limits for rabbitmq"):
> On Fri, Jun 23, 2017 at 02:41:59PM +0100, Ian Jackson wrote:
> > Anthony PERARD writes ("[OSSTEST PATCH v11 11/20] ts-openstack-deploy:
> > Increase fd and memory limits for
Roger Pau Monne writes ("[PATCH v3.1 8/8] osstest: hook FreeBSD flight into
cr-daily-branch"):
> +++ b/daily-cron-email-real--freebsd
> @@ -0,0 +1,4 @@
> +To: xen-de...@lists.xenproject.org,
> +osstest-ad...@xenproject.org,
> +roy...@freebsd.org
> +Bcc: osstest-out...@lists.xenproject.org
On Fri, Jun 23, 2017 at 02:49:11PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 18/20] ts-logs-capture: Capture
> OpenStack logs"):
> > + /var/log/openstack/*.log
> > + /etc/nova/*
> > + /etc/neutron/*
> > +
Roger Pau Monne writes ("[PATCH v3 8/8] osstest: hook FreeBSD flight into
cr-daily-branch"):
> +++ b/daily-cron-email-freebsd
> @@ -0,0 +1 @@
> +To: roy...@freebsd.org
Please at least Bcc osstest-output. See daily-cron-email-osstest for
an example.
Please also provide information about the chan
Roger Pau Monne writes ("[PATCH v3 7/8] osstest: introduce a script to create a
FreeBSD flight"):
> +FreeBSDDist
> + Path to the folder that contains the FreeBSD install image and
> + the FreeBSD compressed install sets, together with the MANIFEST
> + file that holds the checksums. This is r
On Fri, Jun 23, 2017 at 02:47:37PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 16/20] ts-openstack-tempest:
> Update list of skipped tests"):
> > Signed-off-by: Anthony PERARD
>
> Again, does this not mean we're going to suffer a maintenance burden
> as tempest grows n
On 23/06/17 15:38, Jan Beulich wrote:
> CF reflects the last bit shifted out, i.e. can't possibly be derived
> from the result value.
>
> Signed-off-by: Jan Beulich
Reviewed-by: Andrew Cooper
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://
Roger Pau Monne writes ("[PATCH v3 6/8] osstest: introduce a script to set the
hostflags for FreeBSD jobs"):
> Due to the nature of the FreeBSD install media, which is
> self-generated from the ts-freebsd-build script, the hostflags set to
> FreeBSD jobs are related to the current version under te
So far callers of the libxc interface passed in a domain ID which was
then ignored in the hypervisor. Instead, make the hypervisor honor it
(accepting DOMID_INVALID to obtain original behavior), allowing to
query whether a device can be assigned to a particular domain.
Signed-off-by: Jan Beulich
Roger Pau Monne writes ("[PATCH v3 5/8] osstest: introduce a FreeBSD build
script"):
> The following new helpers are also introduced, that are only used by
> the FreeBSD build script: buildcmd_stamped_logged_root and
> target_cmd_build_root, they behave exactly the same as the non-root
> counterpa
On 23/06/17 15:49, Wei Liu wrote:
> On Fri, Jun 23, 2017 at 01:41:29PM +0100, Andrew Cooper wrote:
>> On 08/06/17 18:11, Wei Liu wrote:
>>> Signed-off-by: Wei Liu
>>> ---
>>> xen/arch/x86/pv/hypercall.c | 36
>>> xen/arch/x86/x86_64/traps.c | 36 --
On Fri, Jun 23, 2017 at 01:41:29PM +0100, Andrew Cooper wrote:
> On 08/06/17 18:11, Wei Liu wrote:
> > Signed-off-by: Wei Liu
> > ---
> > xen/arch/x86/pv/hypercall.c | 36
> > xen/arch/x86/x86_64/traps.c | 36
> > x
On Fri, Jun 23, 2017 at 02:45:12PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 15/20] ts-openstack-tempest: Fix
> tempest invocation"):
> > ./run_tempest.sh is deprecated.
> ...
> > target_cmd($ho, < > set -e
> > -$builddir/tempest/run_tempest.sh --virtual-env -- -
On Fri, Jun 23, 2017 at 03:36:51PM +0100, Julien Grall wrote:
>
>
> On 23/06/17 15:35, Konrad Rzeszutek Wilk wrote:
> > On Fri, Jun 23, 2017 at 02:45:22PM +0100, Andrew Cooper wrote:
> > > On 23/06/17 14:43, Julien Grall wrote:
> > > > Hi,
> > > >
> > > > On 23/06/17 14:33, Andrew Cooper wrote:
Roger Pau Monne writes ("[PATCH v3 4/8] osstest: add a FreeBSD host install
recipe"):
> The installation is performed using the bsdinstall tool, which is part
> of the FreeBSD base system. The installer image is setup with the
> osstest ssh keys and sshd enabled by default, which allows the test
>
On Fri, Jun 23, 2017 at 02:42:53PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 12/20] make-flight: Increase
> dom0_mem for openstack flight"):
> > With 4G for dom0_mem, a host running devstack is using about 1.5G of
> > swap.
>
> Is this going to work properly on 8G hos
>>> On 23.06.17 at 16:26, wrote:
> This will make future cleanup more obviously correct. No functional change.
>
> Signed-off-by: Andrew Cooper
Acked-by: Jan Beulich
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-dev
CF reflects the last bit shifted out, i.e. can't possibly be derived
from the result value.
Signed-off-by: Jan Beulich
---
v2: Fix 64-bit testcase build.
--- a/tools/tests/x86_emulator/test_x86_emulator.c
+++ b/tools/tests/x86_emulator/test_x86_emulator.c
@@ -783,6 +783,31 @@ int main(int argc,
On 23/06/17 15:35, Konrad Rzeszutek Wilk wrote:
On Fri, Jun 23, 2017 at 02:45:22PM +0100, Andrew Cooper wrote:
On 23/06/17 14:43, Julien Grall wrote:
Hi,
On 23/06/17 14:33, Andrew Cooper wrote:
On 23/06/17 14:32, Julien Grall wrote:
Hi Andrew,
I am a bit confused, the title says "PATCH fo
On Fri, Jun 23, 2017 at 02:45:22PM +0100, Andrew Cooper wrote:
> On 23/06/17 14:43, Julien Grall wrote:
> > Hi,
> >
> > On 23/06/17 14:33, Andrew Cooper wrote:
> >> On 23/06/17 14:32, Julien Grall wrote:
> >>> Hi Andrew,
> >>>
> >>> I am a bit confused, the title says "PATCH for-4.9 v3 3/3". I have
On 23/06/17 15:23, Sergej Proskurin wrote:
Hi Julien,
[...]
+static bool get_ttbr_and_gran_64bit(uint64_t *ttbr, unsigned int *gran,
+register_t tcr, enum active_ttbr
ttbrx)
+{
+bool disabled;
+
+if ( ttbrx == TTBR0_ACTIVE )
+{
+/* Norm
On Fri, Jun 23, 2017 at 02:41:59PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 11/20] ts-openstack-deploy:
> Increase fd and memory limits for rabbitmq"):
> > Signed-off-by: Anthony PERARD
>
> Does this not mean that the upstream defaults are wrong ?
That depends on w
On 21/06/17 15:41, Wei Liu wrote:
> Otherwise gcc with -Wunused will complain the return value is not
> used.
>
> Reported-by: Olaf Hering
> Signed-off-by: Wei Liu
Acked-by: Andrew Cooper
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lis
This will make future cleanup more obviously correct. No functional change.
Signed-off-by: Andrew Cooper
---
CC: Jan Beulich
---
xen/arch/x86/mm.c | 56 ---
1 file changed, 29 insertions(+), 27 deletions(-)
diff --git a/xen/arch/x86/mm.c b/x
Roger Pau Monne writes ("[PATCH v3 3/8] osstest: introduce helper to get
per-host tftp prefix"):
> This is used in order to get the per-host tftp prefix, used to store
> the host initrd file.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@
Konrad Rzeszutek Wilk writes ("Re: [Xen-devel] [PATCH RFC] Live migration for
VMs with QEMU backed local storage"):
> On Fri, Jun 23, 2017 at 03:31:16AM -0400, Bruno Alvisio wrote:
> > disks). This are the ones I can think of:
> > - Fully Virtualized HVM: QEMU emulation
> > - blkback
> > - blktap
Anthony PERARD writes ("Re: [OSSTEST PATCH v11 10/20] ts-openstack-deploy:
Switch to Neutron for network"):
> On Fri, Jun 23, 2017 at 02:41:02PM +0100, Ian Jackson wrote:
> > Is this kind of thing going to be common ? If so then it will be a
> > constant maintenance burden in osstest.
>
> No, th
On 23/06/17 15:17, Wei Liu wrote:
> On Fri, Jun 23, 2017 at 01:12:15PM +0100, Andrew Cooper wrote:
>> On 08/06/17 18:11, Wei Liu wrote:
>>> Signed-off-by: Wei Liu
>>> ---
>>> There is no copyright header in the original file. Use the default
>>> one?
>> It should at least gain a basic GPLv2 header
On Fri, Jun 23, 2017 at 01:12:15PM +0100, Andrew Cooper wrote:
> On 08/06/17 18:11, Wei Liu wrote:
> > Signed-off-by: Wei Liu
> > ---
> > There is no copyright header in the original file. Use the default
> > one?
>
> It should at least gain a basic GPLv2 header.
>
> Otherwise, LGTM.
>
I'm goi
Hi Julien,
[...]
>> +static bool get_ttbr_and_gran_64bit(uint64_t *ttbr, unsigned int *gran,
>> +register_t tcr, enum active_ttbr
>> ttbrx)
>> +{
>> +bool disabled;
>> +
>> +if ( ttbrx == TTBR0_ACTIVE )
>> +{
>> +/* Normalize granule size. *
On Fri, Jun 23, 2017 at 03:31:16AM -0400, Bruno Alvisio wrote:
> This patch is the first attempt on adding live migration of instances with
> local
> storage to Xen. This patch just handles very restricted case of fully
> virtualized HVMs. The code uses the "drive-mirror" capability provided by
>
>>> On 23.06.17 at 11:59, wrote:
> --- a/xen/include/asm-x86/hvm/vioapic.h
> +++ b/xen/include/asm-x86/hvm/vioapic.h
> @@ -69,5 +69,6 @@ void vioapic_update_EOI(struct domain *d, u8 vector);
>
> int vioapic_get_mask(struct domain *d, unsigned int gsi);
> int vioapic_get_vector(struct domain *d
On Fri, Jun 23, 2017 at 02:41:02PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v11 10/20] ts-openstack-deploy: Switch
> to Neutron for network"):
> > nova-network is not supported anymore and Neutron is the default.
>
> Is this kind of thing going to be common ? If so then
>>> On 23.06.17 at 11:59, wrote:
> +int vioapic_get_mask(struct domain *d, unsigned int gsi);
> +int vioapic_get_vector(struct domain *d, unsigned int gsi);
Both should have their first parameter const qualified. With that
Reviewed-by: Jan Beulich
Jan
_
On 23/06/17 14:59, Wei Liu wrote:
> On Fri, Jun 23, 2017 at 12:00:35PM +0100, Andrew Cooper wrote:
>> On 08/06/17 18:11, Wei Liu wrote:
>>> Signed-off-by: Wei Liu
>> I'd suggest folding this into the next patch, and putting the hypercall
>> in misc-hypercalls.c
>>
>> Despite its name, this hyperca
On Fri, Jun 23, 2017 at 12:00:35PM +0100, Andrew Cooper wrote:
> On 08/06/17 18:11, Wei Liu wrote:
> > Signed-off-by: Wei Liu
>
> I'd suggest folding this into the next patch, and putting the hypercall
> in misc-hypercalls.c
>
> Despite its name, this hypercall is just setting up state in the vc
Anthony PERARD writes ("[OSSTEST PATCH v11 20/20] Introduce flight for stable
branches of OpenStack"):
> OpenStack have many different repo which should be in sync, so this
> patch should grab the revisions of the stable branch of every OpenStack
> tree. Tempest does not have stable branch and sho
On 23/06/17 14:56, Wei Liu wrote:
> On Fri, Jun 23, 2017 at 01:41:59PM +0100, Andrew Cooper wrote:
>> On 08/06/17 18:11, Wei Liu wrote:
>>> Signed-off-by: Wei Liu
>> Same review suggestions as the previous patch.
>>
> And then your Rb applies?
Yes.
~Andrew
__
On Fri, Jun 23, 2017 at 01:41:59PM +0100, Andrew Cooper wrote:
> On 08/06/17 18:11, Wei Liu wrote:
> > Signed-off-by: Wei Liu
>
> Same review suggestions as the previous patch.
>
And then your Rb applies?
> ~Andrew
___
Xen-devel mailing list
Xen-dev
On Fri, Jun 23, 2017 at 01:31:22PM +0100, Andrew Cooper wrote:
> On 08/06/17 18:11, Wei Liu wrote:
> > Signed-off-by: Wei Liu
> > ---
> > xen/arch/x86/traps.c | 22 ++
> > xen/include/asm-x86/pv/traps.h | 4
> > 2 files changed, 18 insertions(+), 8 deletions(-)
Anthony PERARD writes ("[OSSTEST PATCH v11 19/20] ts-openstack-deploy: Increase
devstack timeout"):
> Signed-off-by: Anthony PERARD
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
Anthony PERARD writes ("[OSSTEST PATCH v11 18/20] ts-logs-capture: Capture
OpenStack logs"):
> + /var/log/openstack/*.log
> + /etc/nova/*
> + /etc/neutron/*
> + /etc/cinder/*
This is fine:
> +
> /home/osstest/b
Anthony PERARD writes ("[OSSTEST PATCH v11 17/20] ts-openstack-deploy: Move
logs to /var/log/openstack"):
> Signed-off-by: Anthony PERARD
Acked-by: Ian Jackson
Ian.
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
On 23/06/17 14:45, Andrew Cooper wrote:
On 23/06/17 14:43, Julien Grall wrote:
Hi,
On 23/06/17 14:33, Andrew Cooper wrote:
On 23/06/17 14:32, Julien Grall wrote:
Hi Andrew,
I am a bit confused, the title says "PATCH for-4.9 v3 3/3". I haven't
been CCed on the first two patches. Does it mea
Anthony PERARD writes ("[OSSTEST PATCH v11 16/20] ts-openstack-tempest: Update
list of skipped tests"):
> Signed-off-by: Anthony PERARD
Again, does this not mean we're going to suffer a maintenance burden
as tempest grows new inapplicable tests ?
Other possibilities that come to my mind: ideall
>>> On 23.06.17 at 14:50, wrote:
> On 08/06/17 18:12, Wei Liu wrote:
>> @@ -1081,8 +1081,8 @@ void do_int3(struct cpu_user_regs *regs)
>> pv_inject_hw_exception(TRAP_int3, X86_EVENT_NO_EC);
>> }
>>
>> -static void reserved_bit_page_fault(
>> -unsigned long addr, struct cpu_user_regs *r
Anthony PERARD writes ("[OSSTEST PATCH v11 15/20] ts-openstack-tempest: Fix
tempest invocation"):
> ./run_tempest.sh is deprecated.
...
> target_cmd($ho, < set -e
> -$builddir/tempest/run_tempest.sh --virtual-env -- --concurrency=2 '$regex'
> +cd $builddir/tempest
> +tempest run --concurrenc
On 23/06/17 14:43, Julien Grall wrote:
> Hi,
>
> On 23/06/17 14:33, Andrew Cooper wrote:
>> On 23/06/17 14:32, Julien Grall wrote:
>>> Hi Andrew,
>>>
>>> I am a bit confused, the title says "PATCH for-4.9 v3 3/3". I haven't
>>> been CCed on the first two patches. Does it mean you are only looking
>
Anthony PERARD writes ("[OSSTEST PATCH v11 14/20] ts-openstack-deploy: Ignore
libvirt-python version and use latest"):
> Devstack is going to try to install a specific version of libvirt-python
> (currently 2.5.0) but this fail with libvirt installed by osstest.
> Remove the requirement and use th
Hi,
On 23/06/17 14:33, Andrew Cooper wrote:
On 23/06/17 14:32, Julien Grall wrote:
Hi Andrew,
I am a bit confused, the title says "PATCH for-4.9 v3 3/3". I haven't
been CCed on the first two patches. Does it mean you are only looking
at this patch to be in 4.9?
Sorry - I messed up the CC lis
Anthony PERARD writes ("[OSSTEST PATCH v11 13/20] ts-openstack-deploy: Apply a
Tempest patch"):
> Signed-off-by: Anthony PERARD
Acked-by: Ian Jackson
> ts-openstack-deploy | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/ts-openstack-deploy b/ts-openstack-deploy
> index 0
Anthony PERARD writes ("[OSSTEST PATCH v11 12/20] make-flight: Increase
dom0_mem for openstack flight"):
> With 4G for dom0_mem, a host running devstack is using about 1.5G of
> swap.
Is this going to work properly on 8G hosts ?
Ian.
___
Xen-devel mai
Anthony PERARD writes ("[OSSTEST PATCH v11 11/20] ts-openstack-deploy: Increase
fd and memory limits for rabbitmq"):
> Signed-off-by: Anthony PERARD
Does this not mean that the upstream defaults are wrong ?
In general I like to see at least an upstream bug report url or number
mentioned in a co
Anthony PERARD writes ("[OSSTEST PATCH v11 10/20] ts-openstack-deploy: Switch
to Neutron for network"):
> nova-network is not supported anymore and Neutron is the default.
Is this kind of thing going to be common ? If so then it will be a
constant maintenance burden in osstest.
Is there some wa
>>> On 08.06.17 at 19:11, wrote:
> +}
> +
> +
> +long compat_callback_op(int cmd, XEN_GUEST_HANDLE(void) arg)
Please avoid double blank lines like above. And preferably, please,
just like you did in earlier patches, use "curr" instead of "v". Then
Acked-by: Jan Beulich
Jan
___
1 - 100 of 208 matches
Mail list logo
