Re: [PATCH v3 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP

2023-10-10 Thread Alexander Stein
Hi Iuliana, Am Dienstag, 10. Oktober 2023, 11:09:29 CEST schrieb Iuliana Prodan (OSS): > From: Iuliana Prodan > > Add the reserve-memory nodes used by DSP when the rpmsg > feature is enabled. > > Signed-off-by: Iuliana Prodan > --- > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16

Re: [PATCH 4/5] kbuild: unify vdso_install rules

2023-10-10 Thread Guo Ren
On Mon, Oct 9, 2023 at 8:42 PM Masahiro Yamada wrote: > > Currently, there is no standard implementation for vdso_install, > leading to various issues: > > 1. Code duplication > > Many architectures duplicate similar code just for copying files > to the install destination. > > Some

Re: [PATCH] module: Add CONFIG_MODULE_LOAD_IN_SEQUENCE option

2023-10-10 Thread Luis Chamberlain
Please find a good email client to reply to patches. On Wed, Oct 11, 2023 at 01:57:58AM +, Joey Jiao (QUIC) wrote: > Hi Luis, > > > How is ignoring an error ensuring ordering? > The change is just to disable the schedule_work. That's different and can be made clearer. Try: if

RE: [PATCH] module: Add CONFIG_MODULE_LOAD_IN_SEQUENCE option

2023-10-10 Thread Joey Jiao (QUIC)
Hi Luis, > How is ignoring an error ensuring ordering? It's not ignoring an error ensuring ordering. I think It's because `schedule_work(_free_wq);` changes when the free_wq will happen. If sometimes freeing init for module A happens before loading another module B, that memory is freed. B

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Huang, Kai
On Tue, 2023-10-10 at 11:49 -0500, Haitao Huang wrote: > > > > This patch adds SGX_ENCL_NO_MEMORY.  I guess we can use it for virtual  > > EPC too? > > > > That flag is set for enclaves, do you mean we set similar flag in vepc  > struct? Yes.

Re: [PATCH v1] Ftrace: make sched_wakeup can focus on the target process

2023-10-10 Thread kernel test robot
Hi Jinyu, kernel test robot noticed the following build warnings: [auto build test WARNING on linus/master] [also build test WARNING on rostedt-trace/for-next v6.6-rc5 next-20231010] [cannot apply to rostedt-trace/for-next-urgent] [If your patch is applied to the wrong git tree, kindly drop us

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Huang, Kai
On Tue, 2023-10-10 at 11:49 -0500, Haitao Huang wrote: > On Mon, 09 Oct 2023 20:34:29 -0500, Huang, Kai wrote: > > > On Tue, 2023-10-10 at 00:50 +, Huang, Kai wrote: > > > On Mon, 2023-10-09 at 17:23 -0700, Sean Christopherson wrote: > > > > On Mon, Oct 09, 2023, Kai Huang wrote: > > > > >

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Huang, Kai
On Tue, 2023-10-10 at 12:05 -0500, Haitao Huang wrote: > On Mon, 09 Oct 2023 21:12:27 -0500, Huang, Kai wrote: > > > > > > > > > > > > > > Later the hosting process could migrated/reassigned to another > > > cgroup? > > > > > What to do when the new cgroup is OOM? > > > > > > > > > > > > >

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Sean Christopherson
On Tue, Oct 10, 2023, Haitao Huang wrote: > On Mon, 09 Oct 2023 21:23:12 -0500, Huang, Kai wrote: > > > On Mon, 2023-10-09 at 20:42 -0500, Haitao Huang wrote: > > > Hi Sean > > > > > > On Mon, 09 Oct 2023 19:23:04 -0500, Sean Christopherson > > > wrote: > > > > I can see userspace wanting to

Re: [PATCH] KVM: deprecate KVM_WERROR in favor of general WERROR

2023-10-10 Thread Sean Christopherson
On Tue, Oct 10, 2023, Jakub Kicinski wrote: > On Tue, 10 Oct 2023 11:04:18 +0300 Jani Nikula wrote: > > > If you do invest in build testing automation, why can't your automation > > > count warnings rather than depend on WERROR? I don't understand. > > > > Because having both CI and the

Re: [PATCH v2 0/5] params: harden string ops and allocatio ops

2023-10-10 Thread Luis Chamberlain
On Mon, Oct 02, 2023 at 09:57:59AM -0700, Kees Cook wrote: > On Mon, Oct 02, 2023 at 03:48:51PM +0300, Andy Shevchenko wrote: > > A couple of patches are for get the string ops, used in the module, > > slightly harden. On top a few cleanups. > > > > Since the main part is rather hardening, I

Re: [PATCH] module: Add CONFIG_MODULE_LOAD_IN_SEQUENCE option

2023-10-10 Thread Luis Chamberlain
On Mon, Oct 09, 2023 at 10:26:35AM +0530, Joey Jiao wrote: > When modprobe cmds are executed one by one, the final loaded modules > are not in fixed sequence as expected. > > Add the option to make sure modules are in fixed sequence across reboot. > > Signed-off-by: Joey Jiao > --- >

Re: [PATCH v3 0/7] sysctl: Remove sentinel elements from arch

2023-10-10 Thread Luis Chamberlain
On Mon, Oct 02, 2023 at 01:30:35PM +0200, Joel Granados via B4 Relay wrote: > V3: > * Removed the ia64 patch to avoid conflicts with the ia64 removal > * Rebased onto v6.6-rc4 > * Kept/added the trailing comma for the ctl_table arrays. This was a comment > that we received "drivers/*" patch set.

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Haitao Huang
On Mon, 09 Oct 2023 21:12:27 -0500, Huang, Kai wrote: > > > > > Later the hosting process could migrated/reassigned to another cgroup? > > What to do when the new cgroup is OOM? > > > > You addressed in the documentation, no? > > +Migration > +- > + > +Once an EPC page is charged

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Haitao Huang
On Mon, 09 Oct 2023 20:34:29 -0500, Huang, Kai wrote: On Tue, 2023-10-10 at 00:50 +, Huang, Kai wrote: On Mon, 2023-10-09 at 17:23 -0700, Sean Christopherson wrote: > On Mon, Oct 09, 2023, Kai Huang wrote: > > On Fri, 2023-09-22 at 20:06 -0700, Haitao Huang wrote: > > > +/** > > > + *

Re: [PATCH] bus: mhi: host: Add tracing support

2023-10-10 Thread kernel test robot
: 3006adf3be79cde4d14b1800b963b82b6e5572e0 patch link: https://lore.kernel.org/r/20231005-ftrace_support-v1-1-23a2f394fa49%40quicinc.com patch subject: [PATCH] bus: mhi: host: Add tracing support config: i386-randconfig-062-20231010 (https://download.01.org/0day-ci/archive/20231010/202310102355.6sea9ysi

Re: [PATCH] bpf/btf: Move tracing BTF APIs to the BTF library

2023-10-10 Thread Steven Rostedt
On Tue, 10 Oct 2023 22:54:19 +0900 "Masami Hiramatsu (Google)" wrote: > --- a/kernel/bpf/btf.c > +++ b/kernel/bpf/btf.c > @@ -912,6 +912,121 @@ static const struct btf_type > *btf_type_skip_qualifiers(const struct btf *btf, > return t; > } > > +/* > + * Find a function proto type by

Re: [PATCH v5] scripts/link-vmlinux.sh: Add alias to duplicate symbols for kallsyms

2023-10-10 Thread Kris Van Hees
On Tue, Oct 10, 2023 at 01:33:09PM +0200, Petr Mladek wrote: > On Mon 2023-10-09 15:14:28, Alessandro Carminati wrote: > > Hello Kris, > > > > Thank you for your contribution and for having your thought shared with me. > > > > Allow me to begin this conversation by explaining what came to mind

Re: [PATCH] KVM: deprecate KVM_WERROR in favor of general WERROR

2023-10-10 Thread Jakub Kicinski
On Tue, 10 Oct 2023 11:04:18 +0300 Jani Nikula wrote: > > If you do invest in build testing automation, why can't your automation > > count warnings rather than depend on WERROR? I don't understand. > > Because having both CI and the subsystem/driver developers enable a > local WERROR actually

[PATCH] bpf/btf: Move tracing BTF APIs to the BTF library

2023-10-10 Thread Masami Hiramatsu (Google)
From: Masami Hiramatsu (Google) Move the BTF APIs used in tracing to the BTF library code for sharing it with others. Previously, to avoid complex dependency in a series I made it on the tracing tree, but now it is a good time to move it to BPF tree because these functions are pure BTF

Re: [PATCH v5 12/18] x86/sgx: Add EPC OOM path to forcefully reclaim EPC

2023-10-10 Thread Haitao Huang
On Mon, 09 Oct 2023 21:23:12 -0500, Huang, Kai wrote: On Mon, 2023-10-09 at 20:42 -0500, Haitao Huang wrote: Hi Sean On Mon, 09 Oct 2023 19:23:04 -0500, Sean Christopherson wrote: > On Mon, Oct 09, 2023, Kai Huang wrote: > > On Fri, 2023-09-22 at 20:06 -0700, Haitao Huang wrote: > > > +/**

Re: [PATCH v7 00/13] selftests/sgx: Fix compilation errors

2023-10-10 Thread Jarkko Sakkinen
Dave, since there was already sort of talk about detaching this code from kernel tree so that Jo could work on "pure C" runtime would it make sense to dual-license this first in the kernel tree? E.g. Jo could send a patch once this is merged with a new SPDX platter and we can then ack that? Just

Re: [PATCH v5] scripts/link-vmlinux.sh: Add alias to duplicate symbols for kallsyms

2023-10-10 Thread Petr Mladek
On Mon 2023-10-09 15:14:28, Alessandro Carminati wrote: > Hello Kris, > > Thank you for your contribution and for having your thought shared with me. > > Allow me to begin this conversation by explaining what came to mind when > I decided to propose a patch that creates aliases. > > The

Re: [PATCH rebased] kbuild: rpm-pkg: Fix build with non-default MODLIB

2023-10-10 Thread Michal Suchánek
On Tue, Oct 10, 2023 at 12:14:01AM +0900, Masahiro Yamada wrote: > On Mon, Oct 9, 2023 at 11:07 PM Michal Suchánek wrote: > > > > On Mon, Oct 09, 2023 at 09:34:10PM +0900, Masahiro Yamada wrote: > > > On Mon, Oct 9, 2023 at 5:52 PM Michal Suchánek wrote: > > > > > > > > Hello, > > > > > > > > On

Re: [PATCH v7 00/13] selftests/sgx: Fix compilation errors

2023-10-10 Thread Jarkko Sakkinen
Folks (sorry for top posting): I've now taken my old NUC7 out of the dust and tested the series :-) Tested-by: Jarkko Sakkinen Off-topic: I wish both Intel and AMD straighten up and deliver some  "home friendly" development hardware for the latest stuff. Just my stance but the biggest quality

Re: [PATCH v5 16/18] x86/sgx: Limit process EPC usage with misc cgroup controller

2023-10-10 Thread Huang, Kai
On Fri, 2023-09-22 at 20:06 -0700, Haitao Huang wrote: > From: Kristen Carlson Accardi > > Implement support for cgroup control of SGX Enclave Page Cache (EPC) > memory using the misc cgroup controller. EPC memory is independent > from normal system memory, e.g. must be reserved at boot from RAM

Re: [PATCH 01/10] appletalk: remove localtalk and ppp support

2023-10-10 Thread kernel test robot
/20231009141908.1767241-1-arnd%40kernel.org patch subject: [PATCH 01/10] appletalk: remove localtalk and ppp support config: nios2-randconfig-001-20231010 (https://download.01.org/0day-ci/archive/20231010/202310101724.irnaop3r-...@intel.com/config) compiler: nios2-linux-gcc (GCC) 13.2.0 reproduce (this is a W=1

Re: [PATCH v5 16/18] x86/sgx: Limit process EPC usage with misc cgroup controller

2023-10-10 Thread Huang, Kai
> + > +static inline struct sgx_epc_cgroup *sgx_epc_cgroup_from_misc_cg(struct > misc_cg *cg) > +{ > + if (cg) > + return (struct sgx_epc_cgroup > *)(cg->res[MISC_CG_RES_SGX_EPC].priv); > + > + return NULL; > +} > + > Is it good idea to allow passing a NULL @cg to this

[PATCH v3 2/2] arm64: dts: imx8mp: add reserve-memory nodes for DSP

2023-10-10 Thread Iuliana Prodan (OSS)
From: Iuliana Prodan Add the reserve-memory nodes used by DSP when the rpmsg feature is enabled. Signed-off-by: Iuliana Prodan --- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 1 file changed, 16 insertions(+) diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi

[PATCH v3 1/2] remoteproc: imx_dsp_rproc: add mandatory find_loaded_rsc_table op

2023-10-10 Thread Iuliana Prodan (OSS)
From: Iuliana Prodan Add the .find_loaded_rsc_table operation for i.MX DSP. We need it for inter-process communication between DSP and main core. This callback is used to find the resource table (defined in remote processor linker script) where the address of the vrings along with the other

[PATCH v3 0/2] Rpmsg support for i.MX DSP with resource table

2023-10-10 Thread Iuliana Prodan (OSS)
From: Iuliana Prodan These patches are needed in order to support rpmsg on DSP when a resource table is available. Changes since v2: - add newline between nodes in dtsi (patch 2/2) Changes since v1: - add missing bracket in dtsi (patch 2/2) Iuliana Prodan (2): remoteproc: imx_dsp_rproc:

Re: [PATCH 01/10] appletalk: remove localtalk and ppp support

2023-10-10 Thread Arnd Bergmann
On Tue, Oct 10, 2023, at 09:10, Rodolfo Zitellini wrote: >> Il giorno 9 ott 2023, alle ore 19:29, Arnd Bergmann ha >> scritto: >> On Mon, Oct 9, 2023, at 18:49, Rodolfo Zitellini wrote: >> I can see a few ways this could work out: >> >> - add a custom callback pointer to struct atalk_iface to

Re: [PATCH] KVM: deprecate KVM_WERROR in favor of general WERROR

2023-10-10 Thread Jani Nikula
On Mon, 09 Oct 2023, Jakub Kicinski wrote: > On Mon, 9 Oct 2023 12:33:53 -0700 Sean Christopherson wrote: >> > We do have sympathy for these folks, we are mostly volunteers after >> > all. At the same time someone's under-investment should not be causing >> > pain to those of us who _do_ build

[PATCH v2] appletalk: make localtalk and ppp support conditional

2023-10-10 Thread Arnd Bergmann
From: Arnd Bergmann The last localtalk driver is gone now, and ppp support was never fully merged, but the code to support them for phase1 networking still calls the deprecated .ndo_do_ioctl() helper. In order to better isolate the localtalk and ppp portions of appletalk, guard all of the

Re: [PATCH 4/5] kbuild: unify vdso_install rules

2023-10-10 Thread Sven Schnelle
Masahiro Yamada writes: > Currently, there is no standard implementation for vdso_install, > leading to various issues: > > 1. Code duplication > > Many architectures duplicate similar code just for copying files > to the install destination. > > Some architectures (arm, sparc, x86)

Re: [PATCH 01/10] appletalk: remove localtalk and ppp support

2023-10-10 Thread Rodolfo Zitellini
> Il giorno 9 ott 2023, alle ore 19:29, Arnd Bergmann ha > scritto: > > On Mon, Oct 9, 2023, at 18:49, Rodolfo Zitellini wrote: >>> From: Arnd Bergmann >>> >>> The last localtalk driver is gone now, and ppp support was never fully >>> merged, so clean up the appletalk code by removing the

Re: [PATCH 09/10] wireless: hostap: remove unused ioctl function

2023-10-10 Thread Kalle Valo
Arnd Bergmann writes: > From: Arnd Bergmann > > The ioctl handler has no actual callers in the kernel and is useless. > All the functionality should be reachable through the regualar interfaces. > > Signed-off-by: Arnd Bergmann In the title we prefer "wifi:" over "wireless:" but that's

Re: [PATCH 08/10] wireless: atmel: remove unused ioctl function

2023-10-10 Thread Kalle Valo
Arnd Bergmann writes: > From: Arnd Bergmann > > This function has no callers, and for the past 20 years, the request_firmware > interface has been in place instead of the custom firmware loader. > > Signed-off-by: Arnd Bergmann Yuck, good riddance. In the title we prefer "wifi:" over

Re: [PATCH v1] Ftrace: make sched_wakeup can focus on the target process

2023-10-10 Thread Daniel Bristot de Oliveira
On 10/9/23 17:37, Jinyu Tang wrote: > $ cyclictest --mlockall --smp --priority=99 & rtla timerlat -a will give you an structured analysis of your latency... https://bristot.me/linux-scheduling-latency-debug-and-analysis/ -- Daniel

Re: [PATCH] remoteproc: st: Use device_get_match_data()

2023-10-10 Thread Patrice CHOTARD
On 10/9/23 23:13, Rob Herring wrote: > Use preferred device_get_match_data() instead of of_match_device() to > get the driver match data. With this, adjust the includes to explicitly > include the correct headers. > > Signed-off-by: Rob Herring > --- > drivers/remoteproc/st_remoteproc.c | 14