On Thu, 21 Dec 2023 02:58:40 +,
Haibo Xu wrote:
>
> On Wed, Dec 20, 2023 at 9:58 PM Marc Zyngier wrote:
> >
> > On Wed, 20 Dec 2023 13:51:24 +,
> > Haibo Xu wrote:
> > >
> > > On Wed, Dec 20, 2023 at 5:00 PM Marc Zyngier wrote:
> > > >
> > > > On 2023-12-20 06:50, Haibo Xu wrote:
> > >
Adds a variant of example_static_stub_test() that shows use of a
pointer-to-function with kunit_activate_static_stub().
A const pointer to the add_one() function is declared. This
pointer-to-function is passed to kunit_activate_static_stub() and
kunit_deactivate_static_stub() instead of passing ad
Swap the arguments to typecheck_fn() in kunit_activate_static_stub()
so that real_fn_addr can be either the function itself or a pointer
to that function.
This is useful to simplify redirecting static functions in a module.
Having to pass the actual function meant that it must be exported
from the
On Wed, 2023-12-20 at 13:53 -0300, Marcos Paulo de Souza wrote:
> Changes in v4:
> * Documented how to compile the livepatch selftests without running
> the
> tests (Joe)
> * Removed the mention to lib/livepatch on MAINTAINERS file, reported
> by
> checkpatch.
>
To clarify: this is not a rese
On Tue, Dec 19, 2023 at 12:57:57PM +0200, Roger Quadros wrote:
> From: Vladimir Oltean
>
> Some devices do not support individual 'pmac' and 'emac' stats.
> For such devices, resort to 'aggregate' stats.
>
> Cc: Shuah Khan
> Signed-off-by: Vladimir Oltean
> Tested-by: Roger Quadros
> Signed-o
On Tue, Dec 19, 2023 at 12:57:56PM +0200, Roger Quadros wrote:
> From: Vladimir Oltean
>
> Some devices have errata due to which they cannot report ETH_ZLEN (60)
> in the rx-min-frag-size. This was foreseen of course, and lldpad has
> logic that when we request it to advertise addFragSize 0, it w
Nested translation is a hardware feature that is supported by many modern
IOMMU hardwares. It has two stages (stage-1, stage-2) address translation
to get access to the physical address. stage-1 translation table is owned
by userspace (e.g. by a guest OS), while stage-2 is owned by kernel. Changes
From: Lu Baolu
The updates of the PTEs in the nested page table will be propagated to the
hardware caches on both IOMMU (IOTLB) and devices (DevTLB/ATC).
Add a new domain op cache_invalidate_user for the userspace to flush the
hardware caches for a nested domain through iommufd. No wrapper for i
In nested translation, the stage-1 page table is user-managed but cached
by the IOMMU hardware, so an update on present page table entries in the
stage-1 page table should be followed with a cache invalidation.
Add an IOMMU_HWPT_INVALIDATE ioctl to support such a cache invalidation.
It takes hwpt_
From: Nicolin Chen
Add mock_domain_cache_invalidate_user() data structure to support user
space selftest program to cover user cache invalidation pathway.
Signed-off-by: Nicolin Chen
Co-developed-by: Yi Liu
Signed-off-by: Yi Liu
---
drivers/iommu/iommufd/iommufd_test.h | 34 ++
d
From: Nicolin Chen
Wrap up the data pointer/num sanity and __iommu_copy_struct_from_user
call for iommu drivers to copy driver specific data at a specific location
in the struct iommu_user_data_array, and iommu_respond_struct_to_user_array()
to copy response to a specific location in the struct i
From: Nicolin Chen
Allow to test whether IOTLB has been invalidated or not.
Signed-off-by: Nicolin Chen
Signed-off-by: Yi Liu
---
drivers/iommu/iommufd/iommufd_test.h | 5
drivers/iommu/iommufd/selftest.c | 26 +++
tools/testing/selftests/iommu/iomm
From: Nicolin Chen
Add test cases for the IOMMU_HWPT_INVALIDATE ioctl and verify it by using
the new IOMMU_TEST_OP_MD_CHECK_IOTLB.
Signed-off-by: Nicolin Chen
Co-developed-by: Yi Liu
Signed-off-by: Yi Liu
---
tools/testing/selftests/iommu/iommufd.c | 166 ++
tools/testi
From: Lu Baolu
This allows qi_submit_sync() to return back faults to callers.
Signed-off-by: Lu Baolu
Signed-off-by: Yi Liu
---
drivers/iommu/intel/dmar.c | 29 ++---
drivers/iommu/intel/iommu.h | 2 +-
drivers/iommu/intel/irq_remapping.c | 2 +-
dri
From: Lu Baolu
This makes the pasid based cache invalidation to return QI faults to callers.
This is needed when usersapce wants to invalidate cache after modifying the
stage-1 page table used in nested translation.
Signed-off-by: Lu Baolu
Signed-off-by: Yi Liu
---
drivers/iommu/intel/dmar.c
From: Lu Baolu
This implements the .cache_invalidate_user() callback to support iotlb
flush for nested domain.
Signed-off-by: Lu Baolu
Co-developed-by: Yi Liu
Signed-off-by: Yi Liu
---
drivers/iommu/intel/nested.c | 116 +++
1 file changed, 116 insertions(+)
This series attempts to reduce parsing overhead of IPv6 extension headers
in GRO and GSO, by removing extension header specific code and enabling
the frag0 fast path.
The following changes were made:
- Specific unnecessary HBH conditionals were removed by adding HBH offload
to inet6_offloads
-
This commit adds net_offload to IPv6 Hop-by-Hop extension headers (as it
is done for routing and dstopts) since it is supported in GSO and GRO.
This allows to remove specific HBH conditionals in GSO and GRO when
pulling and parsing an incoming packet.
Signed-off-by: Richard Gobert
---
net/ipv6/e
This commit utilizes a new helper function, ipv6_gro_pull_exthdrs, which
is used in ipv6_gro_receive to pull ipv6 ext headers instead of
ipv6_gso_pull_exthdrs. To use ipv6_gso_pull_exthdr, pskb_pull and
__skb_push must be used, and frag0 must be invalidated. This commit
removes unnecessary code aro
Currently there is no test which checks that IPv6 extension header packets
successfully coalesce. This commit adds a test, which verifies two IPv6
packets with HBH extension headers do coalesce.
I changed the receive socket filter to accept a packet with one extension
header. This change exposed a
>
> This patchset adds a couple of helpers for kunit as well as tests for
> cfg80211 and mac80211 that use them.
I can take this through the wireless tree, but then I'd like to have
ACKs from kunit folks for the kunit patches:
> kunit: add parameter generation macro using description from arra
On Fri, Dec 15, 2023 at 2:21 PM Laura Nao wrote:
>
> Add new basic kselftest that checks if the available rust sample modules
> can be added and removed correctly.
>
> Signed-off-by: Laura Nao
Thanks Laura!
Shuah: do you want that we pick this one? If so, your `Acked-by` would
be nice -- thanks
On 12/21/23 12:39, Johannes Berg wrote:
This patchset adds a couple of helpers for kunit as well as tests for
cfg80211 and mac80211 that use them.
I can take this through the wireless tree, but then I'd like to have
ACKs from kunit folks for the kunit patches:
We have run into conflicts in
On 12/21/23 05:17, Marcos Paulo de Souza wrote:
On Wed, 2023-12-20 at 13:53 -0300, Marcos Paulo de Souza wrote:
Changes in v4:
* Documented how to compile the livepatch selftests without running
the
tests (Joe)
* Removed the mention to lib/livepatch on MAINTAINERS file, reported
by
checkpa
This makes the uevent selftests build not write to the source tree
unconditionally, as that breaks out of tree builds when the source tree
is read-only. It also avoids leaving a git repository in a dirty state
after a build.
Signed-off-by: Antonio Terceiro
---
tools/testing/selftests/uevent/Make
On Thu, 2023-12-21 at 13:06 -0700, Shuah Khan wrote:
> On 12/21/23 12:39, Johannes Berg wrote:
> > >
> > > This patchset adds a couple of helpers for kunit as well as tests for
> > > cfg80211 and mac80211 that use them.
> >
> > I can take this through the wireless tree, but then I'd like to have
This makes the uevent selftests build not write to the source tree
unconditionally, as that breaks out of tree builds when the source tree
is read-only. It also avoids leaving a git repository in a dirty state
after a build.
v2: drop spurious extra SPDX-License-Identifier
Signed-off-by: Antonio T
On 12/21/23 13:49, Antonio Terceiro wrote:
This makes the uevent selftests build not write to the source tree
unconditionally, as that breaks out of tree builds when the source tree
is read-only. It also avoids leaving a git repository in a dirty state
after a build.
Why can't you do that usin
On 12/21/23 13:40, Johannes Berg wrote:
On Thu, 2023-12-21 at 13:06 -0700, Shuah Khan wrote:
On 12/21/23 12:39, Johannes Berg wrote:
This patchset adds a couple of helpers for kunit as well as tests for
cfg80211 and mac80211 that use them.
I can take this through the wireless tree, but then
From: "Steven Rostedt (Google)"
As there were bugs found with the ownership of eventfs dynamic file
creation. Add a test to test it.
It will remount tracefs with a different gid and check the ownership of
the eventfs directory, as well as the system and event directories. It
will also check the
On Thu, 21 Dec 2023 19:35:51 -0500
Steven Rostedt wrote:
> +
> +# find another owner and group that is not the original
> +other_group=`tac /etc/group | grep -v ":$original_group:" | head -1 | cut
> -d: -f3`
> +other_owner=`tac /etc/passwd | grep -v ":$original_group:" | head -1 | cut
> -d: -f3
From: "Steven Rostedt (Google)"
As there were bugs found with the ownership of eventfs dynamic file
creation. Add a test to test it.
It will remount tracefs with a different gid and check the ownership of
the eventfs directory, as well as the system and event directories. It
will also check the
Hi Steve,
On Thu, 21 Dec 2023 19:45:16 -0500
Steven Rostedt wrote:
> From: "Steven Rostedt (Google)"
>
> As there were bugs found with the ownership of eventfs dynamic file
> creation. Add a test to test it.
>
> It will remount tracefs with a different gid and check the ownership of
> the eve
On Fri, 22 Dec 2023 10:21:48 +0900
Masami Hiramatsu (Google) wrote:
> The testcase itself is OK but is there any way to identify the system
> supports eventfs or not? I ran this test on v6.5.13 for checking then
> it failed. We may need to skip (unsupported) this test for such case.
Hmm, honestl
On Thu, 21 Dec 2023 20:28:13 -0500
Steven Rostedt wrote:
> On Fri, 22 Dec 2023 10:21:48 +0900
> Masami Hiramatsu (Google) wrote:
>
> > The testcase itself is OK but is there any way to identify the system
> > supports eventfs or not? I ran this test on v6.5.13 for checking then
> > it failed. W
On Thu, 21 Dec 2023 20:28:13 -0500
Steven Rostedt wrote:
> On Fri, 22 Dec 2023 10:21:48 +0900
> Masami Hiramatsu (Google) wrote:
>
> > The testcase itself is OK but is there any way to identify the system
> > supports eventfs or not? I ran this test on v6.5.13 for checking then
> > it failed. W
On Fri, 22 Dec 2023 10:48:41 +0900
Masami Hiramatsu (Google) wrote:
> On Thu, 21 Dec 2023 20:28:13 -0500
> Steven Rostedt wrote:
>
> > On Fri, 22 Dec 2023 10:21:48 +0900
> > Masami Hiramatsu (Google) wrote:
> >
> > > The testcase itself is OK but is there any way to identify the system
> > >
Note that there's another post-merge fix for TCP-AO selftests, but that
doesn't conflict with these, so I don't resend that:
https://lore.kernel.org/all/20231219-b4-tcp-ao-selftests-out-of-tree-v1-1-0fff92d26...@arista.com/T/#u
Signed-off-by: Dmitry Safonov
---
Dmitry Safonov (2):
selftest
In unsigned-md5 selftests ip_route_add() is not needed in
client_add_ip(): the route was pre-setup in __test_init() => link_init()
for subnet, rather than a specific ip-address.
Currently, __ip_route_add() mistakenly always sets VRF table
to RT_TABLE_MAIN - this seems to have sneaked in during uns
Since commit f5769faeec36 ("net: Namespace-ify sysctl_optmem_max")
optmem_max is per-netns, so need of switching to root namespace.
It seems trivial to keep the old logic working, so going to keep it for
a while (at least, until kernel with netns-optmem_max will be release).
Currently, there is a
On Fri, 22 Dec 2023 10:52:00 +0900
Masami Hiramatsu (Google) wrote:
> > +instance="foo-$(mktemp -u X)"
> >
> > This doesn't work. it needs XX (6 times X). And this is
> > somewhat wrong usage of mktemp because it can not check there is
> > foo-.
> > What about change it as
> >
> > insta
On Fri, 22 Dec 2023 10:52:00 +0900
Masami Hiramatsu (Google) wrote:
> On Fri, 22 Dec 2023 10:48:41 +0900
> Masami Hiramatsu (Google) wrote:
> And I confirmed that this test passed on v6.5.13 with that change.
>
I just ran it on 6.5.13 and it took *forever*!
But I do have a bit of debug, and
From: "Steven Rostedt (Google)"
As there were bugs found with the ownership of eventfs dynamic file
creation. Add a test to test it.
It will remount tracefs with a different gid and check the ownership of
the eventfs directory, as well as the system and event directories. It
will also check the
Shuah,
This patch has no dependencies. You can take it through your tree for the
next merge window if you want. If not, I can take it.
Thanks,
-- Steve
On Thu, 21 Dec 2023 21:12:29 -0500
Steven Rostedt wrote:
> From: "Steven Rostedt (Google)"
>
> As there were bugs found with the ownersh
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> From: Lu Baolu
>
> The updates of the PTEs in the nested page table will be propagated to the
> hardware caches on both IOMMU (IOTLB) and devices (DevTLB/ATC).
this is incorrect. the scope of this cmd is driver specific.
>
>
On Thu, 21 Dec 2023 21:12:29 -0500
Steven Rostedt wrote:
> From: "Steven Rostedt (Google)"
>
> As there were bugs found with the ownership of eventfs dynamic file
> creation. Add a test to test it.
>
> It will remount tracefs with a different gid and check the ownership of
> the eventfs direct
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
> +
> +int iommufd_hwpt_invalidate(struct iommufd_ucmd *ucmd)
> +{
> + struct iommu_hwpt_invalidate *cmd = ucmd->cmd;
> + struct iommu_user_data_array data_array = {
> + .type = cmd->req_type,
> + .uptr
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> From: Nicolin Chen
>
> Wrap up the data pointer/num sanity and __iommu_copy_struct_from_user
> call for iommu drivers to copy driver specific data at a specific location
> in the struct iommu_user_data_array, and
> iommu_respond
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
> +
> +
> + if ((inv.flags & IOMMU_TEST_INVALIDATE_FLAG_ALL) &&
> + (inv.flags &
> IOMMU_TEST_INVALIDATE_FLAG_TRIGGER_ERROR)) {
> + rc = -EINVAL;
> + break;
> +
On 12/21/2023 11:39 PM, Yi Liu wrote:
From: Lu Baolu
This implements the .cache_invalidate_user() callback to support iotlb
flush for nested domain.
Signed-off-by: Lu Baolu
Co-developed-by: Yi Liu
Signed-off-by: Yi Liu
---
drivers/iommu/intel/nested.c | 116 +++
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> From: Nicolin Chen
>
> Allow to test whether IOTLB has been invalidated or not.
>
> Signed-off-by: Nicolin Chen
> Signed-off-by: Yi Liu
Reviewed-by: Kevin Tian
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> From: Nicolin Chen
>
> Add test cases for the IOMMU_HWPT_INVALIDATE ioctl and verify it by using
> the new IOMMU_TEST_OP_MD_CHECK_IOTLB.
>
> Signed-off-by: Nicolin Chen
> Co-developed-by: Yi Liu
> Signed-off-by: Yi Liu
over
On Thu, 21 Dec 2023 21:07:57 -0500
Steven Rostedt wrote:
> On Fri, 22 Dec 2023 10:52:00 +0900
> Masami Hiramatsu (Google) wrote:
>
> > On Fri, 22 Dec 2023 10:48:41 +0900
> > Masami Hiramatsu (Google) wrote:
>
> > And I confirmed that this test passed on v6.5.13 with that change.
> >
>
> I j
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> + fault &= DMA_FSTS_IQE | DMA_FSTS_ITE | DMA_FSTS_ICE;
> + if (fault) {
> + if (fsts)
> + *fsts |= fault;
do we expect the fault to be accumulated? otherwise it's clearer to
just do direct a
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> From: Lu Baolu
>
> This makes the pasid based cache invalidation to return QI faults to callers.
it's not just for pasid cache invalidation, e.g. there is also change on
qi_flush_dev_iotlb().
> This is needed when usersapce wa
> From: Yang, Weijiang
> Sent: Friday, December 22, 2023 11:56 AM
> > +
> > + xa_for_each(&domain->iommu_array, i, info) {
> > + nested_flush_pasid_iotlb(info->iommu, domain, addr,
> npages, 0);
> > +
> > + if (domain->has_iotlb_device)
> > + continue;
>
>
On Fri, Dec 22, 2023 at 01:59:05AM +, Dmitry Safonov wrote:
> Note that there's another post-merge fix for TCP-AO selftests, but that
> doesn't conflict with these, so I don't resend that:
>
> https://lore.kernel.org/all/20231219-b4-tcp-ao-selftests-out-of-tree-v1-1-0fff92d26...@arista.com/T/#
> From: Liu, Yi L
> Sent: Thursday, December 21, 2023 11:40 PM
>
> +
> +static void intel_nested_flush_cache(struct dmar_domain *domain, u64
> addr,
> + unsigned long npages, u32 *error)
> +{
> + struct iommu_domain_info *info;
> + unsigned long i;
> +
> On Dec 22, 2023, at 11:56, Yang, Weijiang wrote:
>
> On 12/21/2023 11:39 PM, Yi Liu wrote:
>> From: Lu Baolu
>>
>> This implements the .cache_invalidate_user() callback to support iotlb
>> flush for nested domain.
>>
>> Signed-off-by: Lu Baolu
>> Co-developed-by: Yi Liu
>> Signed-off-by:
> On Dec 22, 2023, at 14:47, Tian, Kevin wrote:
>
>
>>
>> From: Yang, Weijiang
>> Sent: Friday, December 22, 2023 11:56 AM
>>> +
>>> +xa_for_each(&domain->iommu_array, i, info) {
>>> +nested_flush_pasid_iotlb(info->iommu, domain, addr,
>> npages, 0);
>>> +
>>> +if (domain
> From: Liu, Yi L
> Sent: Friday, December 22, 2023 3:02 PM
>
>
> > On Dec 22, 2023, at 14:47, Tian, Kevin wrote:
> >
> >
> >>
> >> From: Yang, Weijiang
> >> Sent: Friday, December 22, 2023 11:56 AM
> >>> +
> >>> +xa_for_each(&domain->iommu_array, i, info) {
> >>> +nested_flush_pa
61 matches
Mail list logo
