Branch: refs/heads/master
Home: https://github.com/siemens/jailhouse
Commit: 7a3fecfc146052dd154597a0b9e12545ea67afef
https://github.com/siemens/jailhouse/commit/7a3fecfc146052dd154597a0b9e12545ea67afef
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/Makefile
Log Message:
-----------
hypervisor: Makefile: hook-in Wextra
Wextra covers multiple warnings including Wunused-parameter, which
triggers a lot of noise without bringing much benefit for this code.
Completly disable Wunused-parameter and suppress Werror for the warnings
we still want to fix.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 9f96f595f954d267389b67c6f4ad43f0c120033f
https://github.com/siemens/jailhouse/commit/9f96f595f954d267389b67c6f4ad43f0c120033f
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/include/jailhouse/string.h
M hypervisor/include/jailhouse/types.h
M hypervisor/lib.c
Log Message:
-----------
hypervisor: add wordsize dependent size_t type
This solves warnings due to -Wbuiltin-declaration-mismatch
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: b7f3e4f10081e9ed05850e8190e73b9bae1af460
https://github.com/siemens/jailhouse/commit/b7f3e4f10081e9ed05850e8190e73b9bae1af460
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm-common/lib.c
M hypervisor/arch/arm-common/psci.c
M hypervisor/control.c
M hypervisor/include/jailhouse/control.h
M hypervisor/setup.c
Log Message:
-----------
hypervisor, arm-common: provide an explicit uint INVALID_CPU_ID
Substitute the implicit "-1" occurrence for an invalid CPU id with an
unsigned int INVALID_CPU_ID that can be used in all "uint-related"
comparisons.
Signed-off-by: Andrea Bastoni <[email protected]>
[Jan: add extra empty line in control.h]
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 4661a29314a6190517359cb060580e533f42dfbb
https://github.com/siemens/jailhouse/commit/4661a29314a6190517359cb060580e533f42dfbb
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/mmio.c
Log Message:
-----------
hypervisor: mmio: fix Wsign-compare by casting index to u32.
In the branch, index is 0 or positive, and num_mmio_regions is less than
max s32.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 18700dd8bb52935116ba18bb1ec3970b682f8aa6
https://github.com/siemens/jailhouse/commit/18700dd8bb52935116ba18bb1ec3970b682f8aa6
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/printk.c
Log Message:
-----------
hypervisor: printk: Fix Wsign-compare in align()
This is complicated to fix without a cast. A possible alternative is to
cast the char* to an unsigned long "address" type, but that is even less
intuitive. Since p1 > p0, cast the ptrdiff to uint.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: d40176e686498da108d42075d700327f6d18392f
https://github.com/siemens/jailhouse/commit/d40176e686498da108d42075d700327f6d18392f
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M inmates/lib/printk.c
Log Message:
-----------
inmates: lib: fix align() in printk similarly to the hypervisor
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 13769274ae884a7a340b9cc89c455f832ddb2b58
https://github.com/siemens/jailhouse/commit/13769274ae884a7a340b9cc89c455f832ddb2b58
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/pci.c
Log Message:
-----------
hypervisor: Wsign-compare: force the "right" promotion direction
Only part of the problem, Wconversion would complain as well.
Signed-off-by: Andrea Bastoni <[email protected]>
[Jan: move cast to the left side]
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 58842ef0e2c586a910767f5d00dfb47adc7ac2d3
https://github.com/siemens/jailhouse/commit/58842ef0e2c586a910767f5d00dfb47adc7ac2d3
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm-common/paging.c
Log Message:
-----------
arm-common: paging: fix old style declaration (-Wold-style-declaration)
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 6f8a07bbbd8b5c8e975318ecd7382f3e3c9ccfbc
https://github.com/siemens/jailhouse/commit/6f8a07bbbd8b5c8e975318ecd7382f3e3c9ccfbc
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm-common/include/asm/psci.h
Log Message:
-----------
arm-common: provide unsigned int PSCI_INVALID_ADDRESS
Avoid using "-1" to indicate an invalid address.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: ca0e7a0a0e0f3527e353ccaa57582e3ba0e8535d
https://github.com/siemens/jailhouse/commit/ca0e7a0a0e0f3527e353ccaa57582e3ba0e8535d
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm-common/gic-v2.c
M hypervisor/arch/arm-common/gic-v3.c
Log Message:
-----------
arm-common: gic*: use uint instead of int when needed (Wsign-compare)
And similarly to gicv2_handle_irq_target(), rename "i" to "n" to avoid
suggesting that "i" is an integer. For gic-v2, fix it consistently also
in gicv2_read_lr() and gicv2_write_lr().
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: d7fcf4e6b5319e17a21d4bfe5560a13f9ea70f73
https://github.com/siemens/jailhouse/commit/d7fcf4e6b5319e17a21d4bfe5560a13f9ea70f73
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm-common/include/asm/dcaches.h
M hypervisor/arch/arm/mmu_hyp.c
Log Message:
-----------
arm, arm-common: Wsign-compare: arm_dcaches_flush(): take unsigned long as
size
A better overall fix would be to have size always being specified as
size_t.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 58842ffe99b6b476bbab06af2e7c3f5d406a12af
https://github.com/siemens/jailhouse/commit/58842ffe99b6b476bbab06af2e7c3f5d406a12af
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/smmu-v3.c
Log Message:
-----------
arm64: smmu-v3: use uint instead of int when needed (Wsign-compare)
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 315f5cb2c05f8bc7a91ba971a69559a019dd3a1f
https://github.com/siemens/jailhouse/commit/315f5cb2c05f8bc7a91ba971a69559a019dd3a1f
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/smmu-v3.c
Log Message:
-----------
arm64: smmu-v3: rename 'i' -> 'n', 'j' -> 's'
Consistently use 'n' for unsigned int iterators, and use 's' for stream
iterator.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 9a3f1d194487be44774bd92ef2f29a63443028e6
https://github.com/siemens/jailhouse/commit/9a3f1d194487be44774bd92ef2f29a63443028e6
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/ti-pvu.c
Log Message:
-----------
arm64: ti-pvu: fix Wsign-compare by casting size to u64.
No information lost in the cast, size is > 0 there and fits u64.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 3b3673ebd966c4a03efeb9279ef601c16684c23e
https://github.com/siemens/jailhouse/commit/3b3673ebd966c4a03efeb9279ef601c16684c23e
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/smmu.c
Log Message:
-----------
arm64: smmu: Wsign-compare: make iterator uint
Partial fix, Wconversion will complain anyway due to the return int.
Consistently use 'n' for unsigned int iterator.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 958219c056a08bbcbed05cf53e0505ae091025da
https://github.com/siemens/jailhouse/commit/958219c056a08bbcbed05cf53e0505ae091025da
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/paging.c
M hypervisor/arch/x86/svm.c
Log Message:
-----------
x86: fix simple Wsign-compare warnings
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 1e5f64f6e75cffbb2bc9e287bd83868e351cb3ef
https://github.com/siemens/jailhouse/commit/1e5f64f6e75cffbb2bc9e287bd83868e351cb3ef
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/apic.c
Log Message:
-----------
x86: apic: Wsign-compare change to uint and update comparison check
accordingly
Signed-off-by: Andrea Bastoni <[email protected]>
[Jan: fix APIC register loop after making n unsigned]
Signed-off-by: Jan Kiszka <[email protected]>
Commit: eaa4e760d5c0c63fbec00531d335f67ebde64b3d
https://github.com/siemens/jailhouse/commit/eaa4e760d5c0c63fbec00531d335f67ebde64b3d
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/cat.c
Log Message:
-----------
x86: cat: fix Wsign-compare
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 7746f934d8985d038ca9a1b550dea625bd3606e2
https://github.com/siemens/jailhouse/commit/7746f934d8985d038ca9a1b550dea625bd3606e2
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/vtd.c
Log Message:
-----------
x86: vtd: fix Wsign-compare
This is still not 100% clean, since the fix generate wrong
int-conversions on "n" when used as a return value (i.e.,
Wconversion would complain).
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 931e6b69ccdd3422eee33517f67dd26eb4e81563
https://github.com/siemens/jailhouse/commit/931e6b69ccdd3422eee33517f67dd26eb4e81563
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/vcpu.c
Log Message:
-----------
x86: vcpu: force cast to prevent Wsign-compare
The cast is just to prevent fallouts due to the Werror.
rax is really an unsigned value, but int to unsigned promotion is mixed
too many times to be easily fixed.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: ec5673c064a317c58a27adc9fdc64feb33f759bd
https://github.com/siemens/jailhouse/commit/ec5673c064a317c58a27adc9fdc64feb33f759bd
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/x86/control.c
M hypervisor/arch/x86/entry.S
M hypervisor/arch/x86/include/asm/processor.h
Log Message:
-----------
x86: introduce EXCEPTION_NO_ERROR to avoid int to uint comparison on error
frame
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: bd051f287e15dfa254bd45a32eea1f918ace8067
https://github.com/siemens/jailhouse/commit/bd051f287e15dfa254bd45a32eea1f918ace8067
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/Makefile
Log Message:
-----------
hypervisor: Makefile: remove Wno-error from CFLAGS
Re-enable Werror for all the warnings that have been fixed.
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: b9203413465d4d0de429664b97f55180b6e3732e
https://github.com/siemens/jailhouse/commit/b9203413465d4d0de429664b97f55180b6e3732e
Author: Andrea Bastoni <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M configs/arm64/zynqmp-zcu102.c
Log Message:
-----------
configs: arm64: add SMMUv2 configuration for ZCU102
Add SMMU support for: USB, SD, GEM. (See Table 16-3 + 16-11
Ultrascale+ TRM.)
Signed-off-by: Andrea Bastoni <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 856435123457e059327ada14ff559f93f5766b7c
https://github.com/siemens/jailhouse/commit/856435123457e059327ada14ff559f93f5766b7c
Author: Nikhil Devshatwar <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/include/jailhouse/utils.h
Log Message:
-----------
hypervisor: include: Add a BUG() macro
Add a new BUG macro in utils which can be used in places
where you don't expect control to reach.
Whenever this happens, the code will intentionally crash
via a null pointer de reference.
Signed-off-by: Nikhil Devshatwar <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 9af75bf69ab9f20f962e880a3abb43c5768d6a59
https://github.com/siemens/jailhouse/commit/9af75bf69ab9f20f962e880a3abb43c5768d6a59
Author: Nikhil Devshatwar <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/include/asm/traps.h
M hypervisor/arch/arm64/traps.c
Log Message:
-----------
arm64: Report exception link register ELR in the regdump
For an arm64 CPU, when an exception is taken, the PC of the
CPU is copied in the ELR_ELx. This is useful for debugging which
code path caused the exception.
Modify fill_trap_context to populate ELR_EL2 and dump it as part
of the dump_regs
Signed-off-by: Nikhil Devshatwar <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Commit: 097bed0fa5985cd0ee8099f3d24d3b9c38c2b8c7
https://github.com/siemens/jailhouse/commit/097bed0fa5985cd0ee8099f3d24d3b9c38c2b8c7
Author: Nikhil Devshatwar <[email protected]>
Date: 2020-11-08 (Sun, 08 Nov 2020)
Changed paths:
M hypervisor/arch/arm64/include/asm/ti-pvu.h
M hypervisor/arch/arm64/ti-pvu.c
Log Message:
-----------
arm64: pvu: Avoid failure in config_commit
Current PVU iommu implementation ignores possible failures in the
config_commit part. This would allow inconsistent configuration
to run and may introduce unknown bugs.
Solve this by making sure that the pvu_iommu_config_commit never
fails. Catch the errors early in the mapping phase. Use
"free_tlb_count" to track available no of TLBs for chaining.
This can be used to check if any mapping causes it to potentially
use more no of TLBs than that are free. This will ensure that
the allocationg for chaining will not fail.
Change the return type to void for few functions. Add comments to
explain behavior in case of failure. Remove un necessary checks
that would never trigger.
Signed-off-by: Nikhil Devshatwar <[email protected]>
Signed-off-by: Jan Kiszka <[email protected]>
Compare:
https://github.com/siemens/jailhouse/compare/2f52a11d7346...097bed0fa598
--
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 [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/jailhouse-dev/siemens/jailhouse/push/refs/heads/master/2f52a1-097bed%40github.com.
