1 | Couple of trivial fixes for rc3... | 1 | This bug seemed worth fixing for 8.0 since we need an rc4 anyway: |
---|---|---|---|
2 | we were using uninitialized data for the guarded bit when | ||
3 | combining stage 1 and stage 2 attrs. | ||
2 | 4 | ||
3 | The following changes since commit 20661b75ea6093f5e59079d00a778a972d6732c5: | 5 | thanks |
6 | -- PMM | ||
4 | 7 | ||
5 | Merge tag 'pull-ppc-20220404' of https://github.com/legoater/qemu into staging (2022-04-04 15:48:55 +0100) | 8 | The following changes since commit 08dede07030973c1053868bc64de7e10bfa02ad6: |
9 | |||
10 | Merge tag 'pull-ppc-20230409' of https://github.com/legoater/qemu into staging (2023-04-10 11:47:52 +0100) | ||
6 | 11 | ||
7 | are available in the Git repository at: | 12 | are available in the Git repository at: |
8 | 13 | ||
9 | https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20220405 | 14 | https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20230410 |
10 | 15 | ||
11 | for you to fetch changes up to 80b952bb694a90f7e530d407b01066894e64a443: | 16 | for you to fetch changes up to 8539dc00552e8ea60420856fc1262c8299bc6308: |
12 | 17 | ||
13 | docs/system/devices/can.rst: correct links to CTU CAN FD IP core documentation. (2022-04-05 09:29:28 +0100) | 18 | target/arm: Copy guarded bit in combine_cacheattrs (2023-04-10 14:31:40 +0100) |
14 | 19 | ||
15 | ---------------------------------------------------------------- | 20 | ---------------------------------------------------------------- |
16 | target-arm queue: | 21 | target-arm: Fix bug where we weren't initializing |
17 | * docs/system/devices/can.rst: correct links to CTU CAN FD IP core documentation. | 22 | guarded bit state when combining S1/S2 attrs |
18 | * xlnx-bbram: hw/nvram: Fix uninitialized Error * | ||
19 | 23 | ||
20 | ---------------------------------------------------------------- | 24 | ---------------------------------------------------------------- |
21 | Pavel Pisa (1): | 25 | Richard Henderson (2): |
22 | docs/system/devices/can.rst: correct links to CTU CAN FD IP core documentation. | 26 | target/arm: PTE bit GP only applies to stage1 |
27 | target/arm: Copy guarded bit in combine_cacheattrs | ||
23 | 28 | ||
24 | Tong Ho (1): | 29 | target/arm/ptw.c | 11 ++++++----- |
25 | xlnx-bbram: hw/nvram: Fix uninitialized Error * | 30 | 1 file changed, 6 insertions(+), 5 deletions(-) |
26 | |||
27 | docs/system/devices/can.rst | 6 +++--- | ||
28 | hw/nvram/xlnx-bbram.c | 2 +- | ||
29 | 2 files changed, 4 insertions(+), 4 deletions(-) | diff view generated by jsdifflib |
1 | From: Pavel Pisa <pisa@cmp.felk.cvut.cz> | 1 | From: Richard Henderson <richard.henderson@linaro.org> |
---|---|---|---|
2 | 2 | ||
3 | Signed-off-by: Pavel Pisa <pisa@cmp.felk.cvut.cz> | 3 | Only perform the extract of GP during the stage1 walk. |
4 | Reviewed-by: Francisco Iglesias <frasse.iglesias@gmail.com> | 4 | |
5 | Message-id: 20220402204523.32643-1-pisa@cmp.felk.cvut.cz | 5 | Reported-by: Peter Maydell <peter.maydell@linaro.org> |
6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> | ||
7 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
8 | Message-id: 20230407185149.3253946-2-richard.henderson@linaro.org | ||
6 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 9 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
7 | --- | 10 | --- |
8 | docs/system/devices/can.rst | 6 +++--- | 11 | target/arm/ptw.c | 10 +++++----- |
9 | 1 file changed, 3 insertions(+), 3 deletions(-) | 12 | 1 file changed, 5 insertions(+), 5 deletions(-) |
10 | 13 | ||
11 | diff --git a/docs/system/devices/can.rst b/docs/system/devices/can.rst | 14 | diff --git a/target/arm/ptw.c b/target/arm/ptw.c |
12 | index XXXXXXX..XXXXXXX 100644 | 15 | index XXXXXXX..XXXXXXX 100644 |
13 | --- a/docs/system/devices/can.rst | 16 | --- a/target/arm/ptw.c |
14 | +++ b/docs/system/devices/can.rst | 17 | +++ b/target/arm/ptw.c |
15 | @@ -XXX,XX +XXX,XX @@ Links to other resources | 18 | @@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_lpae(CPUARMState *env, S1Translate *ptw, |
16 | (5) `GNU/Linux, CAN and CANopen in Real-time Control Applications Slides from LinuxDays 2017 (include updated RTLWS 2015 content) <https://www.linuxdays.cz/2017/video/Pavel_Pisa-CAN_canopen.pdf>`_ | 19 | result->f.attrs.secure = false; |
17 | (6) `Linux SocketCAN utilities <https://github.com/linux-can/can-utils>`_ | 20 | } |
18 | (7) `CTU CAN FD project including core VHDL design, Linux driver, test utilities etc. <https://gitlab.fel.cvut.cz/canbus/ctucanfd_ip_core>`_ | 21 | |
19 | - (8) `CTU CAN FD Core Datasheet Documentation <http://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/Progdokum.pdf>`_ | 22 | - /* When in aarch64 mode, and BTI is enabled, remember GP in the TLB. */ |
20 | - (9) `CTU CAN FD Core System Architecture Documentation <http://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/ctu_can_fd_architecture.pdf>`_ | 23 | - if (aarch64 && cpu_isar_feature(aa64_bti, cpu)) { |
21 | - (10) `CTU CAN FD Driver Documentation <http://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/driver_doc/ctucanfd-driver.html>`_ | 24 | - result->f.guarded = extract64(attrs, 50, 1); /* GP */ |
22 | + (8) `CTU CAN FD Core Datasheet Documentation <http://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/doc/Datasheet.pdf>`_ | 25 | - } |
23 | + (9) `CTU CAN FD Core System Architecture Documentation <http://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/doc/System_Architecture.pdf>`_ | 26 | - |
24 | + (10) `CTU CAN FD Driver Documentation <https://canbus.pages.fel.cvut.cz/ctucanfd_ip_core/doc/linux_driver/build/ctucanfd-driver.html>`_ | 27 | if (regime_is_stage2(mmu_idx)) { |
25 | (11) `Integration with PCIe interfacing for Intel/Altera Cyclone IV based board <https://gitlab.fel.cvut.cz/canbus/pcie-ctu_can_fd>`_ | 28 | result->cacheattrs.is_s2_format = true; |
29 | result->cacheattrs.attrs = extract32(attrs, 2, 4); | ||
30 | @@ -XXX,XX +XXX,XX @@ static bool get_phys_addr_lpae(CPUARMState *env, S1Translate *ptw, | ||
31 | assert(attrindx <= 7); | ||
32 | result->cacheattrs.is_s2_format = false; | ||
33 | result->cacheattrs.attrs = extract64(mair, attrindx * 8, 8); | ||
34 | + | ||
35 | + /* When in aarch64 mode, and BTI is enabled, remember GP in the TLB. */ | ||
36 | + if (aarch64 && cpu_isar_feature(aa64_bti, cpu)) { | ||
37 | + result->f.guarded = extract64(attrs, 50, 1); /* GP */ | ||
38 | + } | ||
39 | } | ||
40 | |||
41 | /* | ||
26 | -- | 42 | -- |
27 | 2.25.1 | 43 | 2.34.1 | diff view generated by jsdifflib |
1 | From: Tong Ho <tong.ho@xilinx.com> | 1 | From: Richard Henderson <richard.henderson@linaro.org> |
---|---|---|---|
2 | 2 | ||
3 | This adds required initialization of Error * variable. | 3 | The guarded bit comes from the stage1 walk. |
4 | 4 | ||
5 | Signed-off-by: Tong Ho <tong.ho@xilinx.com> | 5 | Fixes: Coverity CID 1507929 |
6 | Reviewed-by: Francisco Iglesias <frasse.iglesias@gmail.com> | 6 | Signed-off-by: Richard Henderson <richard.henderson@linaro.org> |
7 | Reviewed-by: Peter Maydell <peter.maydell@linaro.org> | ||
8 | Message-id: 20230407185149.3253946-3-richard.henderson@linaro.org | ||
7 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> | 9 | Signed-off-by: Peter Maydell <peter.maydell@linaro.org> |
8 | --- | 10 | --- |
9 | hw/nvram/xlnx-bbram.c | 2 +- | 11 | target/arm/ptw.c | 1 + |
10 | 1 file changed, 1 insertion(+), 1 deletion(-) | 12 | 1 file changed, 1 insertion(+) |
11 | 13 | ||
12 | diff --git a/hw/nvram/xlnx-bbram.c b/hw/nvram/xlnx-bbram.c | 14 | diff --git a/target/arm/ptw.c b/target/arm/ptw.c |
13 | index XXXXXXX..XXXXXXX 100644 | 15 | index XXXXXXX..XXXXXXX 100644 |
14 | --- a/hw/nvram/xlnx-bbram.c | 16 | --- a/target/arm/ptw.c |
15 | +++ b/hw/nvram/xlnx-bbram.c | 17 | +++ b/target/arm/ptw.c |
16 | @@ -XXX,XX +XXX,XX @@ static bool bbram_pgm_enabled(XlnxBBRam *s) | 18 | @@ -XXX,XX +XXX,XX @@ static ARMCacheAttrs combine_cacheattrs(uint64_t hcr, |
17 | 19 | ||
18 | static void bbram_bdrv_error(XlnxBBRam *s, int rc, gchar *detail) | 20 | assert(!s1.is_s2_format); |
19 | { | 21 | ret.is_s2_format = false; |
20 | - Error *errp; | 22 | + ret.guarded = s1.guarded; |
21 | + Error *errp = NULL; | 23 | |
22 | 24 | if (s1.attrs == 0xf0) { | |
23 | error_setg_errno(&errp, -rc, "%s: BBRAM backstore %s failed.", | 25 | tagged = true; |
24 | blk_name(s->blk), detail); | ||
25 | -- | 26 | -- |
26 | 2.25.1 | 27 | 2.34.1 | diff view generated by jsdifflib |