Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Puranjay Mohan
Song Liu writes: > On Wed, Feb 12, 2025 at 11:26 PM Puranjay Mohan wrote: >> >> Song Liu writes: >> >> > On Wed, Feb 12, 2025 at 4:10 PM Indu Bhagat wrote: >> >> >> >> On 2/12/25 3:32 PM, Song Liu wrote: >> >> > I run some tests with this set and my RFC set [1]. Most of >> >> > the test is don

Re: [RFC PATCH 15/39] KVM: guest_memfd: hugetlb: allocate and truncate from hugetlb

2025-02-12 Thread Ackerley Tng
Peter Xu writes: > On Tue, Sep 10, 2024 at 11:43:46PM +, Ackerley Tng wrote: >> +static struct folio *kvm_gmem_hugetlb_alloc_folio(struct hstate *h, >> + struct hugepage_subpool >> *spool) >> +{ >> +bool memcg_charge_was_prepared; >> +stru

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Puranjay Mohan
Song Liu writes: > On Wed, Feb 12, 2025 at 6:45 PM Josh Poimboeuf wrote: >> >> On Wed, Feb 12, 2025 at 06:36:04PM -0800, Song Liu wrote: >> > > > [ 81.261748] copy_process+0xfdc/0xfd58 [livepatch_special_static] >> > > >> > > Does that copy_process+0xfdc/0xfd58 resolve to this line in >> > >

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Song Liu
On Wed, Feb 12, 2025 at 11:26 PM Puranjay Mohan wrote: > > Song Liu writes: > > > On Wed, Feb 12, 2025 at 4:10 PM Indu Bhagat wrote: > >> > >> On 2/12/25 3:32 PM, Song Liu wrote: > >> > I run some tests with this set and my RFC set [1]. Most of > >> > the test is done with kpatch-build. I tested

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Puranjay Mohan
Song Liu writes: > On Wed, Feb 12, 2025 at 4:10 PM Indu Bhagat wrote: >> >> On 2/12/25 3:32 PM, Song Liu wrote: >> > I run some tests with this set and my RFC set [1]. Most of >> > the test is done with kpatch-build. I tested both Puranjay's >> > version [3] and my version [4]. >> > >> > For gcc

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Josh Poimboeuf
On Wed, Feb 12, 2025 at 06:40:33PM -0800, Song Liu wrote: > On Wed, Feb 12, 2025 at 4:10 PM Indu Bhagat wrote: > > > > On 2/12/25 3:32 PM, Song Liu wrote: > > > I run some tests with this set and my RFC set [1]. Most of > > > the test is done with kpatch-build. I tested both Puranjay's > > > versi

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Josh Poimboeuf
On Wed, Feb 12, 2025 at 06:36:04PM -0800, Song Liu wrote: > > > [ 81.261748] copy_process+0xfdc/0xfd58 [livepatch_special_static] > > > > Does that copy_process+0xfdc/0xfd58 resolve to this line in > > copy_process()? > > > > refcount_inc(¤t->signal->sigcnt); > > > > Mayb

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Song Liu
On Wed, Feb 12, 2025 at 4:10 PM Indu Bhagat wrote: > > On 2/12/25 3:32 PM, Song Liu wrote: > > I run some tests with this set and my RFC set [1]. Most of > > the test is done with kpatch-build. I tested both Puranjay's > > version [3] and my version [4]. > > > > For gcc 14.2.1, I have seen the fol

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Song Liu
On Wed, Feb 12, 2025 at 3:49 PM Josh Poimboeuf wrote: > > On Wed, Feb 12, 2025 at 03:32:40PM -0800, Song Liu wrote: > > [ 81.250437] [ cut here ] > > [ 81.250818] refcount_t: saturated; leaking memory. > > [ 81.251201] WARNING: CPU: 0 PID: 95 at lib/refcount.c:22 > >

Re: Re: [Patch net 1/2] vsock/virtio: initialize rx_buf_nr and rx_buf_max_nr when resuming

2025-02-12 Thread Junnan Wu
>You need to update the title now that you're moving also queued_replies. > Well, I will update the title in V3 version. >On Tue, Feb 11, 2025 at 03:19:21PM +0800, Junnan Wu wrote: >>When executing suspend to ram twice in a row, >>the `rx_buf_nr` and `rx_buf_max_nr` increase to three times vq->nu

RE: Re: [PATCH 2/2] vsock/virtio: Don't reset the created SOCKET during s2r

2025-02-12 Thread Junnan Wu
>On Mon, Feb 10, 2025 at 12:48:03PM +0100, leona...@redhat.com wrote: >>Like for the other patch, some maintainers have not been CCd. > >Yes, please use `scripts/get_maintainer.pl`. > Ok, I will add other maintainers by this script in next push. >> >>On Fri, Feb 07, 2025 at 01:20:33PM +0800, Jun

Re: [PATCH] selftests: pid_namespace and pidfd missing include

2025-02-12 Thread Alessandro Zanni
> Running "make kselftest" results in several errors like these: > > pidfd_fdinfo_test.c:231:36: error: ‘MS_REC’ undeclared (first use in > this function) > 231 | r = mount(NULL, "/", NULL, MS_REC | MS_PRIVATE, 0); > > pidfd_fdinfo_test.c:231:45: error: ‘MS_PRIVATE’ undeclared (first use

Re: [PATCH net-next v19 25/26] testing/selftests: add test tool and scripts for ovpn module

2025-02-12 Thread kernel test robot
Hi Antonio, kernel test robot noticed the following build errors: [auto build test ERROR on 39f54262ba499d862420a97719d2f0eea0cbd394] url: https://github.com/intel-lab-lkp/linux/commits/Antonio-Quartulli/net-introduce-OpenVPN-Data-Channel-Offload-ovpn/20250211-091029 base: 39f54262ba499d86

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Indu Bhagat
On 2/12/25 3:32 PM, Song Liu wrote: I run some tests with this set and my RFC set [1]. Most of the test is done with kpatch-build. I tested both Puranjay's version [3] and my version [4]. For gcc 14.2.1, I have seen the following issue with this test [2]. This happens with both upstream and 6.13

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Josh Poimboeuf
On Wed, Feb 12, 2025 at 03:32:40PM -0800, Song Liu wrote: > [ 81.250437] [ cut here ] > [ 81.250818] refcount_t: saturated; leaking memory. > [ 81.251201] WARNING: CPU: 0 PID: 95 at lib/refcount.c:22 > refcount_warn_saturate+0x6c/0x140 > [ 81.251841] Modules linked i

Re: [PATCH net-next v19 00/26] Introducing OpenVPN Data Channel Offload

2025-02-12 Thread Sabrina Dubroca
Hello, 2025-02-11, 01:39:53 +0100, Antonio Quartulli wrote: > All minor and major reported problems have been finally addressed. > Big thanks to Sabrina, who took the time to guide me through > converting the peer socket to an RCU pointer. Something is off (not sure if it's new to this version):

Re: [PATCH 0/8] unwind, arm64: add sframe unwinder for kernel

2025-02-12 Thread Song Liu
I run some tests with this set and my RFC set [1]. Most of the test is done with kpatch-build. I tested both Puranjay's version [3] and my version [4]. For gcc 14.2.1, I have seen the following issue with this test [2]. This happens with both upstream and 6.13.2. The livepatch loaded fine, but the

Re: Re: Re: Re: [PATCH 2/2] libbpf: BPF programs dynamic loading and attaching

2025-02-12 Thread Martin Kelly
On Mon, 2025-02-10 at 16:06 -0800, Andrii Nakryiko wrote: > > Tracking associated maps for a program is not necessary. As long as > > the last BPF program using the BPF map is unloaded, the kernel will > > automatically free not-anymore-referenced BPF map. Note that > > bpf_object itself will keep

Re: [PATCH v2 6/6] selftests/mm: remove local __NR_* definitions

2025-02-12 Thread John Hubbard
On 2/12/25 12:34 PM, Dave Hansen wrote: Hi John, On 6/13/24 19:30, John Hubbard wrote: --- a/tools/testing/selftests/mm/protection_keys.c +++ b/tools/testing/selftests/mm/protection_keys.c @@ -42,7 +42,7 @@ #include #include #include -#include +#include #include #include I'm

Re: [PATCH v2 2/2] unicode: kunit: change tests filename and path

2025-02-12 Thread Kees Cook
On Wed, Feb 12, 2025 at 05:31:59PM +0800, David Gow wrote: > On Wed, 12 Feb 2025 at 14:45, Thorsten Leemhuis wrote: > > > > On 04.10.24 21:00, Gabriel Krisman Bertazi wrote: > > > Pedro Orlando writes: > > >> From: Gabriela Bittencourt > > >> > > >> Change utf8 kunit test filename and path to fo

Re: [PATCH] x86: sgx: Don't track poisoned pages for reclaiming

2025-02-12 Thread Jarkko Sakkinen
On Tue, Feb 11, 2025 at 03:31:54PM -0800, Dave Hansen wrote: > On 2/11/25 13:18, Huang, Kai wrote: > >>> This requires low-level SGX implementation knowledge to fully > >>> understand. Both what "ETRACK, EBLOCK and EWB" are in the first place, > >>> how they are involved in reclaim and also why ERE

Re: [PATCH v2 6/6] selftests/mm: remove local __NR_* definitions

2025-02-12 Thread Dave Hansen
Hi John, On 6/13/24 19:30, John Hubbard wrote: > --- a/tools/testing/selftests/mm/protection_keys.c > +++ b/tools/testing/selftests/mm/protection_keys.c > @@ -42,7 +42,7 @@ > #include > #include > #include > -#include > +#include > #include > #include I'm not quite sure how but this b

[PATCH] tools/nolibc: add support for N64 and N32 ABIs

2025-02-12 Thread Thomas Weißschuh
Add support for the MIPS 64bit N64 and ILP32 N32 ABIs. In addition to different byte orders and ABIs there are also different releases of the MIPS architecture. To avoid blowing up the test matrix, only add a subset of all possible test combinations. Signed-off-by: Thomas Weißschuh --- tools/in

[PATCH] selftests/nolibc: only run constructor tests on nolibc

2025-02-12 Thread Thomas Weißschuh
it holes above */ --- base-commit: 16681bea9a80080765c98b545ad74c17de2d513c change-id: 20250212-nolibc-test-constructor-42491ba71a19 Best regards, -- Thomas Weißschuh

Re: [PATCH] selftests/nolibc: split up architecture list in run-tests.sh

2025-02-12 Thread Thomas Weißschuh
FYI Willy, On 2025-02-11 12:03:52+0100, Thomas Weißschuh wrote: > The list is getting overly long and any modifications introduce a lot of > noise and are prone to conflicts. Split the string into a bash array > and break that into multiple lines. I picked up this trivial change already. > > Si

[PATCH 1/2] selftests/mm: Fix check for running THP tests

2025-02-12 Thread Mark Brown
When testing if we should try to compact memory or drop caches before we run the THP or HugeTLB tests we use | as an or operator. This doesn't work since run_vmtests.sh is written in shell where this is used to pipe the output of the first argument into the second. Instead use the shell's -o operat

[PATCH 2/2] selftests/mm: Allow tests to run with no huge pages support

2025-02-12 Thread Mark Brown
Currently the mm selftests refuse to run if huge pages are not available in the current system but this is an optional feature and not all the tests actually require them. Change the test during startup to be non-fatal and skip or omit tests which actually rely on having huge pages, allowing the ot

[PATCH 0/2] selftests/mm: Allow execution on systems without huge pages

2025-02-12 Thread Mark Brown
Currently the mm selftests refuse to run if we don't have huge page support but there are plenty of tests that don't depend on this feature, relax this requirement to allow coverage on relevant systems (eg, most 32 bit arm ones). While doing this I noticed a bug with an existing check if we're run

Re: [PATCH v3 bpf 1/2] bpf: skip non exist keys in generic_map_lookup_batch

2025-02-12 Thread Jiri Olsa
On Mon, Feb 10, 2025 at 10:21:38AM -0600, Yan Zhai wrote: > Hi Brian, Jiri > > thanks for the comments. > > On Mon, Feb 10, 2025 at 8:47 AM Brian Vazquez wrote: > > > > On Mon, Feb 10, 2025 at 4:19 AM Jiri Olsa wrote: > > > > > > On Sun, Feb 09, 2025 at 11:22:35PM -0800, Yan Zhai wrote: > > > >

Re: [PATCH v7 0/3] scanf: convert self-test to KUnit

2025-02-12 Thread Tamir Duberstein
On Tue, Feb 11, 2025 at 12:26 PM Tamir Duberstein wrote: > > > Is it me who cut something or the above missing this information (total > > tests)? > > If the latter, how are we supposed to answer to the question if the failed > > test > > is from new bunch of cases I hypothetically added or regr

Re: selftest/vDSO: vdso_test_abi tests with gcc-13 and pass with clang-19

2025-02-12 Thread Arnd Bergmann
On Wed, Feb 12, 2025, at 15:29, Naresh Kamboju wrote: > The kselftest-vDSO/vdso_test_abi test encounters failures when built > and executed using GCC-13 on armv7 architecture. > The issue has been observed on both TI X15 devices and QEMU-armv7. > > Interestingly, the same test passes without any is

Re: [PATCH] ww_mutex: convert self-test to KUnit

2025-02-12 Thread Dan Carpenter
On Wed, Feb 12, 2025 at 09:33:46AM -0500, Tamir Duberstein wrote: > > > > As an aside, how can I compile with the warning settings used by > kernel test robot? > This is a Smatch warning. https://github.com/error27/smatch https://github.com/error27/smatch/blob/master/Documentation/smatch.rst R

[PATCH] fs/dax: Remove unnecessary check in dax_break_layout_final()

2025-02-12 Thread Dan Carpenter
The "page" pointer is always NULL at this point. Adding a check is a bit confusing. Delete it. Signed-off-by: Dan Carpenter --- fs/dax.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/dax.c b/fs/dax.c index 9e4940a0b286..21a743996f90 100644 --- a/fs/dax.c +++ b/fs/dax

Re: [PATCH v2 1/2] get_maintainer: add --substatus for reporting subsystem status

2025-02-12 Thread Uwe Kleine-König
On Wed, Feb 12, 2025 at 01:42:15PM +0100, Geert Uytterhoeven wrote: > Hi Vlastimil, > > On Tue, 11 Feb 2025 at 17:01, Vlastimil Babka wrote: > > On 2/3/25 12:13, Vlastimil Babka wrote: > > > The subsystem status is currently reported with --role(stats) by > > > adjusting the maintainer role for a

Re: [PATCH] ww_mutex: convert self-test to KUnit

2025-02-12 Thread Tamir Duberstein
0250211-000245 > base: a64dcfb451e254085a7daee5fe51bf22959d52d3 > patch link: > https://lore.kernel.org/r/20250210-ww_mutex-kunit-convert-v1-1-972f0201f71e%40gmail.com > patch subject: [PATCH] ww_mutex: convert self-test to KUnit > config: i386-randconfig-141-20250212 > (https://download.01.or

selftest/vDSO: vdso_test_abi tests with gcc-13 and pass with clang-19

2025-02-12 Thread Naresh Kamboju
The kselftest-vDSO/vdso_test_abi test encounters failures when built and executed using GCC-13 on armv7 architecture. The issue has been observed on both TI X15 devices and QEMU-armv7. Interestingly, the same test passes without any issues when built using Clang-19.This failure is specific to GCC-

Re: [PATCH v15 2/8] remoteproc: Add TEE support

2025-02-12 Thread Arnaud POULIQUEN
Hello, On 2/12/25 04:18, Bjorn Andersson wrote: > On Tue, Dec 10, 2024 at 09:57:40AM +0100, Arnaud POULIQUEN wrote: >> Hello Bjorn, >> >> On 12/6/24 23:07, Bjorn Andersson wrote: >>> On Thu, Nov 28, 2024 at 09:42:09AM GMT, Arnaud Pouliquen wrote: Add a remoteproc TEE (Trusted Execution Enviro

Re: kvm: nVHE hyp panic at: __kvm_nvhe_kvm_hyp_handle_sysre

2025-02-12 Thread Marc Zyngier
On Wed, 12 Feb 2025 11:41:38 +, Dan Carpenter wrote: > > On Tue, Feb 11, 2025 at 11:36:31AM +, Marc Zyngier wrote: > > For the crash at hand, which clearly shows nVHE, can you report > > whether the following hack fixes it for you? > > > > M. > > No luck, I'm afraid. It still crash

Re: [PATCH v2 2/2] unicode: kunit: change tests filename and path

2025-02-12 Thread Thorsten Leemhuis
On 12.02.25 07:45, Thorsten Leemhuis wrote: > On 04.10.24 21:00, Gabriel Krisman Bertazi wrote: >> Pedro Orlando writes: > [...] >> This breaks compilation for me. >> >> fs/unicode/tests/utf8_kunit.c:11:10: fatal error: utf8n.h: No such file or >> directory >>11 | #include "utf8n.h" >>

Re: [PATCH v15 3/8] remoteproc: Introduce load_fw and release_fw optional operation

2025-02-12 Thread Arnaud POULIQUEN
On 2/12/25 04:54, Bjorn Andersson wrote: > On Tue, Dec 10, 2024 at 11:33:31AM +0100, Arnaud POULIQUEN wrote: >> >> >> On 12/10/24 00:14, Bjorn Andersson wrote: >>> On Thu, Nov 28, 2024 at 09:42:10AM GMT, Arnaud Pouliquen wrote: This patch updates the rproc_ops structures to include two new

[PATCH net-next v7 3/3] selftests: drv-net: Test queue xsk attribute

2025-02-12 Thread Joe Damato
Test that queues which are used for AF_XDP have the xsk nest attribute. The attribute is currently empty, but its existence means the AF_XDP is being used for the queue. Enable CONFIG_XDP_SOCKETS for selftests/drivers/net tests, as well. Signed-off-by: Joe Damato Suggested-by: Jakub Kicinski ---

[PATCH net-next v7 0/3] netdev-genl: Add an xsk attribute to queues

2025-02-12 Thread Joe Damato
Greetings: Welcome to v7. Added CONFIG_XDP_SOCKETS to the selftests/driver/net kernel config in patch 3. Updated the xdp_helper.c to return -1 on AF_XDP non-existence, and updated queues.py to skip the test if AF_XDP does not exist (but fail in other failure cases). Tested on kernels with and with

[syzbot] [modules?] KMSAN: uninit-value in __request_module (6)

2025-02-12 Thread syzbot
Hello, syzbot found the following issue on: HEAD commit:febbc555cf0f Merge tag 'nfsd-6.14-1' of git://git.kernel.o.. git tree: upstream console+strace: https://syzkaller.appspot.com/x/log.txt?x=137a78e458 kernel config: https://syzkaller.appspot.com/x/.config?x=48f90cac5eea091a das

Re: [PATCH net-next v6 3/3] selftests: drv-net: Test queue xsk attribute

2025-02-12 Thread Joe Damato
On Tue, Feb 11, 2025 at 06:37:06PM -0800, Jakub Kicinski wrote: > On Tue, 11 Feb 2025 15:10:52 -0800 Stanislav Fomichev wrote: > > > I can't comment on NIPA because I have no idea how it works. Maybe > > > there is a kernel with some options enabled and other kernels with > > > various options disa

[PATCH v5 2/2] selftests/futex: Create test for robust list

2025-02-12 Thread André Almeida
Create a test for the robust list mechanism. Test the following uAPI operations: - Creating a robust mutex where the lock waiter is wake by the kernel when the lock owner died - Setting a robust list to the current task - Getting a robust list from the current task - Getting a robust list from a

[PATCH v5 1/2] selftests/futex: Add ASSERT_ macros

2025-02-12 Thread André Almeida
Create ASSERT_{EQ, NE, TRUE, FALSE} macros to make test creation easier. Signed-off-by: André Almeida --- .../testing/selftests/futex/include/logging.h | 38 +++ 1 file changed, 38 insertions(+) diff --git a/tools/testing/selftests/futex/include/logging.h b/tools/testing/selfte

[PATCH 2/2] kunit: qemu_configs: Add MIPS configurations

2025-02-12 Thread Thomas Weißschuh
Add basic support to run various MIPS variants via kunit_tool using the virtualized malta platform. Signed-off-by: Thomas Weißschuh --- tools/testing/kunit/qemu_configs/mips.py | 18 ++ tools/testing/kunit/qemu_configs/mips64.py | 19 +++ tools/testing/kunit

[PATCH 1/2] MIPS: mm: Avoid blocking DMA zone with memory map memblock allocation

2025-02-12 Thread Thomas Weißschuh
On MIPS the memblock allocator is configured to allocate bottom-up. The memory map is allocated by the mm core through memblock and uses MEMBLOCK_LOW_LIMIT as minimal address. This constant is defined as zero because it assumes that "we are using top down, so it is safe to use 0 here". So the memor

[PATCH 0/2] kunit: qemu_configs: Add MIPS configurations

2025-02-12 Thread Thomas Weißschuh
Add basic support to run various MIPS variants via kunit_tool using the virtualized malta platform. Various kunit tests from drivers/firmware/cirrus/ are failing on MIPS. They are fixed in [0]. [0] https://lore.kernel.org/lkml/20250211-cs_dsp-kunit-strings-v1-1-d9bc2035d...@linutronix.de/ Signed

Re: [PATCH v2 1/2] get_maintainer: add --substatus for reporting subsystem status

2025-02-12 Thread Geert Uytterhoeven
Hi Vlastimil, On Tue, 11 Feb 2025 at 17:01, Vlastimil Babka wrote: > On 2/3/25 12:13, Vlastimil Babka wrote: > > The subsystem status is currently reported with --role(stats) by > > adjusting the maintainer role for any status different from Maintained. > > This has two downsides: > > > > - if a

Re: [PATCH v1 1/1] virtio_console: Get rid of unneeded temporary variable

2025-02-12 Thread Amit Shah
On Mon, 2025-02-10 at 11:59 +0200, Andy Shevchenko wrote: > When compiling a kernel with GCC using `make W=1` with > CONFIG_WERROR=y > (which is default nowadays), the build fails: > > drivers/char/virtio_console.c:1427:9: note: ‘snprintf’ output between > 9 and 27 bytes into a destination of size

Re: [PATCH] virtio: console: Use str_yes_no() helper in port_debugfs_show()

2025-02-12 Thread Amit Shah
On Wed, 2025-02-12 at 12:48 +0100, Thorsten Blum wrote: > Remove hard-coded strings by using the str_yes_no() helper function. > > Signed-off-by: Thorsten Blum > --- >  drivers/char/virtio_console.c | 4 ++-- >  1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Amit Shah

Re: [PATCH] ww_mutex: convert self-test to KUnit

2025-02-12 Thread Dan Carpenter
-ww_mutex-kunit-convert-v1-1-972f0201f71e%40gmail.com patch subject: [PATCH] ww_mutex: convert self-test to KUnit config: i386-randconfig-141-20250212 (https://download.01.org/0day-ci/archive/20250212/202502121806.cs6r741y-...@intel.com/config) compiler: clang version 19.1.3 (https://github.com/llvm/llvm

[PATCH] virtio: console: Use str_yes_no() helper in port_debugfs_show()

2025-02-12 Thread Thorsten Blum
Remove hard-coded strings by using the str_yes_no() helper function. Signed-off-by: Thorsten Blum --- drivers/char/virtio_console.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index 24442485e73e..35af0cc11d

Re: kvm: nVHE hyp panic at: __kvm_nvhe_kvm_hyp_handle_sysre

2025-02-12 Thread Naresh Kamboju
On Wed, 12 Feb 2025 at 17:11, Dan Carpenter wrote: > > On Tue, Feb 11, 2025 at 11:36:31AM +, Marc Zyngier wrote: > > For the crash at hand, which clearly shows nVHE, can you report > > whether the following hack fixes it for you? > > > > M. > > No luck, I'm afraid. It still crashes the

Re: kvm: nVHE hyp panic at: __kvm_nvhe_kvm_hyp_handle_sysre

2025-02-12 Thread Dan Carpenter
On Tue, Feb 11, 2025 at 11:36:31AM +, Marc Zyngier wrote: > For the crash at hand, which clearly shows nVHE, can you report > whether the following hack fixes it for you? > > M. No luck, I'm afraid. It still crashes the same way. https://tuxapi.tuxsuite.com/v1/groups/linaro/projects/a

Re: kvm: nVHE hyp panic at: __kvm_nvhe_kvm_hyp_handle_sysre

2025-02-12 Thread Dan Carpenter
On Tue, Feb 11, 2025 at 11:36:31AM +, Marc Zyngier wrote: > On Tue, 11 Feb 2025 11:24:06 +, > Naresh Kamboju wrote: > > > > Regression on rk3399-rock-pi-4b while running kvm-unit-tests with > > nvhe, protected and vhe mode with virtualization enabled. > > I do not buy this. RK3399 only h

[PATCH bpf-next v5 6/6] selftests/bpf: Remove test_xdp_redirect_multi.sh

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
The tests done by test_xdp_redirect_multi.sh are now fully covered by the CI through test_xdp_veth.c. Remove test_xdp_redirect_multi.sh Remove xdp_redirect_multi.c that was used by the script to load and attach the BPF programs. Remove their entries in the Makefile Acked-by: Stanislav Fomichev S

[PATCH bpf-next v5 4/6] selftests/bpf: test_xdp_veth: Add XDP broadcast redirection tests

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
XDP redirections with BPF_F_BROADCAST and BPF_F_EXCLUDE_INGRESS flags are tested by test_xdp_redirect_multi.sh but not within the test_progs framework. Add a broadcast test case in test_xdp_veth.c to test them. Use the same BPF programs than the one used by test_xdp_redirect_multi.sh. Use a BPF ma

[PATCH bpf-next v5 5/6] selftests/bpf: test_xdp_veth: Add XDP program on egress test

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
XDP programs loaded on egress is tested by test_xdp_redirect_multi.sh but not by the test_progs framework. Add a test case in test_xdp_veth.c to test the XDP program on egress. Use the same BPF program than test_xdp_redirect_multi.sh that replaces the source MAC address by one provided through a B

[PATCH bpf-next v5 3/6] selftests/bpf: Optionally select broadcasting flags

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
Broadcasting flags are hardcoded for each kind for protocol. Create a redirect_flags map that allows to select the broadcasting flags to use in the bpf_redirect_map(). The protocol ID is used as a key. Set the old hardcoded values as default if the map isn't filled by the BPF caller. Acked-by: St

[PATCH bpf-next v5 2/6] selftests/bpf: test_xdp_veth: Use a dedicated namespace

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
Tests use the root network namespace, so they aren't fully independent of each other. For instance, the index of the created veth interfaces is incremented every time a new test is launched. Wrap the network topology in a network namespace to ensure full isolation. Use the append_tid() helper to e

[PATCH bpf-next v5 1/6] selftests/bpf: test_xdp_veth: Create struct net_configuration

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
The network configuration is defined by a table of struct veth_configuration. This isn't convenient if we want to add a network configuration that isn't linked to a veth pair. Create a struct net_configuration that holds the veth_configuration table to ease adding new configuration attributes in u

[PATCH bpf-next v5 0/6] selftests/bpf: Migrate test_xdp_redirect_multi.sh to test_progs

2025-02-12 Thread Bastien Curutchet (eBPF Foundation)
Hi all, This patch series continues the work to migrate the *.sh tests into prog_tests framework. test_xdp_redirect_multi.sh tests the XDP redirections done through bpf_redirect_map(). This is already partly covered by test_xdp_veth.c that already tests map redirections at XDP level. What isn't

Re: [PATCH v5 6/8] iio: light: stk3310: use dev_err_probe where possible

2025-02-12 Thread Andy Shevchenko
On Tue, Feb 11, 2025 at 07:43:11PM +, Jonathan Cameron wrote: > On Sun, 9 Feb 2025 16:47:44 +0200 > Andy Shevchenko wrote: > > On Sat, Feb 08, 2025 at 04:13:24PM -0500, Aren Moynihan wrote: > > > Using dev_err_probe instead of dev_err and return makes the errors > > > > Use dev_err_probe()

Re: [PATCH RFC v2 rcu] Fix get_state_synchronize_rcu_full() GP-start detection

2025-02-12 Thread Paul E. McKenney
On Wed, Feb 12, 2025 at 02:14:19AM -0800, Paul E. McKenney wrote: > On Mon, Feb 10, 2025 at 08:22:24PM -0500, Joel Fernandes wrote: > > On Mon, Feb 10, 2025 at 7:28 PM Joel Fernandes > > wrote: > > > On Mon, Jan 27, 2025 at 1:45 PM Joel Fernandes > > > wrote: > > > > On Mon, Jan 27, 2025 at 11:

Re: [PATCH] x86: sgx: Don't track poisoned pages for reclaiming

2025-02-12 Thread Huang, Kai
On 12/02/2025 12:31 pm, Dave Hansen wrote: On 2/11/25 13:18, Huang, Kai wrote: This requires low-level SGX implementation knowledge to fully understand. Both what "ETRACK, EBLOCK and EWB" are in the first place, how they are involved in reclaim and also why EREMOVE doesn't lead to the same fa

Re: [PATCH RFC v2 rcu] Fix get_state_synchronize_rcu_full() GP-start detection

2025-02-12 Thread Paul E. McKenney
On Mon, Feb 10, 2025 at 08:22:24PM -0500, Joel Fernandes wrote: > On Mon, Feb 10, 2025 at 7:28 PM Joel Fernandes wrote: > > On Mon, Jan 27, 2025 at 1:45 PM Joel Fernandes > > wrote: > > > On Mon, Jan 27, 2025 at 11:49 AM Paul E. McKenney > > > wrote: > > > > On Sun, Jan 26, 2025 at 09:58:11PM

Re: [PATCH] ww_mutex: convert self-test to KUnit

2025-02-12 Thread David Gow
On Wed, 12 Feb 2025 at 06:43, Tamir Duberstein wrote: > > On Tue, Feb 11, 2025 at 5:38 PM Boqun Feng wrote: > > > > Hi Tamir, > > Hi Boqun, thanks for taking a look. > > > On Mon, Feb 10, 2025 at 10:59:12AM -0500, Tamir Duberstein wrote: > > > Convert this unit test to a KUnit test. > > > > > > >

Re: [PATCH v2 2/2] unicode: kunit: change tests filename and path

2025-02-12 Thread David Gow
On Wed, 12 Feb 2025 at 14:45, Thorsten Leemhuis wrote: > > On 04.10.24 21:00, Gabriel Krisman Bertazi wrote: > > Pedro Orlando writes: > >> From: Gabriela Bittencourt > >> > >> Change utf8 kunit test filename and path to follow the style > >> convention on Documentation/dev-tools/kunit/style.rst

Re: [PATCH 0/2] printf: convert self-test to KUnit

2025-02-12 Thread David Gow
On Tue, 11 Feb 2025 at 16:40, Rasmus Villemoes wrote: > > On Tue, Feb 11 2025, David Gow wrote: > > > On Mon, 10 Feb 2025 at 19:57, Rasmus Villemoes > > wrote: > >> > >> On Fri, Feb 07 2025, Tamir Duberstein wrote: > >> > >> > On Fri, Feb 7, 2025 at 5:01 AM Rasmus Villemoes > >> > wrote: > >>

Re: [PATCH v6 2/6] rust: str: implement `Index` for `BStr`

2025-02-12 Thread Gary Guo
On Tue, 11 Feb 2025 21:24:44 +0100 Andreas Hindborg wrote: > "Gary Guo" writes: > > > On Tue, 11 Feb 2025 16:57:36 +0100 > > Andreas Hindborg wrote: > > > >> The `Index` implementation on `BStr` was lost when we switched `BStr` from > >> a type alias of `[u8]` to a newtype. This patch adds b

Re: [PATCH v6 5/6] rust: str: add radix prefixed integer parsing functions

2025-02-12 Thread Gary Guo
On Tue, 11 Feb 2025 21:13:10 +0100 Andreas Hindborg wrote: > "Gary Guo" writes: > > > On Tue, 11 Feb 2025 16:57:39 +0100 > > Andreas Hindborg wrote: > > > >> Add the trait `ParseInt` for parsing string representations of integers > >> where the string representations are optionally prefixed