vcswatch reports that
this package seems to have a new changelog entry (version
1.5.1-1~0, distribution
UNRELEASED) and new commits
in its VCS. You should consider whether it's time to make
an upload.
Here are the relevant commit messages:
commit 655f9f9094632a8f7735bbb6a96d7d884eea0fea
Author: Vagrant Cascadian <vagrant@debian.org>
Date: Mon Aug 5 15:52:28 2024 -0700
debian/changelog: Update to 1.5.1.
commit 8fe5af2bc066b4f845185a93636dee8e2aa9eefa
Merge: 4f7a154 43cace6
Author: Vagrant Cascadian <vagrant@debian.org>
Date: Mon Aug 5 14:25:43 2024 -0700
Merge tag 'v1.5.1' into debian/latest
OpenSBI v1.5.1
commit 43cace6c3671e5172d0df0a8963e552bb04b7b20
Author: Carlos López <carlos.lopezr4096@gmail.com>
Date: Thu Aug 1 14:58:52 2024 +0200
lib: sbi: check result of pmp_get() in is_pmp_entry_mapped()
pmp_get() may return an error if the given entry, given by the caller
of is_pmp_entry_mapped(), is invalid. This results in the output
parameters for pmp_get() being uninitialized. To avoid using garbage
values, check the result and return early if necessary.
This issue is not being hit because at the moment
is_pmp_entry_mapped() is only being called from a single site with a
valid hardcoded value.
Signed-off-by: Carlos López <carlos.lopezr4096@gmail.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
commit cb0f4757fc2904cbf579e34feba891a5878625c5
Author: Carlos López <carlos.lopezr4096@gmail.com>
Date: Thu Aug 1 14:48:11 2024 +0200
lib: sbi: fwft: fix incorrect size passed to sbi_zalloc()
The fwt_hart_state struct inciludes a flexible array member, so its
allocation size will be that of the struct itself, plus that of each
of the members in the array. When calculating this size, instead of
taking the size of the struct, the size of a pointer to it was taken,
which is incorrect. Luckily, this happenned to not produce memory
corruption because the size of the non-flexible members of the struct
is the same as the size of a pointer.
Signed-off-by: Carlos López <carlos.lopezr4096@gmail.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
commit baf6a75e3f967aa8b65a20963bd9ae21975ba75a
Author: Carlos López <carlos.lopezr4096@gmail.com>
Date: Thu Aug 1 14:27:42 2024 +0200
lib: sbi: dbtr: fix potential NULL pointer dereferences
In several dbtr functions, we first check that the dbtr trigger is not
NULL and that its state is what we expect. However, it only makes
sense to perform the second check if the dbtr trigger is not NULL.
Othwerwise we will dereference a NULL pointer. Thus, change the
condition so that it shortcuts to the first check if necessary.
Signed-off-by: Carlos López <carlos.lopezr4096@gmail.com>
Reviewed-By: Anup Patel <anup@brainfault.org>
commit 778949eeb8f36e01c37cb3ae6d82b0660a1bf6f7
Author: Eric Lin <eric.lin@sifive.com>
Date: Tue Jul 30 17:30:22 2024 +0800
include: Adjust Sscofpmf mhpmevent mask for upper 8 bits
Currently, OpenSBI reserves the upper 16 bits in mhpmevent for
the Sscofpmf extension.
However, according to the Sscofpmf extension specification[1],
it only defines the upper 8 bits in mhpmevent for privilege mode
inhibit and counter overflow disable. Other bits are defined by
the platform for event selection.
Since vendors might define raw event encoding exceeding 48 bits in
mhpmevent, we should adjust the MHPMEVENT_SSCOF_MASK to support it.
Link: https://github.com/riscvarchive/riscv-count-overflow [1]
Signed-off-by: Eric Lin <eric.lin@sifive.com>
Reviewed-by: Xiang W <wxjstz@126.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
commit 65a3938fadc537af178c84a349ade31ec6a70846
Author: Samuel Holland <samuel.holland@sifive.com>
Date: Wed Jul 10 07:26:06 2024 -0700
lib: sbi_hsm: Save/restore menvcfg only when it exists
Attempting to access the menvcfg CSR raises an illegal instruction
exception on hardware which implements Sm1p11 or older.
Fixes: e9ee9678ba50 ("lib: sbi: fwft: add support for SBI_FWFT_PTE_AD_HW_UPDATING")
Signed-off-by: Samuel Holland <samuel.holland@sifive.com>
Reviewed-by: Anup Patel <anup@brainfault.org>