[Xen-devel] [linux-3.10 test] 110516: regressions - FAIL

2017-06-17 Thread osstest service owner
flight 110516 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/110516/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 15 guest-localmigrate/x10 fail REGR. vs. 105694 Tests which a

[Xen-devel] [linux-linus test] 110515: tolerable FAIL - PUSHED

2017-06-17 Thread osstest service owner
flight 110515 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/110515/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stop fail blocked in 110427 test-amd64-i386-xl-qemut-win7-amd64 1

[Xen-devel] [linux-4.9 test] 110513: regressions - FAIL

2017-06-17 Thread osstest service owner
flight 110513 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/110513/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-credit2 19 guest-start/debian.repeat fail REGR. vs. 110456 test-amd64-amd64-xl-q

Re: [Xen-devel] [PATCH 42/44] powerpc/cell: use the dma_supported method for ops switching

2017-06-17 Thread Benjamin Herrenschmidt
On Fri, 2017-06-16 at 20:10 +0200, Christoph Hellwig wrote: > Besides removing the last instance of the set_dma_mask method this also > reduced the code duplication. What is your rationale here ? (I have missed patch 0 it seems). dma_supported() was supposed to be pretty much a "const" function s

[Xen-devel] [xen-unstable test] 110510: regressions - FAIL

2017-06-17 Thread osstest service owner
flight 110510 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/110510/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-win7-amd64 15 guest-localmigrate/x10 fail REGR. vs. 110465 build-armhf-

[Xen-devel] [libvirt test] 110512: tolerable all pass - PUSHED

2017-06-17 Thread osstest service owner
flight 110512 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/110512/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 13 saverestore-support-checkfail like 110460 test-armhf-armhf-libvirt-raw 12 saveresto

[Xen-devel] [xen-4.9-testing test] 110499: regressions - trouble: blocked/broken/fail/pass

2017-06-17 Thread osstest service owner
flight 110499 xen-4.9-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/110499/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 4 host-build-prep fail REGR. vs. 110417 test-amd64-amd6

Re: [Xen-devel] [PATCH v7 29/36] x86, drm, fbdev: Do not specify encrypted memory for video mappings

2017-06-17 Thread kbuild test robot
/Tom-Lendacky/x86-Secure-Memory-Encryption-AMD/20170617-210824 config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 reproduce: wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod

Re: [Xen-devel] [PATCH 03/44] dmaengine: ioat: don't use DMA_ERROR_CODE

2017-06-17 Thread Alexander Duyck
On Fri, Jun 16, 2017 at 11:10 AM, Christoph Hellwig wrote: > DMA_ERROR_CODE is not a public API and will go away. Instead properly > unwind based on the loop counter. > > Signed-off-by: Christoph Hellwig > Acked-by: Dave Jiang > Acked-By: Vinod Koul > --- > drivers/dma/ioat/init.c | 24 ++

Re: [Xen-devel] [PATCH v7 32/36] xen/x86: Remove SME feature in PV guests

2017-06-17 Thread Juergen Gross
On 16/06/17 20:55, Tom Lendacky wrote: > Xen does not currently support SME for PV guests. Clear the SME cpu > capability in order to avoid any ambiguity. > > Signed-off-by: Tom Lendacky Reviewed-by: Juergen Gross Juergen > --- > arch/x86/xen/enlighten_pv.c |1 + > 1 file changed, 1 ins

[Xen-devel] [PATCH v2 06/20] rbtree: empty nodes have no color

