On Thu, May 1, 2025, at 7:28 AM, Lorenz Bauer wrote:
> I'd like to cut down the memory usage of parsing vmlinux BTF in ebpf-go.
> With some upcoming changes the library is sitting at 5MiB for a parse.
> Most of that memory is simply copying the BTF blob into user space.
> By allowing vmlinux BTF to
Hi Shawn,
Gentle ping.
On Thu, Mar 20, 2025 at 2:08 PM Daniel Baluta wrote:
>
> DSP found in i.MX8MP SOC can be used by multiple frameworks in order to
> enable various applications:
> - rproc/rpmsg framework, used to load for example Zephyr samples
> - Soun
ed) ChromeOS kernel trees, so the
> > compatibility impact should be minimal. I will also try to clean up
> > the leftover bits on the crosvm side just to clear things up.
>
> Thanks for your reply, and thanks for clarifying+cleaning it up.
>
[...]
> >> IIRC, in that com
bpf_check(), as it currently exists, will soon be going away to make way
for loadable BPF verifier support. Fixup selftests so they fentry attach
to a more reliable location.
Signed-off-by: Daniel Xu
---
tools/testing/selftests/bpf/progs/exceptions_assert.c | 2 +-
tools/testing/selftests/bpf
> >
> > Which does not seem to reflect reality ...
Please feel free to disregard these features and reuse their bits and
queue indexes; as far as I know, they are not actually enabled
anywhere currently and the corresponding guest patches were only
applied to some (no-longer-used) Chrome
eset controller API (changes merged ->
https://patchwork.kernel.org/project/linux-arm-kernel/cover/20250311085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
contr
With imx8mp-evk board we are now configuring 'dsp' node for rproc usage,
so add rproc specific memory regions.
Also, enable dsp node because it is ready to be used.
Reviewed-by: Ahmad Fatoum
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/
11085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
controller API.
Daniel Baluta (5):
arm64: dts: imx8mp: Use resets property
arm64: dts: imx8mp: Add mu2 root clock
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
e patch matches the spec, so from that
perspective, looks fine:
Reviewed-by: Daniel Verkamp
Since the driver code has been using the wrong order since support for
this message was added in 2010, but there is no support for sending
this message in the current qemu device implementation, I wondered
wh
be writable - tainting kernel\n",
+ pr_warn(
+"%s: ro_after_init failed with %d, data might be writable - tainting kernel\n",
mod->name, ret);
add_taint_module(mod, TAINT_BAD_PAGE, LOCKDEP_STILL_OK);
}
Daniel
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
Enable all relevant nodes to support remoteproc with imx8mp-evk board.
- add rproc specific memory regions
- enable dsp_reserved node
- enable mu2 node
- enable dsp node
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 14
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/freescale
oject/linux-arm-kernel/cover/20250311085812.1296243-1-daniel.bal...@nxp.com/)
- drop patches related to DSP run/stall/reset via syscon
- picked up patch related to using run_stall via reset
controller API.
Daniel Baluta (5):
arm64: dts: imx8mp: Use resets property
arm6
: 0||
1024 -> 2047 : 10748||
Tested-by: Daniel Gomez
Andreas, Petr, Miguel,
Based on the discussion in v7, it seems that all these patches will go through
the Rust tree. Is that correct? What would be missing from t
-862b-fd77552ae...@suse.com/ [1]
> Signed-off-by: Andreas Hindborg
And this is what we signed up for here:
https://lore.kernel.org/all/zspanzx4-5dro...@bombadil.infradead.org/
Acked-by: Daniel Gomez
> ---
> MAINTAINERS | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 4
arch/arm64/boot/dts/freescale/imx8mp.dtsi| 1 +
2 files changed, 5
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
lback to handle resource table parsing and to
> process DSP-specific resource, to determine if waiting is needed.
>
> Update imx_dsp_rproc_start() to handle this condition accordingly.
>
> Signed-off-by: Iuliana Prodan
Reviewed-by: Daniel Baluta
rved_mem_region_to_resource() which is checked for an error.
>
> Signed-off-by: Rob Herring (Arm)
For IMX part:
Reviewed-by: Daniel Baluta
" and then get
> the region's address and size. As this is a standard property, it should
> have common functions for drivers to use. Add new functions to count the
> number of regions and retrieve the region's address as a resource.
>
> Signed-off-by: Rob Herring (Arm)
Reviewed-by: Daniel Baluta
On Mon, Mar 17, 2025 at 5:30 PM Alexander Stein
wrote:
>
> Am Montag, 17. März 2025, 14:33:03 CET schrieb Daniel Baluta:
> > Enable MU2 node and add mu2 root clock.
> > MU2 is used to communicate with DSP core.
> >
> > Reviewed-by: Iuliana Prodan
> > Revi
Enable MU2 node and add mu2 root clock.
MU2 is used to communicate with DSP core.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot
DSP core needs ocram, core and debug clocks.
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
b/arch/arm64/boot/dts
ware) will need to
create a separate dts file (or an overlay).
This change follows the approach taken for other i.MX8 boards
in commit 391a319c81f6d7 ("arm64: dts: imx8-ss-audio: configure dsp node
for rproc usage")
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by:
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
With imx8mp-evk board we are now configuring 'dsp' node for rproc usage,
so add rproc specific memory regions.
Also, enable dsp node because it is ready to be used.
Reviewed-by: Ahmad Fatoum
Reviewed-by: Iuliana Prodan
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
arch/
trol the
DSP")
Reported-by: kernel test robot
Closes:
https://lore.kernel.org/oe-kbuild-all/202503142125.ie33scto-...@intel.com/
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/remoteproc/imx_dsp_rproc.c
b/
https://patchwork.kernel.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(
ff-by: Daniel Baluta
---
.../devicetree/bindings/dsp/fsl,dsp.yaml | 24 ++-
1 file changed, 23 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
b/Documentation/devicetree/bindings/dsp/fsl,dsp.yaml
index ab93ffd3d2e5..b8693e4b4b0d 1
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c
On Mon, Mar 10, 2025 at 5:43 PM Mathieu Poirier
wrote:
>
> Good day,
>
> On Wed, Mar 05, 2025 at 12:00:36PM +0200, Daniel Baluta wrote:
> > DSP on i.MX8MP doesn't have a direct reset line so according to hardware
> > design team in order to handle assert/deassert/re
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch
Add reset ids used for EARC and DSP on i.MX8MP platform.
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
- add correct include in the yaml dts snippet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel Baluta (8):
dt-bindings: reset: audiomix: Add reset ids for EA
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
stall control now, but softreset will
be added in the future when we will convert the softreset functionality
to use reset controller API.
[1]
https://patchwork.kernel.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Reviewed-by: Frank Li
Signed-off-by: Daniel B
07102-6-daniel.bal...@nxp.com/
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/drivers/remot
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 8
1 file
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
drivers/reset
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale
emove 'resets' description as it is a common property
- add correct include in the yaml dts snippet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel B
Add reset ids used for EARC and DSP on i.MX8MP platform.
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
Signed-off-by: Daniel Baluta
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
Hello Philipp,
Thanks for your comments!
> The DAP core reset is mentioned in the commit message. Why is it
> missing here? After reading the discussion in [1], I'd expect both the
> stall and the (core) reset signal to be documented, something like:
There is no reset controller driver for DAP a
On Tue, Feb 25, 2025 at 3:18 PM Philipp Zabel wrote:
>
> On Di, 2025-02-25 at 12:19 +0200, Daniel Baluta wrote:
> > Add reset ids used for EARC and DSP on i.MX8MP platform.
> >
> > Signed-off-by: Daniel Baluta
> > Acked-by: Rob Herring (Arm)
> > Reviewed-b
Current code supports EARC PHY Software Reset and EARC Software Reset
but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits in the
future.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
drivers/reset/reset-imx8mp-audiomix.c
This adds IMX8MP_AUDIOMIX_ prefix to internal macros in order to show
that specific macros are related to audiomix.
Signed-off-by: Daniel Baluta
Reviewed-by: Philipp Zabel
Reviewed-by: Frank Li
Reviewed-by: Peng Fan
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file
07102-6-daniel.bal...@nxp.com/
Signed-off-by: Daniel Baluta
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc/imx_rproc.h | 2 ++
2 files changed, 19 insertions(+), 8 deletions(-)
diff --git a/drivers/remot
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
drivers/reset/reset-imx8mp-audiomix.c | 8
1 file
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account the
active_low configuration option.
Signed-off-by: Daniel Baluta
Reviewed-by: Philipp Zabel
Reviewed-by: Peng Fan
Reviewed-by: Frank Li
---
drivers/reset
Add reset ids used for EARC and DSP on i.MX8MP platform.
Signed-off-by: Daniel Baluta
Acked-by: Rob Herring (Arm)
Reviewed-by: Frank Li
---
include/dt-bindings/reset/imx8mp-reset-audiomix.h | 13 +
1 file changed, 13 insertions(+)
create mode 100644 include/dt-bindings/reset
Add resets property to dsp node in order to be able to control the dsp
run/stall bit from audio block control.
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale
l.org/project/imx/patch/20250212085222.107102-6-daniel.bal...@nxp.com/
Signed-off-by: Daniel Baluta
Reviewed-by: Frank Li
---
.../devicetree/bindings/dsp/fsl,dsp.yaml | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/ds
ppet example
Changes since v1:
(https://lore.kernel.org/imx/20250219030809.GD6537@nxa18884-linux/T/)
- addresed comments received on v1
- picked up R-b and A-b tags
Daniel Baluta (8):
dt-bindings: reset: audiomix: Add reset ids for EARC and DSP
dt-bindings: dsp: fsl,dsp: Add res
"params" => info.params = Some(expect_params(it)),
> _ => panic!(
> "Unknown key \"{}\". Valid keys are: {:?}.",
> key, EXPECTED_KEYS
> @@ -183,28 +333,30 @@ pub(crate) fn module(ts: TokenStream) -> TokenStream {
> let info = ModuleInfo::parse(&mut it);
>
> let mut modinfo = ModInfoBuilder::new(info.name.as_ref());
> -if let Some(author) = info.author {
> -modinfo.emit("author", &author);
> +if let Some(author) = &info.author {
> +modinfo.emit("author", author);
> }
> -if let Some(description) = info.description {
> -modinfo.emit("description", &description);
> +if let Some(description) = &info.description {
> +modinfo.emit("description", description);
> }
> modinfo.emit("license", &info.license);
> -if let Some(aliases) = info.alias {
> +if let Some(aliases) = &info.alias {
> for alias in aliases {
> -modinfo.emit("alias", &alias);
> +modinfo.emit("alias", alias);
> }
> }
> -if let Some(firmware) = info.firmware {
> +if let Some(firmware) = &info.firmware {
> for fw in firmware {
> -modinfo.emit("firmware", &fw);
> +modinfo.emit("firmware", fw);
> }
> }
These seem a bit unrelated?
>
> // Built-in modules also export the `file` modinfo string.
> let file =
> std::env::var("RUST_MODFILE").expect("Unable to fetch RUST_MODFILE
> environmental variable");
> -modinfo.emit_only_builtin("file", &file);
> +modinfo.emit_only_builtin("file", &file, false);
> +
> +modinfo.emit_params(&info);
>
> format!(
> "
> @@ -362,14 +514,17 @@ unsafe fn __exit() {{
> __MOD.assume_init_drop();
> }}
> }}
> -
> {modinfo}
> }}
> }}
> +mod module_parameters {{
> +{params}
> +}}
> ",
> type_ = info.type_,
> name = info.name,
> modinfo = modinfo.buffer,
> +params = modinfo.param_buffer,
> initcall_section = ".initcall6.init"
> )
> .parse()
> diff --git a/samples/rust/rust_minimal.rs b/samples/rust/rust_minimal.rs
> index 4aaf117bf8e3c..d999a77c6eb9a 100644
I wonder if the changes to rust_minimal.rs should be a separate patch.
> --- a/samples/rust/rust_minimal.rs
> +++ b/samples/rust/rust_minimal.rs
> @@ -10,6 +10,12 @@
> author: "Rust for Linux Contributors",
> description: "Rust minimal sample",
> license: "GPL",
> +params: {
> +test_parameter: i64 {
> +default: 1,
> +description: "This parameter has a default of 1",
> +},
> +},
> }
>
> struct RustMinimal {
> @@ -20,6 +26,10 @@ impl kernel::Module for RustMinimal {
> fn init(_module: &'static ThisModule) -> Result {
> pr_info!("Rust minimal sample (init)\n");
> pr_info!("Am I built-in? {}\n", !cfg!(MODULE));
> +pr_info!(
> +"My parameter: {}\n",
> +*module_parameters::test_parameter.get()
> +);
>
> let mut numbers = KVec::new();
> numbers.push(72, GFP_KERNEL)?;
>
> --
> 2.47.0
>
>
>
— Daniel
Hi Andreas,
> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>
> Add the trait `ParseInt` for parsing string representations of integers
> where the string representations are optionally prefixed by a radix
> specifier. Implement the trait for the primitive integer types.
>
> Signed-off-by:
> On 21 Feb 2025, at 13:01, Daniel Almeida wrote:
>
>
>
>> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>>
>> Implement `AsRef` for `[u8]` and `BStr` so these can be used
>> interchangeably for operations on `BStr`.
>>
>> Signed-off-by
quot;)));
> +/// assert_eq!(Some(b_str!("")),
> b_str!("foobar").strip_prefix(b_str!("foobar")));
> +/// ```
This is passing.
> +pub fn strip_prefix(&self, pattern: impl AsRef) -> Option<&BStr> {
> +self.deref()
> +.strip_prefix(pattern.as_ref().deref())
> +.map(Self::from_bytes)
> +}
> }
>
> impl fmt::Display for BStr {
>
> --
> 2.47.0
>
>
Reviewed-by: Daniel Almeida
> On 18 Feb 2025, at 10:00, Andreas Hindborg wrote:
>
> Implement `AsRef` for `[u8]` and `BStr` so these can be used
> interchangeably for operations on `BStr`.
>
> Signed-off-by: Andreas Hindborg
> ---
> rust/kernel/str.rs | 12
> 1 file changed, 12 insertions(+)
>
> diff --git
BStr::from_bytes(&self.0[index])
> +}
> +}
> +
> /// Creates a new [`BStr`] from a string literal.
> ///
> /// `b_str!` converts the supplied string literal to byte string, so non-ASCII
>
> --
> 2.47.0
>
>
>
Reviewed-by: Daniel Almeida
t; +impl PartialEq for BStr {
> +fn eq(&self, other: &Self) -> bool {
> +self.deref().eq(other.deref())
> +}
> +}
> +
> /// Creates a new [`BStr`] from a string literal.
> ///
> /// `b_str!` converts the supplied string literal to byte string, so non-ASCII
>
> --
> 2.47.0
>
>
>
Reviewed-by: Daniel Almeida
y for `rnull`, the Rust
> null block driver [2].
>
```
$ sudo modprobe rust_minimal test_parameter=2
[ 251.384125] rust_minimal: Rust minimal sample (init)
[ 251.384600] rust_minimal: Am I built-in? false
[ 251.385010] rust_minimal: My parameter: 2
```
Tested-by: Daniel Almeida
I
> > + case IMX_RPROC_RESET_CONTROLLER:
> > + priv->reset = devm_reset_control_get_optional_exclusive(dev,
> > NULL);
>
> Is this optional on purpose? There is no mention of it in the commit
> message. Where is this resets property documented in the dt-bindings?
For this particular
On Tue, Feb 18, 2025 at 5:56 PM Frank Li wrote:
>
> On Tue, Feb 18, 2025 at 10:57:09AM +0200, Daniel Baluta wrote:
> > Current code supports EARC PHY Software Reset and EARCSoftware
> > Reset but it is not easily extensible to more reset bits.
> >
> > So,
For EARC and EARC PHY the reset happens when clearing the reset bits.
Refactor assert/deassert function in order to take into account
the active_low configuratin option.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 45 ++-
1 file changed, 23
We can Run/Stall the DSP via audio block control bits found in audiomix.
Implement this functionality using the reset controller and use assert
for Stall and deassert for Run.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 12 ++--
1 file changed, 10 insertions
Use the reset controller API to control the DSP on i.MX8MP. This way
we can have a better control of the resources and avoid using a syscon
to access the audiomix bits.
Signed-off-by: Daniel Baluta
---
drivers/remoteproc/imx_dsp_rproc.c | 25 +
drivers/remoteproc
://patchwork.kernel.org/project/imx/patch/20241210125338.104959-6-daniel.bal...@nxp.com/
Daniel Baluta (5):
reset: imx8mp-audiomix: Add prefix for internal macro
reset: imx8mp-audiomix: Prepare the code for more reset bits
reset: imx8mp-audiomix: Introduce active_low configuration option
reset
Current code supports EARC PHY Software Reset and EARC Software
Reset but it is not easily extensible to more reset bits.
So, refactor the code in order to easily allow more reset bits
in the future.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 53
This adds IMX8MP_AUDIOMIX_ prefix to internal macros
in order to show that specific macros are related to
audiomix.
Signed-off-by: Daniel Baluta
---
drivers/reset/reset-imx8mp-audiomix.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/reset/reset
Test that very high constant map keys are not interpreted as an error
value by the verifier. This would previously fail.
Acked-by: Eduard Zingerman
Signed-off-by: Daniel Xu
---
.../selftests/bpf/progs/verifier_array_access.c | 15 +++
1 file changed, 15 insertions(+)
diff --git
Two fixes for nullness elision. See commits for more details.
=== Changelog ===
Changes from v1:
* Reword commit message in patch 1
* Add tags
Daniel Xu (3):
bpf: verifier: Do not extract constant map keys for irrelevant maps
bpf: selftests: Test constant key extraction on irrelevant maps
Test that very high constant map keys are not interpreted as an error
value by the verifier. This would previously fail.
Signed-off-by: Daniel Xu
---
.../selftests/bpf/progs/verifier_array_access.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/tools/testing/selftests
Two fixes for nullness elision. See commits for more details.
Daniel Xu (3):
bpf: verifier: Do not extract constant map keys for irrelevant maps
bpf: selftests: Test constant key extraction on irrelevant maps
bpf: verifier: Disambiguate get_constant_map_key() errors
kernel/bpf/verifier.c
On Thu, Jan 30, 2025, at 7:41 PM, Daniel Xu wrote:
> Hi Ilya,
>
> On Thu, Jan 30, 2025, at 2:06 AM, Ilya Leoshkevich wrote:
>> On Wed, 2025-01-29 at 10:45 -0700, Daniel Xu wrote:
>>> On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
>>> > Hi Ilya,
Hi Yonghong,
On Thu, Jan 30, 2025 at 10:28:11PM -0800, Yonghong Song wrote:
>
>
>
> On 1/30/25 2:33 PM, Daniel Xu wrote:
> > Since 67ab80a01886 ("selftests/bpf: Prefer static linking for LLVM
> > libraries"), only statically linking test_progs is support
e. If both options are available, static linking
is chosen.
Signed-off-by: Daniel Xu
---
tools/testing/selftests/bpf/Makefile | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/bpf/Makefile
b/tools/testing/selftests/bpf/Makefile
index 67
Hi Ilya,
On Thu, Jan 30, 2025, at 2:06 AM, Ilya Leoshkevich wrote:
> On Wed, 2025-01-29 at 10:45 -0700, Daniel Xu wrote:
>> On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
>> > Hi Ilya,
>> >
>> > On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leosh
On Wed, Jan 29, 2025 at 09:49:12AM -0700, Daniel Xu wrote:
> Hi Ilya,
>
> On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leoshkevich wrote:
> > On Tue, 2025-01-14 at 13:28 -0700, Daniel Xu wrote:
> > > This commit allows progs to elide a null check on statically known
>
Hi Ilya,
On Wed, Jan 29, 2025 at 03:58:54PM +0100, Ilya Leoshkevich wrote:
> On Tue, 2025-01-14 at 13:28 -0700, Daniel Xu wrote:
> > This commit allows progs to elide a null check on statically known
> > map
> > lookup keys. In other words, if the verifier can statically p
On 1/15/25 2:47 AM, Li Zhijian wrote:
Fixes an issue where out-of-tree kselftest builds fail when building
the BPF and bpftools components. The failure occurs because the top-level
Makefile passes a relative srctree path to its sub-Makefiles, which
leads to errors in locating necessary files.
Fo
ointer. So check that we disallow
both.
Signed-off-by: Daniel Xu
---
.../bpf/progs/verifier_array_access.c | 188 ++
1 file changed, 188 insertions(+)
diff --git a/tools/testing/selftests/bpf/progs/verifier_array_access.c
b/tools/testing/selftests/bpf/progs/verifier_arra
.
An additional benefit is the verifier logs are also more precise. For
this particular error, users will enjoy a slightly clearer message. See
included selftest updates for examples.
Acked-by: Eduard Zingerman
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c | 45
d in sync with these changes, as the
verifier is more efficient with this change. Notable, iters.c tests had
to be changed to use a map type that still requires null checks, as it's
exercising verifier tracking logic w.r.t iterators.
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c
elision
* Added test case for when R2 can be both const and non-const
Daniel Xu (5):
bpf: verifier: Add missing newline on verbose() call
bpf: tcp: Mark bpf_load_hdr_opt() arg2 as read-write
bpf: verifier: Refactor helper access type tracking
bpf: verifier: Support eliding map lookup nulln
: Daniel Xu
---
tools/testing/selftests/bpf/veristat.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/bpf/veristat.c
b/tools/testing/selftests/bpf/veristat.c
index 974c808f9321..7d0a9cb753e3 100644
--- a/tools/testing/selftests/bpf/veristat.c
On Thu, Jan 02, 2025 at 06:53:54PM -0800, Eduard Zingerman wrote:
> On Thu, 2024-12-19 at 21:09 -0700, Daniel Xu wrote:
>
> lgtm, but please see a note below.
>
> [...]
>
> > +/* Returns constant key value if possible, else negative error */
> > +static s
Hey,
it looks like this patch is missing in 6.13-rc6 while the driver and DTS
changes got merged. Gentle ping if it's not somewhere else in a queue
for 6.14.
- Daniel
On 02.08.24 17:21, Andrew Davis wrote:
> From: Hari Nagalla
>
> Some K3 platform devices (AM64x, AM62x) have a
ed so ro_after_init is performed
and module is unloaded when this fails? I guess init does not necessary
mean LIVE.
Daniel
>
> --
> Kees Cook
ointer. So check that we disallow
both.
Signed-off-by: Daniel Xu
---
.../bpf/progs/verifier_array_access.c | 168 ++
1 file changed, 168 insertions(+)
diff --git a/tools/testing/selftests/bpf/progs/verifier_array_access.c
b/tools/testing/selftests/bpf/progs/verifier_arra
d in sync with these changes, as the
verifier is more efficient with this change. Notable, iters.c tests had
to be changed to use a map type that still requires null checks, as it's
exercising verifier tracking logic w.r.t iterators.
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c
.
An additional benefit is the verifier logs are also more precise. For
this particular error, users will enjoy a slightly clearer message. See
included selftest updates for examples.
Acked-by: Eduard Zingerman
Signed-off-by: Daniel Xu
---
kernel/bpf/verifier.c | 45
1 - 100 of 6354 matches
Mail list logo
