Skip to content

NetworkPkg: Add PCD for HTTP transfer buffer size #11007

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

NetworkPkg: Add PCD for HTTP transfer buffer size #11007

wants to merge 1 commit into from
+22 −5

Conversation

baranee
Copy link
Contributor

@baranee baranee commented Apr 25, 2025

Description

Current HTTP transfer size have a default value of 2M.
This PCD change allows the platform code to change the transfer buffer size.

  • Breaking change?
    • Breaking change - Does this PR cause a break in build or boot behavior?
    • Examples: Does it add a new library class or move a module to a different repo.
  • Impacts security?
    • Security - Does this PR have a direct security impact?
    • Examples: Crypto algorithm change or buffer overflow fix.
  • Includes tests?
    • Tests - Does this PR include any explicit test code?
    • Examples: Unit tests or integration tests.

How This Was Tested

Verified basic HTTP file download still works.

Integration Instructions

N/A

@baranee
NetworkPkg: Add PCD for HTTP transfer buffer size
a2f7fd0 
Current HTTP transfer size have a default value of 2M.
This PCD change allows the platform code to change
the transfer buffer size.

Signed-off-by: Anbazhagan Baraneedharan <[email protected]>
@baranee baranee marked this pull request as ready for review April 25, 2025 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajfish ajfish Awaiting requested review from ajfish

@ardbiesheuvel ardbiesheuvel Awaiting requested review from ardbiesheuvel

@leiflindholm leiflindholm Awaiting requested review from leiflindholm

@makubacki makubacki Awaiting requested review from makubacki

@mdkinney mdkinney Awaiting requested review from mdkinney

@SaloniKasbekar SaloniKasbekar Awaiting requested review from SaloniKasbekar

@Zclarkwilliams Zclarkwilliams Awaiting requested review from Zclarkwilliams

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@baranee