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

Revert "Helper reallocate packet (#3110)" #3263

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

shankarseal
Copy link
Collaborator

@shankarseal shankarseal commented Feb 14, 2024

This reverts commit 13eed74.

Description

#3110 breaks existing extensions as it makes a change in public data struct ebpf_program_info_t. Specifically, extension store export routine and extension binding to eBPF core will be broken.

Testing

CI/CD will cover the testing.

Documentation

Revert list contains a MD file as well.

Installation

No.

@shankarseal shankarseal added this pull request to the merge queue Feb 15, 2024
Merged via the queue into microsoft:main with commit 66087c5 Feb 15, 2024
76 checks passed
@shankarseal shankarseal deleted the revert_3110 branch February 15, 2024 02:49
@shpalani
Copy link
Collaborator

Can you create an issue and attach to this PR?

shpalani pushed a commit to shpalani/ebpf-for-windows that referenced this pull request Feb 15, 2024
shpalani pushed a commit to shpalani/ebpf-for-windows that referenced this pull request Feb 15, 2024
@shpalani shpalani mentioned this pull request Feb 15, 2024
shpalani added a commit that referenced this pull request Feb 16, 2024
* fix check for static initialization of hash-of-maps key length. (#3260)

* fix check for static initialization of hash-of-maps key length.

* Delete tests/bpf2c_tests/expected/xdp_adjust_head_unsafe_dll.c

* Delete tests/bpf2c_tests/expected/xdp_adjust_head_unsafe_raw.c

* Delete tests/bpf2c_tests/expected/xdp_adjust_head_unsafe_sys.c

* fix build failure

---------

Co-authored-by: Anurag Saxena <anusa@microsoft.com>

* Revert "Helper reallocate packet (#3110)" (#3263)

This reverts commit 13eed74.

* bump version to 0.14.1

* fix test

---------

Co-authored-by: Shankar Seal <74580197+shankarseal@users.noreply.github.com>
Co-authored-by: Anurag Saxena <anusa@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants