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

examples: Added an example of a problematic patch with an explanation #500

Merged
merged 1 commit into from Oct 1, 2015

Conversation

euspectre
Copy link
Contributor

examples/tcp_cubic-better-follow-cubic-curve-original.patch is the
original patch, combined from two mainline commits (see the description
in the patch). It cannot be used with Kpatch as it is because the
change is in the initialization of a global structure.

examples/tcp_cubic-better-follow-cubic-curve-converted.patch is a
modification of the patch that Kpatch can process. Still, this
modification has its issues, see the description there.

examples/tcp_cubic-better-follow-cubic-curve-original.patch is the
original patch, combined from two mainline commits (see the description
in the patch). It cannot be used with Kpatch as it is because the
change is in the initialization of a global structure.

examples/tcp_cubic-better-follow-cubic-curve-converted.patch is a
modification of the patch that Kpatch can process. Still, this
modification has its issues, see the description there.

Signed-off-by: Evgenii Shatokhin <eshatokhin@odin.com>
@jpoimboe
Copy link
Member

@euspectre Thanks for starting this! and congrats on opening our 500th PR :)

👍

@euspectre
Copy link
Contributor Author

:-) Yes, let us start from here.

sjenning added a commit that referenced this pull request Oct 1, 2015
examples: Added an example of a problematic patch with an explanation
@sjenning sjenning merged commit f049dcf into dynup:master Oct 1, 2015
@euspectre euspectre deleted the examples branch November 23, 2015 12:38
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

3 participants