Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

xsk: fix destination buffer address when copying with metadata #3950

Closed
wants to merge 1 commit into from

Conversation

kernel-patches-bot
Copy link

Pull request for series with
subject: xsk: fix destination buffer address when copying with metadata
version: 1
url: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907

@kernel-patches-bot
Copy link
Author

Upstream branch: 07ec7b5
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907
version: 1

@kernel-patches-bot
Copy link
Author

Upstream branch: aa3496a
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907
version: 1

@kernel-patches-bot
Copy link
Author

Upstream branch: af085f5
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907
version: 1

@kernel-patches-bot
Copy link
Author

Upstream branch: db55911
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907
version: 1

@kernel-patches-bot
Copy link
Author

Upstream branch: f71b2f6
series: https://patchwork.kernel.org/project/netdevbpf/list/?series=691907
version: 1

While working on a simplified test for [0] it occurred to me that
the following looks fishy:

	data = xsk_umem__get_data(xsk->umem_area, rx_desc->addr);
	data_meta = data - sizeof(my metadata);

Since the data points to umem frame at addr X, data_mem points to
the end of umem frame X-1.

I don't think it's by design?

0: https://lore.kernel.org/bpf/20221028181431.05173968@kernel.org/T/#t

Cc: Björn Töpel <bjorn@kernel.org>
Cc: Magnus Karlsson <magnus.karlsson@intel.com>
Cc: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
Cc: Jonathan Lemon <jonathan.lemon@gmail.com>
Signed-off-by: Stanislav Fomichev <sdf@google.com>
@kernel-patches-bot
Copy link
Author

At least one diff in series https://patchwork.kernel.org/project/netdevbpf/list/?series=691907 expired. Closing PR.

@kernel-patches-bot kernel-patches-bot added rfc and removed new labels Nov 4, 2022
@kernel-patches-bot kernel-patches-bot deleted the series/691907=>bpf-next branch November 6, 2022 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants