[Xen-devel] [PATCH 5/5] libxl: Add explicit cast to libxl_psr_cat_set_cbm

2015-12-27 Thread Joshua Otto
From: Chester Lin Fixes Coverity CID 1343299. The call to xc_psr_cat_set_domain_data() expects type xc_psr_cat_type but is provided libxl_psr_cbm_type which is defined in IDL. This change adds an explicit cast to fix the Coverity warning, and tweaks the surrounding code to more closely conform to

[Xen-devel] Coverity tidying

2015-12-27 Thread Joshua Otto
On Mon, Dec 14, 2015 at 11:08:43AM +, Ian Campbell wrote: > On Sat, 2015-12-12 at 17:07 -0500, Joshua Otto wrote: > > On Fri, Dec 11, 2015 at 01:52:41PM +, Ian Campbell wrote: > > > Cool! Just to be clear, you are looking for one project for the 3 of > > > you to > > > work on as a group (v

[Xen-devel] [PATCH 3/5] ns16550: widen an integer constant for Coverity.

2015-12-27 Thread Joshua Otto
From: Harley Armstrong Fix CID 1343302 by widening a constant used with uart_param.reg_shift to avoid overflow for large values of reg_shift. Signed-off-by: Harley Armstrong --- xen/drivers/char/ns16550.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/drivers/char/ns16

[Xen-devel] [PATCH 2/5] libxl: make GC_FREE reachable in libxl_get_scheduler()

2015-12-27 Thread Joshua Otto
Coverity CID 1343309 This patch preserves the multiple error paths in order to avoid meaninglessly assigning the ERROR_FAIL libxl error code to the return variable, which is of type libxl_scheduler. Signed-off-by: Joshua Otto --- tools/libxl/libxl.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[Xen-devel] [PATCH 1/5] libxl: tidy libxl_get_scheduler() according to CODING_STYLE

2015-12-27 Thread Joshua Otto
To more closely follow the guidelines in CODING_STYLE, store the result of the libxc call in the local variable r, and the check the result of the call in a separate statement. Additionally, change the error log statement to more accurately reflect the failure. This is the only functional change

[Xen-devel] [PATCH 4/5] credit: remove pointless local variable initialization.

2015-12-27 Thread Joshua Otto
Coverity CID 1343301 No functional changes. Signed-off-by: Joshua Otto --- xen/common/sched_credit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/common/sched_credit.c b/xen/common/sched_credit.c index 0dce790..02afddf 100644 --- a/xen/common/sched_credit.c +++ b/xen/

Re: [Xen-devel] how to enable kdb for xen

2015-12-27 Thread quizyjones
No, it won't work on neither 4.4 nor 4.6 > Did it work with 4.6 as you asked for? > > >> Date: Fri, 18 Dec 2015 23:58:52 -0500 > >> Subject: Re: [Xen-devel] how to enable kdb for xen > >> From: ufimts...@gmail.com > >> To: quizy_jo...@outlook.com > >> CC: konrad.w...@oracle.com; elena.ufimts...@

Re: [Xen-devel] Hypercalls: ignored output registers?

2015-12-27 Thread Andrew Cooper
On 27/12/2015 03:42, Carl Patenaude Poulin wrote: Hi all, I'm looking at how hypercalls are implemented in Mini-OS and Andrew Cooper's xen-test-framework. In both cases, several registers are defined as outputs, they're bound by input registers, and their outputs are ignored. Is there any reason