[PATCH v3 14/14] pyjailhouse: x86: implement pio_regions generator

2019-09-30 Thread Andrej Utz
This replaces the old static port list with actual port regions from '/proc/ioports'. The static regions from said list are kept and override the data in case of region overlap to retain compability. The generated port list is virtually identicall to the old one but eases manual configuration. Sig

[PATCH v3 13/14] pyjailhouse: sysfs_parser: simplify integer formatting for regions in config template

2019-09-30 Thread Andrej Utz
Signed-off-by: Andrej Utz --- pyjailhouse/sysfs_parser.py | 11 +++ tools/root-cell-config.c.tmpl | 6 +++--- 2 files changed, 14 insertions(+), 3 deletions(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_parser.py index 0f91d928..0dcc7475 100644 --- a/pyjailhouse/sysf

[PATCH v3 10/14] pyjailhouse: sysfs_parser: remove parse_iomem_file

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer We don't need it, call IORegionTree parser directly. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_parser.py index 50ca62fc..d2b36876 1

[PATCH v3 12/14] pyjailhouse: sysfs_parser: abstract parts of MemRegion into IORegion

2019-09-30 Thread Andrej Utz
This prepares for the refactor in following commits. Signed-off-by: Andrej Utz --- pyjailhouse/sysfs_parser.py | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_parser.py index 3027f82e..0f91d928 100644 --- a

[PATCH v3 11/14] pyjailhouse: sysfs_parser: move find_regions_by_name to IORegionTree

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer We will reuse that function, move it to IORegionTree. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 42 + 1 file changed, 19 insertions(+), 23 deletions(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_par

[PATCH v3 09/14] pyjailhouse: sysfs_parser: entirely separate IO parsers

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer Everything is in place, we can separate IOMemRegionTree from IORegionTree. Let's give IORegionTree its own constructor. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/pyjailho

Re: [PATCH v3 00/14] pyjailhouse: x86: Implement config generator for port I/O

2019-09-30 Thread Jan Kiszka
On 30.09.19 21:25, Andrej Utz wrote: Hi Jan, On 30.09.19 21:19, Jan Kiszka wrote: On 30.09.19 21:13, Andrej Utz wrote: This patch series eases configuration of port I/O devices for x86 plattforms by generating an initial PIO region list. To sustain previous behavior, most entries are disabled

Re: [PATCH v3 00/14] pyjailhouse: x86: Implement config generator for port I/O

2019-09-30 Thread Andrej Utz
Hi Jan, On 30.09.19 21:19, Jan Kiszka wrote: On 30.09.19 21:13, Andrej Utz wrote: This patch series eases configuration of port I/O devices for x86 plattforms by generating an initial PIO region list. To sustain previous behavior, most entries are disabled (commented out). Only whitelisted devi

Re: [PATCH v3 00/14] pyjailhouse: x86: Implement config generator for port I/O

2019-09-30 Thread Jan Kiszka
On 30.09.19 21:13, Andrej Utz wrote: This patch series eases configuration of port I/O devices for x86 plattforms by generating an initial PIO region list. To sustain previous behavior, most entries are disabled (commented out). Only whitelisted device ports are allowed. This includes the periphe

[PATCH v3 07/14] pyjailhouse: sysfs_parser: move parse_iomem_file to the new parser

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer Move the next part to the new class: the whole file parser. For the moment, this leaves an ugly one-liner in parse_iomem_file, but let's keep it for the moment -- we'll clean that up later. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 50 +

[PATCH v3 05/14] pyjailhouse: sysfs_parser: simplify statement

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer No functional change. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_parser.py index 464b6a80..b0a9bf44 100644 --- a/pyjailhouse/sysfs_parser.py

[PATCH v3 04/14] pyjailhouse: sysfs_parser: minor stylistic fixups

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer The sysfs_parser is written in python and not in C. We can save some parentheses. No functional change. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 29 + 1 file changed, 13 insertions(+), 16 deletions(-) diff --git a/pyjailho

[PATCH v3 00/14] pyjailhouse: x86: Implement config generator for port I/O

2019-09-30 Thread Andrej Utz
This patch series eases configuration of port I/O devices for x86 plattforms by generating an initial PIO region list. To sustain previous behavior, most entries are disabled (commented out). Only whitelisted device ports are allowed. This includes the peripheral PCI port space. Additionally the c

[PATCH v3 08/14] pyjailhouse: sysfs_parser: make regions_split_by_kernel static

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer No need to access our own tree, make this method static. This allows us to fully seperate IORegionTree from IOMemRegionTree soon. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/p

[PATCH v3 03/14] pyjailhouse: sysfs_parser: remove dead code

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer There are no callers of __str__. Remove it. Seems to be a development artifact of earlier versions. Signed-off-by: Ralf Ramsauer --- pyjailhouse/sysfs_parser.py | 11 --- 1 file changed, 11 deletions(-) diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_p

[PATCH v3 02/14] tools: jailhoues-cell-linux: cosmetic fixup

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer Just for the sake of consistency: s/memregion/mem_region/. This is the only spot where we still had memregion instead of mem_region. Signed-off-by: Ralf Ramsauer --- tools/jailhouse-cell-linux | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools

[PATCH v3 06/14] pyjailhouse: sysfs_parser: introduce new class IORegionTree

2019-09-30 Thread Andrej Utz
From: Ralf Ramsauer Do this step by step. First of all, let's create a new routine that is able to parse a line from /proc/iomem or /proc/ioports. Both files share the same layout, so we can use a common parser. Passing the destination type of the entry to the parser allows to share code. Signe

[PATCH v3 01/14] tools: jailhouse-config-create: Rename regions to mem_regions in preparation for port_regions

2019-09-30 Thread Andrej Utz
Signed-off-by: Andrej Utz Signed-off-by: Ralf Ramsauer --- tools/jailhouse-config-create | 10 +- tools/root-cell-config.c.tmpl | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/jailhouse-config-create b/tools/jailhouse-config-create index 55601a6d..cfa7fbad

[siemens/jailhouse] 4c7214: inmates: Add support for %% format

2019-09-30 Thread Andrej
Branch: refs/heads/next Home: https://github.com/siemens/jailhouse Commit: 4c72146d788461c30269aa29377c51712f6c3c57 https://github.com/siemens/jailhouse/commit/4c72146d788461c30269aa29377c51712f6c3c57 Author: Jan Kiszka Date: 2019-09-30 (Mon, 30 Sep 2019) Changed paths:

Re: [PATCH] pyjailhouse: Remove superfluous definition and fix linter warnings

2019-09-30 Thread Jan Kiszka
On 30.09.19 18:43, Andrej Utz wrote: Hi Jan, On 30.09.19 18:37, Jan Kiszka wrote: On 30.09.19 16:52, Andrej Utz wrote: 'vector_size' was set, but not used. No functional change. Fixes: f6fef92ffaba ("pyjailhouse: sysfs_parser: Add more precise length of some extended caps") Signed-off-by:

Re: [PATCH] pyjailhouse: Remove superfluous definition and fix linter warnings

2019-09-30 Thread Andrej Utz
Hi Jan, On 30.09.19 18:37, Jan Kiszka wrote: On 30.09.19 16:52, Andrej Utz wrote: 'vector_size' was set, but not used. No functional change. Fixes: f6fef92ffaba ("pyjailhouse: sysfs_parser: Add more precise length of some extended caps") Signed-off-by: Andrej Utz ---   pyjailhouse/sysfs_pa

[PATCH] inmates: Add support for %% format

2019-09-30 Thread Jan Kiszka
From: Jan Kiszka Needed for warning-free output of the "%" character. Signed-off-by: Jan Kiszka --- inmates/lib/printk.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/inmates/lib/printk.c b/inmates/lib/printk.c index fd157bbf..29cbc3f6 100644 --- a/inmates/lib/printk.c +++ b/inmates/l

Re: [PATCH] pyjailhouse: Remove superfluous definition and fix linter warnings

2019-09-30 Thread Jan Kiszka
On 30.09.19 16:52, Andrej Utz wrote: 'vector_size' was set, but not used. No functional change. Fixes: f6fef92ffaba ("pyjailhouse: sysfs_parser: Add more precise length of some extended caps") Signed-off-by: Andrej Utz --- pyjailhouse/sysfs_parser.py | 9 + 1 file changed, 5 insert

[PATCH] pyjailhouse: Remove superfluous definition and fix linter warnings

2019-09-30 Thread Andrej Utz
'vector_size' was set, but not used. No functional change. Fixes: f6fef92ffaba ("pyjailhouse: sysfs_parser: Add more precise length of some extended caps") Signed-off-by: Andrej Utz --- pyjailhouse/sysfs_parser.py | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/pyj

[siemens/jailhouse] 421840: Documentation: Add specification of IVSHMEM v2 device

2019-09-30 Thread Jan Kiszka
Branch: refs/heads/wip/ivshmem2 Home: https://github.com/siemens/jailhouse Commit: 421840a5eb169c1f6a65ab2c75c6bb3012253086 https://github.com/siemens/jailhouse/commit/421840a5eb169c1f6a65ab2c75c6bb3012253086 Author: Jan Kiszka Date: 2019-09-30 (Mon, 30 Sep 2019) Changed pa