Hi all,
On 05.12.22 15:41, Ralf Ramsauer wrote:
[Adding Andrej]
Hi Karim,
On 05/12/2022 13:30, Karim Manaouil wrote:
Hi Ralf,
I am trying to boot a Linux guest (based on
configs/x86/linux-x86-demo.c).
I tried to debug and solve this issue for a while but no success so far.
The cell is cre
Hi Jan,
On 27/08/2020 11:33, Jan Kiszka wrote:
On 25.08.20 16:50, Andrej Utz wrote:
Theses patches mostly improve non-functional aspects of the Jailhouse
configuration parser. Logic for unpacking binary data is consolidated
into CStruct class, which all parser classes are inherited from.
To
Just like the array of 4 in the C struct.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse/config_parser.py
index 6ab769ff..a45aa7d7 100644
--- a
This enables more flexibility in input types as long as they provide
binary I/O capabilities.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 59 +++-
tools/jailhouse-cell-linux | 2 +-
tools/jailhouse-config-check | 4 +--
3 files changed, 34
commit
Changes v1->v2:
- reference class variables via class object or class name
- drop implicit PEP8 formatting
- dropped unused patches
Andrej Utz (7):
pyjailhouse: config_parser: store binary format specification in
struct.Struct
pyjailhouse: config_parser: move parsing into c
Improves its handling in the code and slightly increases the overall
performance as well.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 50 +++-
1 file changed, 20 insertions(+), 30 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
slot additions are ignored by
CStruct as they must be constructed by the owning class itself.
For complex parsing the class method `parse` needs to be overridden,
see `CellConfig`.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 169 +++
1 file
This also enables probing for a configuration type.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 99 +++-
tools/jailhouse-cell-linux | 2 +-
tools/jailhouse-config-check | 5 +-
3 files changed, 57 insertions(+), 49 deletions(-)
diff --git a
... and use constructor for initialization only. This separation
provides clarity on how to instantiate config components.
This commit also serves as preparation for following one.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 93 +---
tools
Adds two checks for CPU specification:
- overlap check detects two inmates using the same CPU(s)
- boundary check detects CPU usage outside of what system
config provides
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 14 --
tools/jailhouse-config-check | 28
rom next again. Please fix and resend.
Ack. Will send a v3 soon'ish.
Thanks,
Andrej Utz
Thanks,
Jan
j.kiszka...@gmail.com schrieb am Dienstag, 25. August 2020 um 12:20:10
UTC+2:
On 25.08.20 10:37, contact@gmail.com wrote:
> At the current head of the
... and use constructor for initialization only. This separation
provides clarity on how to instantiate config components.
This commit also serves as preparation for following one.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 93 +---
tools
Just like the array of 4 in the C struct.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse/config_parser.py
index 6ab769ff..a45aa7d7 100644
--- a
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 14 --
tools/jailhouse-config-check | 34 +++---
2 files changed, 43 insertions(+), 5 deletions(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse/config_parser.py
index 2b47d9b6
ropped unused patches
Andrej Utz (7):
pyjailhouse: config_parser: store binary format specification in
struct.Struct
pyjailhouse: config_parser: move parsing into class methods
pyjailhouse: config_parser: consolidate binary parsing into CStruct
class
pyjailhouse: config_parser: use I/O s
This enables more flexibility in input types as long as they provide
binary I/O capabilities.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 59 +++-
tools/jailhouse-config-check | 4 +--
2 files changed, 33 insertions(+), 30 deletions(-)
diff
This also enables probing for a configuration type.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 99 +++-
tools/jailhouse-config-check | 5 +-
2 files changed, 56 insertions(+), 48 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
Improves its handling in the code and slightly increases the overall
performance as well.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 50 +++-
1 file changed, 20 insertions(+), 30 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
slot additions are ignored by
CStruct as they must be constructed by the owning class itself.
For complex parsing the class method `parse` needs to be overridden,
see `CellConfig`.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 169 +++
1 file
Hi all,
On 02/07/2020 18:26, Ralf Ramsauer wrote:
On 02/07/2020 18:17, Jan Kiszka wrote:
On 02.07.20 18:07, Moritz Walker wrote:
Smells like a regression in that branch for non-root Linux. Is that
562b04e51bb5e2f04bf175383080333237067c63? Can you share you kernel
config?
Yes, its 562b0
On 05/07/2020 23:07, Jan Kiszka wrote:
On 30.06.20 08:47, Andrej Utz wrote:
They may not be used right now but will certainly ease the usage of the
parser API in the future.
Besides I already had written them long ago, so it would be a pity to
withehld them.
"withheld"
Ack.
On 05/07/2020 22:57, Jan Kiszka wrote:
On 30.06.20 08:42, Andrej Utz wrote:
The class slots define component fields in a more grounded way.
This greatly simplifies definition of parseable compoments.
The first `__slots__` tuple in each class defines a constant list of
fields and also the
On 05/07/2020 22:55, Jan Kiszka wrote:
On 30.06.20 08:42, Andrej Utz wrote:
Improves its handling in the code and slightly increases the overall
performance as well.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 51 ++--
1 file changed, 20
Hello Jan,
thanks for reviewing. I will address the replies shortly.
On 30/06/2020 21:12, Jan Kiszka wrote:
On 30.06.20 08:42, Andrej Utz wrote:
Improves its handling in the code and slightly increases the overall
performance as well.
Signed-off-by: Andrej Utz
---
pyjailhouse
They may not be used right now but will certainly ease the usage of the
parser API in the future.
Besides I already had written them long ago, so it would be a pity to
withehld them.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 178 ---
1 file
This enables to create Jailhouse configurations without relying on a C
compiler.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 124 +--
1 file changed, 118 insertions(+), 6 deletions(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse
.. since it is able to generate configs now.
Also remove 'Config' suffix from Cell and System class names as the
config module itself declares them as such.
Signed-off-by: Andrej Utz
---
pyjailhouse/{config_parser.py => config.py} | 22 ++---
tools/jailhouse
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse/config_parser.py
index 7b4872e0..a45aa7d7 100644
--- a/pyjailhouse/config_parser.py
+++ b/pyjailhouse/config_parser.py
slot additions are ignored by
CStruct as they must be constructed by the owning class itself.
For complex parsing the class method `parse` needs to be overriden, see
`CellConfig`.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 169 +++
1 file changed
Improves its handling in the code and slightly increases the overall
performance as well.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 51 ++--
1 file changed, 20 insertions(+), 31 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
This enables more flexibility in input types as long as they provide
binary I/O capabilities.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 59 +++-
tools/jailhouse-config-check | 4 +--
2 files changed, 33 insertions(+), 30 deletions(-)
diff
... and use constructor for initialization only. This separation
provides clarity on how to instantiate config components.
This commit also serves as preparation for following one.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 88 +---
tools
This also enables probing for a configuration type.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 99 +++-
tools/jailhouse-config-check | 5 +-
2 files changed, 56 insertions(+), 48 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
Also includes some minor PEP8 formatting.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 14 +--
tools/jailhouse-config-check | 48
2 files changed, 50 insertions(+), 12 deletions(-)
diff --git a/pyjailhouse/config_parser.py b
Just like the array of 4 in the C struct.
Signed-off-by: Andrej Utz
---
pyjailhouse/config_parser.py | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/pyjailhouse/config_parser.py b/pyjailhouse/config_parser.py
index 3f20bc61..7b4872e0 100644
--- a
Signed-off-by: Andrej Utz
---
configs/x86/f2a88xm-hd3.c | 43 +++
1 file changed, 12 insertions(+), 31 deletions(-)
diff --git a/configs/x86/f2a88xm-hd3.c b/configs/x86/f2a88xm-hd3.c
index e5dfd78f..b529e6a6 100644
--- a/configs/x86/f2a88xm-hd3.c
+++ b
Move lines with similar context together and add some comments.
This commit serves also as a preparation for the following commit.
No functional change.
Signed-off-by: Andrej Utz
---
tools/jailhouse-config-create | 80 ---
1 file changed, 36 insertions(+), 44
Fixes generation of unreferenced PCI capabilities inside cell configs
on AMD systems. They occur due to removal of the IOMMU from the PCI
devices list after its capabilities have been collected.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py| 19 +--
tools
Jailhouse.
For now I've appended a patch with a simple workaround for the kernel.
Hope that helps.
[1]:
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/arch/x86/kernel/apic/apic.c?h=v5.4.24#n1716
Thanks,
Andrej Utz
--
You received this message because you are subscr
As in linux/kernel/gcov/gcov.h.
Otherwise the extract tool will access Gcov data using garbage as an
index and segfault.
Signed-off-by: Andrej Utz
---
tools/jailhouse-gcov-extract.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/jailhouse-gcov-extract.c b/tools
Starting from GCC 7.1, __gcov_exit is a new symbol expected to be
implemented when using GCOV.
Signed-off-by: Andrej Utz
---
hypervisor/gcov.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/hypervisor/gcov.c b/hypervisor/gcov.c
index 6055bdd5..ec3a0d9e 100644
--- a/hypervisor/gcov.c
Hi Ralf,
On 15.10.19 18:22, Ralf Ramsauer wrote:
From: Andrej Utz
This replaces the former static port list with actual port regions as listed in
/proc/ioports.
A whitelist selectively allows access to known ports (if present). PCI devices
above 0xcff are permitted as well.
However, not all
figuration.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 60 +++
tools/jailhouse-config-create | 15 ++---
tools/root-cell-config.c.tmpl | 11 +++
3 files changed, 66 insertions(+), 20 deletions(-)
diff --git a/pyjailhouse/sysfs_parser.py b/p
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
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
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
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
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
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
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 +
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
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
code paths for memory region generation are cleaned up
and streamlined.
Andrej Utz (4):
tools: jailhouse-config-create: Rename regions to mem_regions in
preparation for port_regions
pyjailhouse: sysfs_parser: abstract parts of MemRegion into IORegion
pyjailhouse: sysfs_parser: simplify
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
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
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
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
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
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 Ut
'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 deleti
Hi all,
On 12.07.19 17:59, Ralf Ramsauer wrote:
Hi,
On 6/21/19 12:06 AM, Andrej Utz wrote:
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 c
On 02.07.19 16:12, Jan Kiszka wrote:
On 02.07.19 15:58, Andrej Utz wrote:
On some systems the config generator permissively maps huge chunks of
PCI Bus MMIO space. This area needs to be intercepted by the hypervisor,
as parts of ivshmem-net devices may be behind that area.
Just the make
On some systems the config generator permissively maps huge chunks of
PCI Bus MMIO space. This area needs to be intercepted by the hypervisor,
as parts of ivshmem-net devices may be behind that area.
Hence, ignore such regions.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 4
Hello Henning,
On 28/06/2019 10:29, Henning Schild wrote:
> Hey Andrej,
>
> this feature was already proposed and discussed before, but never
> resulted in patches getting merged.
>
> https://groups.google.com/d/topic/jailhouse-dev/BSfMKio91BQ/discussion
>
> I did not look into it yet. But you
This allows us to use the 'iomem' file format parsing more flexible
i.e. for ioports.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 30 ++
1 file changed, 14 insertions(+), 16 deletions(-)
diff --git a/pyjailhouse/sysfs_parser.py b/p
figuration.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 150 ++
tools/jailhouse-config-create | 26 ++
tools/root-cell-config.c.tmpl | 31 ---
3 files changed, 176 insertions(+), 31 deletions(-)
diff --git a/pyjailhouse/sysfs_parser.py b/p
On 18.06.19 18:04, Jan Kiszka wrote:
On 18.06.19 17:55, Andrej Utz wrote:
Hi Jan,
On 07.06.19 09:23, Jan Kiszka wrote:
On 05.06.19 18:17, Andrej Utz wrote:
This replaces the old static port list with actual port regions from
'/proc/ioports'. The static regions from said list ar
Hi Jan,
On 07.06.19 09:23, Jan Kiszka wrote:
On 05.06.19 18:17, Andrej Utz wrote:
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 compab
This refactor allows us amore flexible use of the 'iomem' file format parsing
i.e. for ioports in the following commit.
No functional change.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 30 ++
1 file changed, 14 insertions(+), 16 deletion
figuration.
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 135 ++
tools/jailhouse-config-create | 14 +---
tools/root-cell-config.c.tmpl | 15 ++--
3 files changed, 142 insertions(+), 22 deletions(-)
diff --git a/pyjailhouse/sysfs_parser.py b/p
Second parameter to MemRegion must be its end (inclusive).
Fixes: 5fe206927c05 ("tools: Implement ACPI IVRS table parser")
Signed-off-by: Andrej Utz
---
pyjailhouse/sysfs_parser.py | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/pyjailhouse/sysfs_parser.py b/p
ary will use x2APIC access
methods which will be translated by JH into native xAPIC access.
If we switch to native xAPIC access in apic-demo, would it still need
hypervisor interception?
Also how far is AVIC from being usable?
Thanks,
Andrej Utz
--
You received this message because you
On 27/03/2019 17:50, Henning Schild wrote:
> Am Wed, 27 Mar 2019 17:15:37 +0100
> schrieb Ralf Ramsauer :
>
>> On 3/26/19 6:03 PM, Jan Kiszka wrote:
>>> On 26.03.19 17:41, Ralf Ramsauer wrote:
>>>> Hi Jan,
>>>>
>>>> On 3/25/19 4:03
On 04/04/2019 19:19, Jan Kiszka wrote:
> On 04.04.19 19:02, Andrej Utz wrote:
>> Hello everyone,
>>
>> after recompiling the kernel (minor changes), I noticed a big
>> reduction of mmio regions after regenerating the root cell config with
>> critical hardware l
7;d like to know the rationale behind this.
Thanks,
Andrej Utz
--
You received this message because you are subscribed to the Google Groups
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to jailhouse-dev+unsubscr...@googleg
ement ACPI IVRS table parses")
Signed-off-by: Andrej Utz
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 552c9462..46c95fc2 100644
--- a/p
PU2 back online.
Linux version: 4.19.15-rt11-1-rt-lts-jailhouse+ SMP PREEMPT RT (from
github/siemens/linux)
HyperThreading: disabled in BIOS
Thanks,
Andrej Utz
--
You received this message because you are subscribed to the Google Groups
"Jailhouse" group.
To unsubscribe from this group
77 matches
Mail list logo