2017-06-17 Thread Praveen Kumar
Empty nodes have no color. We can make use of this property to simplify the code emitted by the RB_EMPTY_NODE and RB_CLEAR_NODE macros. Also, we can get rid of the rb_init_node function which had been introduced by commit 88d19cf37952 ("timers: Add rb_init_node() to allow for stack allocated rb n

[Xen-devel] [PATCH 20/20] lib/rbtree.c: fix typo in comment of __rb_insert()

2017-06-17 Thread Praveen Kumar
In case 1, it passes down the BLACK color from G to p and u, and maintains the color of n. By doing so, it maintains the black height of the sub-tree. While in the comment, it marks the color of n to BLACK. This is a typo and not consistents with the code. This patch fixs this typo in comment.

[Xen-devel] [PATCH v2 07/20] rbtree: move some implementation details from rbtree.h to rbtree.c

2017-06-17 Thread Praveen Kumar
rbtree users must use the documented APIs to manipulate the tree structure. Low-level helpers to manipulate node colors and parenthood are not part of that API, so move them to lib/rbtree.c Signed-off-by: Michel Lespinasse Cc: Andrea Arcangeli Acked-by: David Woodhouse Cc: Rik van Riel Cc: Pe

[Xen-devel] [PATCH v2 12/20] rbtree: optimize case selection logic in __rb_erase_color()

2017-06-17 Thread Praveen Kumar
In __rb_erase_color(), we have to select one of 3 cases depending on the color on the 'other' node children. If both children are black, we flip a few node colors and iterate. Otherwise, we do either one or two tree rotations, depending on the color of the 'other' child opposite to 'node', and th

[Xen-devel] [PATCH v2 15/20] rbtree: optimize fetching of sibling node

2017-06-17 Thread Praveen Kumar
When looking to fetch a node's sibling, we went through a sequence of: - check if node is the parent's left child - if it is, then fetch the parent's right child This can be replaced with: - fetch the parent's right child as an assumed sibling - check that node is NOT the fetched child This avoid

[Xen-devel] [PATCH v2 10/20] rbtree: low level optimizations in rb_insert_color()

2017-06-17 Thread Praveen Kumar
- Use the newly introduced rb_set_parent_color() function to flip the color of nodes whose parent is already known. - Optimize rb_parent() when the node is known to be red - there is no need to mask out the color in that case. - Flipping gparent's color to red requires us to fetch its rb_parent

[Xen-devel] [PATCH v2 18/20] rbtree: handle 1-child recoloring in rb_erase() instead of rb_erase_color()

2017-06-17 Thread Praveen Kumar
An interesting observation for rb_erase() is that when a node has exactly one child, the node must be black and the child must be red. An interesting consequence is that removing such a node can be done by simply replacing it with its child and making the child black, which we can do efficiently in

[Xen-devel] [PATCH v2 17/20] rbtree: place easiest case first in rb_erase()

2017-06-17 Thread Praveen Kumar
In rb_erase, move the easy case (node to erase has no more than 1 child) first. I feel the code reads easier that way. Signed-off-by: Michel Lespinasse Reviewed-by: Rik van Riel Cc: Peter Zijlstra Cc: Andrea Arcangeli Cc: David Woodhouse Signed-off-by: Andrew Morton Signed-off-by: Linus Torv

[Xen-devel] [PATCH v2 13/20] rbtree: low level optimizations in __rb_erase_color()

2017-06-17 Thread Praveen Kumar
In __rb_erase_color(), we often already have pointers to the nodes being rotated and/or know what their colors must be, so we can generate more efficient code than the generic __rb_rotate_left() and __rb_rotate_right() functions. Also when the current node is red or when flipping the sibling's col

[Xen-devel] [PATCH v2 19/20] rbtree: low level optimizations in rb_erase()

2017-06-17 Thread Praveen Kumar
Various minor optimizations in rb_erase(): - Avoid multiple loading of node->__rb_parent_color when computing parent and color information (possibly not in close sequence, as there might be further branches in the algorithm) - In the 1-child subcase of case 1, copy the __rb_parent_color field f

[Xen-devel] [PATCH v2 16/20] rbtree: add __rb_change_child() helper function

2017-06-17 Thread Praveen Kumar
Add __rb_change_child() as an inline helper function to replace code that would otherwise be duplicated 4 times in the source. No changes to binary size or speed. Signed-off-by: Michel Lespinasse Reviewed-by: Rik van Riel Cc: Peter Zijlstra Cc: Andrea Arcangeli Cc: David Woodhouse Signed-off

[Xen-devel] [PATCH v2 14/20] rbtree: coding style adjustments

2017-06-17 Thread Praveen Kumar
Set comment and indentation style to be consistent with linux coding style and the rest of the file, as suggested by Peter Zijlstra Signed-off-by: Michel Lespinasse Cc: Andrea Arcangeli Acked-by: David Woodhouse Cc: Rik van Riel Cc: Peter Zijlstra Cc: Daniel Santos Cc: Jens Axboe Cc: "Eric

[Xen-devel] [PATCH v2 11/20] rbtree: adjust node color in __rb_erase_color() only when necessary

2017-06-17 Thread Praveen Kumar
In __rb_erase_color(), we were always setting a node to black after exiting the main loop. And in one case, after fixing up the tree to satisfy all rbtree invariants, we were setting the current node to root just to guarantee a loop exit, at which point the root would be set to black. However thi

[Xen-devel] [PATCH v2 09/20] rbtree: adjust root color in rb_insert_color() only when necessary

2017-06-17 Thread Praveen Kumar
The root node of an rbtree must always be black. However, rb_insert_color() only needs to maintain this invariant when it has been broken - that is, when it exits the loop due to the current (red) node being the root. In all other cases (exiting after tree rotations, or exiting due to an existing

[Xen-devel] [PATCH v2 08/20] rbtree: break out of rb_insert_color loop after tree rotation

2017-06-17 Thread Praveen Kumar
It is a well known property of rbtrees that insertion never requires more than two tree rotations. In our implementation, after one loop iteration identified one or two necessary tree rotations, we would iterate and look for more. However at that point the node's parent would always be black, whi

[Xen-devel] [PATCH 01/20] rbtree: add const qualifier to some functions

2017-06-17 Thread Praveen Kumar
The 'rb_first()', 'rb_last()', 'rb_next()' and 'rb_prev()' calls take a pointer to an RB node or RB root. They do not change the pointed objects, so add a 'const' qualifier in order to make life of the users of these functions easier. Indeed, if I have my own constant pointer &const struct my_type

[Xen-devel] [PATCH v2 03/20] rb_tree: reorganize code in rb_erase() for additional changes

2017-06-17 Thread Praveen Kumar
First, move some code around in order to make the next change more obvious. [a...@linux-foundation.org: coding-style fixes] Signed-off-by: Peter Zijlstra Signed-off-by: Wolfram Strepp Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds [Linux commit 16c047add3ceaf0ab882e3e094d1ec904d023

[Xen-devel] [PATCH v3] xen: common: rbtree: ported updates from

2017-06-17 Thread Praveen Kumar
Hi All, The patch imports the changes and updates of the rbtree implementaiton from Linux tree. But since, the only current implementation is with tmem.c, which am not much aware off much and therefore, was unable to test the changes thoroughly. Having said that, I do have plans of adding futher c

[Xen-devel] [PATCH v2 04/20] rb_tree: make clear distinction between two different cases in rb_erase()

2017-06-17 Thread Praveen Kumar
There are two cases when a node, having 2 childs, is erased: 'normal case': the successor is not the right-hand-child of the node to be erased 'special case': the successor is the right-hand child of the node to be erased Here some ascii-art, with following symbols (referring to the code): O: node

[Xen-devel] [PATCH v2 05/20] rb_tree: remove redundant if()-condition in rb_erase()

2017-06-17 Thread Praveen Kumar
Furthermore, notice that the initial checks: if (!node->rb_left) child = node->rb_right; else if (!node->rb_right) child = node->rb_left; else { ... } guarantee that old->rb_righ

[Xen-devel] [PATCH 02/20] lib/rbtree.c: optimize rb_erase()

2017-06-17 Thread Praveen Kumar
Tfour 4 redundant if-conditions in function __rb_erase_color() in lib/rbtree.c are removed. In pseudo-source-code, the structure of the code is as follows: if ((!A || B) && (!C || D)) { . . . } else { if (!C || D) {//if this is true, it implies: (A == true) && (B == false)

[Xen-devel] [distros-debian-stretch test] 71582: regressions - trouble: blocked/broken/fail/pass

2017-06-17 Thread Platform Team regression test user
flight 71582 distros-debian-stretch real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71582/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-amd64-stretch-netboot-pygrub 9 debian-di-install fail REGR. vs. 71538

[Xen-devel] [linux-3.10 test] 110489: regressions - FAIL

2017-06-17 Thread osstest service owner
flight 110489 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/110489/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 15 guest-localmigrate/x10 fail REGR. vs. 105694 test-amd64-i3