[Xen-devel] [PATCH 5/7] xenalyze: disable sched_switch_process

2015-04-27 Thread Olaf Hering
It causes a crash, dump data and return early Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/misc/xenalyze.c | 4 1

[Xen-devel] [PATCH 4/7] xenalyze: handle TRC_TRACE_WRAP_BUFFER

2015-04-27 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/misc/xenalyze.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Xen-devel] [PATCH 0/7] add xenalyze to staging

2015-04-27 Thread Olaf Hering
Having xenalyze in the source tree makes it much easier to keep private debug code in hypervisor and xenalyze in sync. It helped alot while debugging the root cause for commit 607e8494c42397fb249191904066cace6ac9a880. Olaf Olaf Hering (7): xenalyze: add to tools/misc xenalyze: print newline

[Xen-devel] [PATCH 2/7] xenalyze: print newline after unknown hvm events

2015-04-27 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/misc/xenalyze.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [Xen-devel] [PATCH 5/7] xenalyze: disable sched_switch_process

2015-05-03 Thread Olaf Hering
On Sat, May 02, Wei Liu wrote: Ergh... George might have an idea of the bug you met. I'm not in favour of short-circuiting code like this. If this is really necessary please either delete dead code or use #ifdef 0 with proper comment. I will drop that change for now. Olaf

Re: [Xen-devel] [PATCH 1/7] xenalyze: add to tools/misc

2015-04-27 Thread Olaf Hering
On Mon, Apr 27, David Vrabel wrote: On 27/04/15 14:22, Olaf Hering wrote: This merges xenalyze.hg, changeset 150:24308507be1d into tools/misc to have the tool and public/trace.h in one place. Would this not be better in tools/xentrace/xenalyze ? Yes, I forgot that tools/xentrace/ exists

Re: [Xen-devel] [PATCH 5/7] xenalyze: disable sched_switch_process

2015-04-27 Thread Olaf Hering
On Tue, Apr 28, Wei Liu wrote: On Mon, Apr 27, 2015 at 01:22:14PM +, Olaf Hering wrote: It causes a crash, dump data and return early This doesn't look right. You skip all things below. Yes, to avoid the crash when booted with tbuf_size=-1. When I noticed that I did not have the energy

[Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-04-28 Thread Olaf Hering
Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Jim Fehlig jfeh...@suse.com --- src/libxl/libxl_conf.c | 6

[Xen-devel] [PATCH] libxl: support vscsi

2015-04-28 Thread Olaf Hering
devices with persistant names. This patch got very little runtime testing up to now... Signed-off-by: Olaf Hering o...@aepfle.de --- src/libxl/libxl_conf.c | 59 + src/libxl/libxl_conf.h | 1 + src/libxl/libxl_domain.c | 2 +- src/libxl/libxl_driver.c | 167

[Xen-devel] [PATCH 1/2] vns/tls: don't use depricated gnutls functions

2015-04-28 Thread Olaf Hering
...@gmail.com Upstream commit f40d55081667a716312b9a8b6e13835c4074f56b Signed-off-by: Gerd Hoffmann kra...@redhat.com Signed-off-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Olaf Hering o...@aepfle.de --- vnc.c | 65 + 1 file changed

[Xen-devel] [PATCH 0/2] qemu-trad: fix build with recent gnutls

2015-04-28 Thread Olaf Hering
Building qemu-traditional fails with current gnutls: ../libqemu_common.a(vnc.o): In function `vnc_start_tls': /home/abuild/rpmbuild/BUILD/xen-4.6.30745/non-dbg/tools/qemu-xen-traditional-dir/vnc.c:2164: undefined reference to `gnutls_kx_set_priority'

[Xen-devel] [PATCH 2/2] vnc-tls: Fix compilation with newer versions of GNU-TLS

2015-04-28 Thread Olaf Hering
-id: 1350551818-14717-1-git-send-email-andre.przyw...@amd.com Signed-off-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Olaf Hering o...@aepfle.de --- vnc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/vnc.c b/vnc.c index 96f2fba..8ed8a26 100644 --- a/vnc.c

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-05-06 Thread Olaf Hering
On Wed, May 06, Ian Campbell wrote: On Wed, 2015-05-06 at 10:24 +0200, Olaf Hering wrote: The code flow was essentially like this: libxl_device_vfb_init(libxl); switch(libvirt-type) { case SDL: libxl_defbool_set(libxl-sdl.enable, 1); break; case VNC

Re: [Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-05-06 Thread Olaf Hering
On Wed, May 06, Olaf Hering wrote: Since libvirtd runs forever there is very little code to undo most things. But I will see if there is any unload function, did not actively look for such thing. Looking through libxlStateDriver it seems that libxl and libxlu remains as is. Not sure if thats

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-05-06 Thread Olaf Hering
On Fri, May 01, Ian Campbell wrote: Olaf, please can you use gdb to capture the stack trace so we can fix this (and the other issue) properly in libxl instead of just hacking around it in libvirt (which might also be appropriate for compat with old libxl but shouldn't be done without also

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: +'pdev' describes the physical device, preferable in a persistant format. persistent, and please explain when persistent format is. Done. +The backend driver in the pvops kernel is part

Re: [Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: On Tue, Apr 28, 2015 at 06:40:05AM +, Olaf Hering wrote: Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Do

[Xen-devel] [PATCH v2] tools: add configure --with-dumpdir=DIR option

2015-05-06 Thread Olaf Hering
The current base directory /var/xen for domU dumps will be patched to /var/lib/xen by most distros. Provide a configure option to avoid patching the source. If the option is not specified the default remains /var/xen/dump. Please rerun autogen.sh after applying this patch. Signed-off-by: Olaf

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: The first step is to not use anything hardcoded domid in you xenstore paths, i.e. do not make assumption on the domain that runs the backend. Once someone wants that for pvscsi the vscsi=[] syntax has to be adjusted to also recognize another string. Olaf

Re: [Xen-devel] [PATCH v4 4/5] vscsiif.h: add some notes about xenstore layout

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: On Fri, Apr 17, 2015 at 08:30:59AM +, Olaf Hering wrote: + * The backend driver uses a single_host:many_devices notation to manage domU + * devices. Everything is stored in /local/domain/0/backend/vscsi/. The Don't assume backend domid to be 0. Once

Re: [Xen-devel] [PATCH] tools: add conigure --with-dumpdir=DIR option

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: On Thu, Apr 30, 2015 at 02:24:06PM +, Olaf Hering wrote: The current base directory /var/xen for domU dumps will be patched to /var/lib/xen by most distros. Provide a configure option to avoid patching the source. If the option is not specified

[Xen-devel] maintaining frontend state in libxl

2015-05-06 Thread Olaf Hering
While testing 'xl scsi-detach' I noticed that nothing removes the frontend directory from xenstore. In the example below /local/domain/8/device/vscsi/1/vscsi-devs/dev-2 becomes stale. Should xl actively remove also frontend related entries from xenstore, or is that all up to the frontend itself?

Re: [Xen-devel] [PATCH 0/7] tools: remove private Makefile variables

2015-05-06 Thread Olaf Hering
On Wed, May 06, George Dunlap wrote: On Fri, Apr 24, 2015 at 11:25 AM, Olaf Hering o...@aepfle.de wrote: Replace all private variables in Makefiles with automake variables. This series is based on 92ff75384bce7a11e27fbfaf0c531e88dd1ab4c7. Why? Why use private variables? Olaf

[Xen-devel] [PATCH v5 2/5] docs: add vscsi to xenstore-paths.markdown

2015-05-06 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k...@xen.org Cc: Tim Deegan t...@xen.org --- docs/misc/xenstore-paths.markdown | 10 ++ 1 file changed, 10

[Xen-devel] [PATCH v5 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

2015-05-06 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de --- tools/misc/Makefile | 4 + tools/misc/target-create-xen-scsiback.sh | 135 +++ tools/misc/target-delete-xen-scsiback.sh | 41 ++ 3 files changed, 180 insertions(+) diff --git a/tools/misc/Makefile

[Xen-devel] [PATCH v5 3/5] libxl: add support for vscsi

2015-05-06 Thread Olaf Hering
Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb

[Xen-devel] [PATCH v5 0/5] libbxl: add support for pvscsi, iteration 5

2015-05-06 Thread Olaf Hering
with SLES as backend and frontend. This version has been tested with pvops as backend and SLES as frontend. Olaf Hering (5): vscsiif.h: fix WWN notation for p-dev property docs: add vscsi to xenstore-paths.markdown libxl: add support for vscsi vscsiif.h: add some notes about xenstore layout

[Xen-devel] [PATCH v5 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-05-06 Thread Olaf Hering
The pvops kernel expects either naa.WWN:LUN or h:c:t:l in the p-dev property. Add the missing :LUN part to the comment. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k

[Xen-devel] [PATCH v5 4/5] vscsiif.h: add some notes about xenstore layout

2015-05-06 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k...@xen.org Cc: Tim Deegan t...@xen.org --- xen/include/public/io/vscsiif.h | 68

Re: [Xen-devel] [PATCH v2] tools: add configure --with-dumpdir=DIR option

2015-05-07 Thread Olaf Hering
On Thu, May 07, Wei Liu wrote: On Wed, May 06, 2015 at 07:08:44AM +, Olaf Hering wrote: The current base directory /var/xen for domU dumps will be patched to /var/lib/xen by most distros. Provide a configure option to avoid patching the source. If the option is not specified

Re: [Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-05-07 Thread Olaf Hering
On Wed, May 06, Wei Liu wrote: On Wed, May 06, 2015 at 10:58:11AM +0200, Olaf Hering wrote: On Wed, May 06, Olaf Hering wrote: Since libvirtd runs forever there is very little code to undo most things. But I will see if there is any unload function, did not actively look

Re: [Xen-devel] [PATCH 0/7] tools: remove private Makefile variables

2015-05-07 Thread Olaf Hering
On Wed, May 06, George Dunlap wrote: On Fri, Apr 24, 2015 at 11:25 AM, Olaf Hering o...@aepfle.de wrote: Replace all private variables in Makefiles with automake variables. This series is based on 92ff75384bce7a11e27fbfaf0c531e88dd1ab4c7. Why? Thanks for your reply. The rationale comes

Re: [Xen-devel] [PATCH] MAINTAINERS: Remove Frediano Ziglio from HISILICON HIP04 Support

2015-05-07 Thread Olaf Hering
On Thu, May 07, Zoltan Kiss wrote: Yes, he left indeed. M: Zoltan Kiss zoltan.k...@huawei.com What about you? ;-) Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH] libxlu: handle NULL pointer in xlu_cfg_destroy

2015-05-07 Thread Olaf Hering
. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- Not sure if this is correct, or if libvirt should call this function only if cfg

[Xen-devel] [PATCH v2] libxl: include a XLU_Config in _libxlDriverConfig

2015-05-07 Thread Olaf Hering
Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Jim Fehlig jfeh...@suse.com --- v2: - call xlu_cfg_destroy

[Xen-devel] [PATCH v2 7/8] xenalyze: remove traling whitespaces

2015-05-07 Thread Olaf Hering
Result of sed -i 's@[[:blank:]]\+$@@' tools/xentrace/xenalyze/xenalyze.c Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools

[Xen-devel] [PATCH v2 3/8] xenalyze: print newline after unknown hvm events

2015-05-07 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/xentrace/xenalyze/xenalyze.c | 2 +- 1 file changed, 1 insertion(+), 1

[Xen-devel] [PATCH v2 8/8] xenalyze: remove argp_program_version

2015-05-07 Thread Olaf Hering
Since xenalyze is now upstream its Open Source and part of the given release. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com

[Xen-devel] [PATCH v2 2/8] xenalyze: increase NR_CPUS to 256

2015-05-07 Thread Olaf Hering
To match the hypervisor default which was introduced in 9da0c5b63933b9912e3903190601661813954d0d, bump the limit. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc

[Xen-devel] [PATCH v2 4/8] xenalyze: include odd mmio states in default output

2015-05-07 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/xentrace/xenalyze/xenalyze.c | 2 +- 1 file changed, 1 insertion(+), 1

[Xen-devel] [PATCH v2 6/8] xenalyze: handle more events in sched_process

2015-05-07 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/xentrace/xenalyze/xenalyze.c | 73

[Xen-devel] [PATCH v2 0/8] add xenalyze to staging

2015-05-07 Thread Olaf Hering
which touches sched_switch_process - add patch to remove argp_program_version - add patch to bump NR_CPUS Olaf Olaf Hering (8): xenalyze: add to tools/xentrace/xenalyze xenalyze: increase NR_CPUS to 256 xenalyze: print newline after unknown hvm events xenalyze: include odd mmio states

[Xen-devel] [PATCH v2 5/8] xenalyze: handle TRC_TRACE_WRAP_BUFFER

2015-05-07 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/xentrace/xenalyze/xenalyze.c | 2 ++ 1 file changed, 2 insertions(+) diff

Re: [Xen-devel] [PATCH v2] tools: add configure --with-dumpdir=DIR option

2015-05-08 Thread Olaf Hering
On Thu, May 07, Ian Campbell wrote: On Thu, 2015-05-07 at 10:10 +0200, Olaf Hering wrote: On Thu, May 07, Wei Liu wrote: On Wed, May 06, 2015 at 07:08:44AM +, Olaf Hering wrote: The current base directory /var/xen for domU dumps will be patched to /var/lib/xen by most distros

Re: [Xen-devel] [PATCH v2 2/8] xenalyze: increase NR_CPUS to 256

2015-05-08 Thread Olaf Hering
On Thu, May 07, George Dunlap wrote: On Thu, May 7, 2015 at 11:35 AM, Olaf Hering o...@aepfle.de wrote: To match the hypervisor default which was introduced in 9da0c5b63933b9912e3903190601661813954d0d, bump the limit. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack

Re: [Xen-devel] [PATCH v2 6/8] xenalyze: handle more events in sched_process

2015-05-08 Thread Olaf Hering
On Fri, May 08, Wei Liu wrote: On Thu, May 07, 2015 at 10:35:42AM +, Olaf Hering wrote: +void sched_class(struct record_info *ri) +{ +} Huh, this does nothing? Is it because this is only a stub (you haven't got around to implement it) or it is supposed to be like this? Whatever

Re: [Xen-devel] [PATCH v2 7/8] xenalyze: remove traling whitespaces

2015-05-08 Thread Olaf Hering
On Thu, May 07, Andrew Cooper wrote: On 07/05/15 11:35, Olaf Hering wrote: Result of sed -i 's@[[:blank:]]\+$@@' tools/xentrace/xenalyze/xenalyze.c Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell

Re: [Xen-devel] [PATCH v2 1/8] xenalyze: add to tools/xentrace/xenalyze

2015-05-11 Thread Olaf Hering
On Mon, May 11, Ian Campbell wrote: On Mon, 2015-05-11 at 16:15 +0200, Olaf Hering wrote: On Thu, May 07, George Dunlap wrote: On Thu, May 7, 2015 at 11:35 AM, Olaf Hering o...@aepfle.de wrote: tools/xentrace/xenalyze/Makefile |21 + I personally think it would make

[Xen-devel] [PATCH v2 6/7] tools: replace private LIBDIR with automake libdir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl LIBDIR | xargs sed -i 's@LIBDIR@libdir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v2 7/7] tools: replace private DOCDIR with automake docdir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl DOCDIR | xargs sed -i 's@DOCDIR@docdir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v2 0/7] tools: remove private Makefile variables

2015-05-11 Thread Olaf Hering
Replace all private variables in Makefiles with automake variables. This series is based on f48218fd2d35e274ef58caee889aecd6610c8cb6. changes between v1 and v2: - rebase to staging - update description of each patch Olaf Olaf Hering (7): tools: replace private SBINDIR with automake sbindir

[Xen-devel] [PATCH v2 5/7] tools: replace private MANDIR with automake mandir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl MANDIR | xargs sed -i 's@MANDIR@mandir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v2 3/7] tools: replace private PREFIX with automake prefix

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. Only qemu-traditional expects PREFIX in environment, which is provided by buildmakevars2shellvars. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell

Re: [Xen-devel] [PATCH v2 1/8] xenalyze: add to tools/xentrace/xenalyze

2015-05-11 Thread Olaf Hering
On Thu, May 07, George Dunlap wrote: On Thu, May 7, 2015 at 11:35 AM, Olaf Hering o...@aepfle.de wrote: tools/xentrace/xenalyze/Makefile |21 + I personally think it would make more sense to just put the xenalyze directory in tools/. Ok, so shall I resend with a new subdir tools

Re: [Xen-devel] [PATCH v2] tools: add configure --with-dumpdir=DIR option

2015-05-11 Thread Olaf Hering
On Fri, May 08, Ian Campbell wrote: On Fri, 2015-05-08 at 08:54 +0200, Olaf Hering wrote: On Thu, May 07, Ian Campbell wrote: On Thu, 2015-05-07 at 10:10 +0200, Olaf Hering wrote: Does anyone actually use the non-FHS compliant current default of /var/xen/dump? Should we consider

[Xen-devel] [PATCH v2 1/7] tools: replace private SBINDIR with automake sbindir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl SBINDIR | xargs sed -i 's@SBINDIR@sbindir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v2 2/7] tools: replace private BINDIR with automake bindir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl BINDIR | xargs sed -i 's@BINDIR@bindir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v2 4/7] tools: replace private INCLUDEDIR with automake includedir

2015-05-11 Thread Olaf Hering
Reuse automake variable instead of inventing a private variable. The result of this command: git grep -wnl INCLUDEDIR | xargs sed -i 's@INCLUDEDIR@includedir@g' Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell

[Xen-devel] [PATCH v2] tools: add configure --with-dumpdir=DIR option

2015-05-11 Thread Olaf Hering
to refer to the new default location Please rerun autogen.sh after applying this patch. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com

Re: [Xen-devel] [PATCH v5 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

2015-05-13 Thread Olaf Hering
On Wed, May 13, Ian Campbell wrote: On Wed, 2015-05-06 at 13:28 +, Olaf Hering wrote: Just to make them public, not meant for merging: The scripts used during development to create a bunch of SCSI devices in dom0 using the Linux target framework. targetcli3 and rtslib3 is used

Re: [Xen-devel] [PATCH v9 4/4] libxl: Add interface for querying hypervisor about PCI topology

2015-05-15 Thread Olaf Hering
On Mon, May 11, Boris Ostrovsky wrote: +++ b/tools/libxl/xl_cmdimpl.c @@ -5420,12 +5420,15 @@ static void output_numainfo(void) static void output_topologyinfo(void) +int valid_devs; +valid_devs++; +if (valid_devs == 0) This is broken as well, merged as

Re: [Xen-devel] [PATCH v5 3/5] libxl: add support for vscsi

2015-05-15 Thread Olaf Hering
On Fri, May 15, Jürgen Groß wrote: Multi-LUN devices do exist and they are required to be presented as those to the guest. Ok, this means we need the bus concept. I will see if the API for add/remove can be changed that only devices get passed to the libxl. Olaf

[Xen-devel] [PATCH] docs: fix typo in xl.cfg:vfb=

2015-05-15 Thread Olaf Hering
Use singular for option, it refers to vfb= itself. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com --- docs/man/xl.cfg.pod.5 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/man/xl.cfg.pod.5 b

Re: [Xen-devel] [PATCH v5 3/5] libxl: add support for vscsi

2015-05-15 Thread Olaf Hering
On Wed, May 13, Ian Campbell wrote: On Wed, 2015-05-06 at 13:28 +, Olaf Hering wrote: +++ b/docs/man/xl.pod.1 @@ -1328,6 +1328,24 @@ List virtual trusted platform modules for a domain. =back +=head2 PVSCSI DEVICES + +=over 4 + +=item Bscsi-attach Idomain-id Ipdev

Re: [Xen-devel] [PATCH v9 4/4] libxl: Add interface for querying hypervisor about PCI topology

2015-05-15 Thread Olaf Hering
On Fri, May 15, Boris Ostrovsky wrote: I will send a patch but I am curious whether this broke the build for you too? Because I didn't get any warnings. Builds with -O0, but fails without it. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH V3 3/6] libxl: add pvusb API

2015-05-18 Thread Olaf Hering
On Sun, Apr 19, Chunyan Liu wrote: +static int libxl__usbctrl_add_xenstore(libxl__gc *gc, uint32_t domid, + libxl_device_usbctrl *usbctrl) +{ +flexarray_append_pair(back, state, 1); +flexarray_append_pair(front, state, 1); This (and perhaps

Re: [Xen-devel] [PATCH v9 1/4] libxl/libxc: Move libxl_get_cpu_topology()'s hypercall buffer management to libxc

2015-05-13 Thread Olaf Hering
On Mon, May 11, Boris Ostrovsky wrote: xc_cputopoinfo() is not expected to be used on a hot path and therefore hypercall buffer management can be pushed into libxc. This will simplify life for callers. This got just merged and cause a compile error. Looks like gcc is right, if xc_cputopoinfo

Re: [Xen-devel] [PATCH v9 1/4] libxl/libxc: Move libxl_get_cpu_topology()'s hypercall buffer management to libxc

2015-05-13 Thread Olaf Hering
On Wed, May 13, Boris Ostrovsky wrote: Ian, do you want me to resubmit the series or send a fix for this? Not Ian, but: This change is in staging now, so a fix is needed. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v5 3/5] libxl: add support for vscsi

2015-05-13 Thread Olaf Hering
On Wed, May 13, Ian Campbell wrote: You've represented this as a libxl_vscsi_dev which is actually a bus containing an array of devices, which leads to an API where to remove a device you have to provide libxl_device_scsi_remove with a libxl_device_vscsi where the individual devices are

Re: [Xen-devel] [PATCH v5 3/5] libxl: add support for vscsi

2015-05-13 Thread Olaf Hering
On Wed, May 13, Ian Campbell wrote: Just looking at the API functions on this pass. @@ -3628,6 +3643,7 @@ static inline void libxl__update_config_vtpm(libxl__gc *gc, * devices have same identifier. */ #define COMPARE_DEVID(a, b) ((a)-devid == (b)-devid) #define COMPARE_DISK(a, b)

Re: [Xen-devel] wating for backend changes (was Re: [PATCH v3 4/4] libxl: add support for vscsi)

2015-04-14 Thread Olaf Hering
On Fri, Apr 10, Olaf Hering wrote: How is new code supposed to wait for backend changes? Right now there are two APIs for that: - libxl__wait_for_backend loops for a while until it returns an error. - libxl__ev_devstate_wait registers a watch and a timer. In case of pvscsi

Re: [Xen-devel] [PATCH 2/9] raisin: use timestamps for dpkg Version to avoid versions that start with letters

2015-04-15 Thread Olaf Hering
On Wed, Apr 15, Stefano Stabellini wrote: -Version: $(git show --oneline | head -1 | cut -d -f 1) +Version: $(date +%s) Maybe this should be for humans, like 'date +%Y%m%d.%H%M%S'? Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH 1/2] Introduce configure option --with-system-qemu-traditional

2015-04-15 Thread Olaf Hering
On Wed, Apr 15, Stefano Stabellini wrote: tools/configure.ac | 51 +++- tools/libxl/libxl_dm.c | 11 ++- 2 files changed, 39 insertions(+), 23 deletions(-) This change should be mentioned in the INSTALL file. Olaf

[Xen-devel] building xen and libvirt with odd --prefix fails

2015-04-16 Thread Olaf Hering
My xen is configured with --prefix=/odd/path --enable-rpath. My libvirt is configured with env PKG_CONFIG_PATH=/odd/path/share/pkgconfig bash -x autogen.sh --without-xen --with-libxl. Now make in libvirt fails to find xen/xen.h needed by xenconfig/xen_common.c, I think it expects it in

Re: [Xen-devel] building xen and libvirt with odd --prefix fails

2015-04-16 Thread Olaf Hering
On Thu, Apr 16, Olaf Hering wrote: And initially I did not pass the matching --prefix to libvirts configure. Even with matching --prefix libvirt does not look in $prefix/include for xen/xen.h, even with --includedir=$prefix/include. Using env CFLAGS=-I/odd/path/include CPPFLAGS=-I/odd/path

[Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-04-17 Thread Olaf Hering
Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb

[Xen-devel] [PATCH v4 0/5] libbxl: add support for pvscsi, iteration 4

2015-04-17 Thread Olaf Hering
/msg03958.html Most comments are addressed. This version has been tested with SLES as backend and frontend. This version has been tested with pvops as backend and SLES as frontend. Olaf Hering (5): vscsiif.h: fix WWN notation for p-dev property docs: add vscsi to xenstore-paths.markdown

[Xen-devel] [PATCH v4 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

2015-04-17 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de --- tools/misc/Makefile | 4 ++ tools/misc/target-create-xen-scsiback.sh | 94 tools/misc/target-delete-xen-scsiback.sh | 39 + 3 files changed, 137 insertions(+) diff --git a/tools/misc

[Xen-devel] [PATCH v4 2/5] docs: add vscsi to xenstore-paths.markdown

2015-04-17 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k...@xen.org Cc: Tim Deegan t...@xen.org --- docs/misc/xenstore-paths.markdown | 10 ++ 1 file changed, 10

[Xen-devel] [PATCH v4 4/5] vscsiif.h: add some notes about xenstore layout

2015-04-17 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k...@xen.org Cc: Tim Deegan t...@xen.org --- xen/include/public/io/vscsiif.h | 68

[Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-04-17 Thread Olaf Hering
The pvops kernel expects either naa.WWN:LUN or h:c:t:l in the p-dev property. Add the missing :LUN part to the comment. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Campbell ian.campb...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Jan Beulich jbeul...@suse.com Cc: Keir Fraser k

Re: [Xen-devel] [PATCH v4 2/5] docs: add vscsi to xenstore-paths.markdown

2015-04-17 Thread Olaf Hering
On Fri, Apr 17, Olaf Hering wrote: @@ -302,6 +307,10 @@ A virtual keyboard device backend. Described by A virtual network device backend. Described by [xen/include/public/io/netif.h][NETIF] + ~/backend/vscsi/$DOMID/$DEVID/* [] + +A PV SCSI backend. Described in [pvscsi.txt

[Xen-devel] crash in efi_runtime_call

2015-04-17 Thread Olaf Hering
Since booting xen fails on my ProBook unless I specify maxcpus=1 I tried the EFI firmware today. To my surprise it boots and finds all cpus. But once some efi driver in dom0 is loaded xen crashes. The same happens with xen-4.4 as included in SLE12. ... (XEN) Xen call trace: (XEN)

Re: [Xen-devel] crash in efi_runtime_call

2015-04-17 Thread Olaf Hering
On Fri, Apr 17, Konrad Rzeszutek Wilk wrote: The /noexitboot will inhibit Xen from calling ExitBootServices. How is that supposed to be passed to xen.efi? Looks like I have no cmdline interface. Olaf ___ Xen-devel mailing list

[Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-04-17 Thread Olaf Hering
Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Jim Fehlig jfeh...@suse.com --- src/libxl/libxl_conf.c | 6

Re: [Xen-devel] [PATCH] libxl: initialize allocated libxl_device_vfb array

2015-04-17 Thread Olaf Hering
On Fri, Apr 17, Olaf Hering wrote: Its already allocated by calloc, but the init function sets -devid. Scratch that one, libxlMakeVfb already calls libxl_device_vfb_init. Somehow I missed that. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org

[Xen-devel] [PATCH] libxl: initialize allocated libxl_device_vfb array

2015-04-17 Thread Olaf Hering
Its already allocated by calloc, but the init function sets -devid. Just in case anything cares. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Jim Fehlig jfeh...@suse.com --- src/libxl/libxl_conf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/libxl/libxl_conf.c b/src/libxl

[Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-04-17 Thread Olaf Hering
If the domU configu has sdl enabled libvirtd crashes: libvirtd[5158]: libvirtd: libxl.c:343: libxl_defbool_val: Assertion `!libxl_defbool_is_default(db)' failed. Initialize the relevant defbool variables in libxl_device_vfb. Signed-off-by: Olaf Hering o...@aepfle.de Cc: Jim Fehlig jfeh

[Xen-devel] converting gatewaydev= from domU.cfg to libvirt.xml fails

2015-04-17 Thread Olaf Hering
How should this be converted? /etc/init.d/boot.local tap=xentap tunctl -pt ${tap} ip addr add 1.1.1.1/29 dev ${tap} ip link set up dev ${tap} domU.cfg vif=[ 'mac=00:16:3e:13:01:00,ip=1.1.1.2,type=vif,gatewaydev=xentap,script=vif-route' ] The result from convert-xml xen-xl domU,cfg is:

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-04-18 Thread Olaf Hering
On Fri, Apr 17, Jim Fehlig wrote: On 04/17/2015 11:19 AM, Olaf Hering wrote: +libxl_defbool_set(x_vfb-vnc.enable, 0); Not shown here, but just before the switch is libxl_device_vfb_init(x_vfb); which IIUC (looking at the impl in $xensrc/tools/libxl/_libxl_types.c

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-04-18 Thread Olaf Hering
On Fri, Apr 17, Jim Fehlig wrote: On 04/17/2015 11:59 AM, Olaf Hering wrote: On Fri, Apr 17, Olaf Hering wrote: If the domU configu has sdl enabled libvirtd crashes: libvirtd[5158]: libvirtd: libxl.c:343: libxl_defbool_val: Assertion `!libxl_defbool_is_default(db)' failed. Initialize

Re: [Xen-devel] converting gatewaydev= from domU.cfg to libvirt.xml fails

2015-04-18 Thread Olaf Hering
On Fri, Apr 17, Jim Fehlig wrote: On 04/17/2015 12:50 PM, Olaf Hering wrote: How should this be converted? /etc/init.d/boot.local tap=xentap tunctl -pt ${tap} ip addr add 1.1.1.1/29 dev ${tap} ip link set up dev ${tap} domU.cfg vif=[ 'mac=00:16:3e:13:01:00,ip=1.1.1.2,type=vif

Re: [Xen-devel] sudden hang on boot on AMD Notebook

2015-04-07 Thread Olaf Hering
On Mon, Mar 30, Olaf Hering wrote: Just last week I was running staging to test pvscsi on this HP ProBook 6555b, since around Wednesday I think. But todays staging fails to boot. So I started a bisect in xen/, but every attempt to boot xen.gz results in a hang at the same place. Even going

Re: [Xen-devel] [PATCH] hvmloader: fix building error

2015-04-08 Thread Olaf Hering
On Wed, Apr 08, Wen Congyang wrote: On 04/08/2015 03:21 PM, Olaf Hering wrote: On Wed, Apr 08, Wen Congyang wrote: commit b9245b75 introduces a building error: make[1]: Entering directory `/root/work/xen/tools/firmware/hvmloader' gcc -O1 -fno-omit-frame-pointer -m32 -march=i686 -g

Re: [Xen-devel] sudden hang on boot on AMD Notebook

2015-04-08 Thread Olaf Hering
On Tue, Apr 07, Konrad Rzeszutek Wilk wrote: You might also want to use 'sync_console'. This does not give any extra info in the case of the reported hang. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] ENC:[Xen-users] Failed to insert 'xen_evtchn': No such device

2015-04-02 Thread Olaf Hering
On Thu, Apr 02, Carlos Gustavo Ramirez Rodriguez wrote: systemctl enable xen-qemu-dom0-disk-backend.service systemctl enable xen-init-dom0.service systemctl enable xenconsoled.service These MAGIC lines did the trick! Thanks for the help, this solved the First Problem, BUT, if you know those

Re: [Xen-devel] [PATCH v4 0/9] add xenalyze to staging

2015-06-09 Thread Olaf Hering
On Wed, Jun 03, Julien Grall wrote: xentrace is not working as we don't have the infrastructure for ARM in Xen. Why is that? After a very quick look __trace_var is available unconditionally. Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v4 7/9] xenalyze: handle more events in sched_process

2015-06-09 Thread Olaf Hering
On Tue, Jun 02, George Dunlap wrote: On 05/23/2015 09:24 AM, Olaf Hering wrote: void sched_process(struct pcpu_info *p) +fprintf(warn, %s: event:%x (min:%x sub:%x main:%x)\n, +__func__, ri-event, ri-evt.minor, ri-evt.sub, ri-evt.main

[Xen-devel] [PATCH v5 7/8] xenalyze: remove trailing whitespaces

2015-06-09 Thread Olaf Hering
Result of sed -i 's@[[:blank:]]\+$@@' tools/xentrace/xenalyze.c Signed-off-by: Olaf Hering o...@aepfle.de Acked-by: George Dunlap george.dun...@eu.citrix.com Acked-by: Wei Liu wei.l...@citrix.com Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc

[Xen-devel] [PATCH v5 6/8] xenalyze: handle more events in sched_process

2015-06-09 Thread Olaf Hering
Signed-off-by: Olaf Hering o...@aepfle.de Cc: Ian Jackson ian.jack...@eu.citrix.com Cc: Stefano Stabellini stefano.stabell...@eu.citrix.com Cc: Ian Campbell ian.campb...@citrix.com Cc: Wei Liu wei.l...@citrix.com --- tools/xentrace/xenalyze.c | 73

<    1   2   3   4   5   6   7   8   9   >