public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Jiaxin Wu <Jiaxin.wu@intel.com>
To: edk2-devel@lists.01.org
Cc: Ye Ting <ting.ye@intel.com>, Fu Siyuan <siyuan.fu@intel.com>,
	Carsey Jaben <jaben.carsey@intel.com>,
	Shao Ming <ming.shao@intel.com>, Laszlo Ersek <lersek@redhat.com>,
	Wu Jiaxin <jiaxin.wu@intel.com>
Subject: [PATCH v2 0/5] Support windowsize to benefit tftp/pxe download performance.
Date: Tue, 25 Sep 2018 09:11:43 +0800	[thread overview]
Message-ID: <20180925011145.14316-1-Jiaxin.wu@intel.com> (raw)

*v2: The first three patches(1/2/3) are the same with version 1, just update the last two
patches (4/5):
I) <MdeModulePkg/MdeModulePkg.dec: Define one PCD for PXE to specify MTFTP windowsize.>
This patch has been discarded since we rename and redefine the PCD in NetworkPkg instead 
of MdeModulePkg. The replacement is: 
[PATCH v2 4/5] NetworkPkg: Define one PCD for PXE to specify MTFTP windowsize.
II) <NetworkPkg/UefiPxeBcDxe: Use the specified MTFTP windowsize.>
Since the new PCD (PcdPxeTftpWindowSize) was renamed/defined in NetworkPkg instead of 
MdeModulePkg, we udpate the consuming PXE driver. The new version patch is:
[PATCH v2 5/5] NetworkPkg/UefiPxeBcDxe: Use the specified MTFTP windowsize.

Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=886

The series patches are to support the TFTP windowsize option described in RFC 7440.
TFTP shell command and UEFI PXE driver will use the feature to benefit the download 
performance.

Cc: Ye Ting <ting.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Cc: Carsey Jaben <jaben.carsey@intel.com>
Cc: Shao Ming <ming.shao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com>

Jiaxin Wu (5):
  MdeModulePke/Mtftp4Dxe: Support windowsize in read request operation.
  NetworkPkg/Mtftp6Dxe: Support windowsize in read request operation.
  ShellPkg/TftpDynamicCommand: Add one option for tftp command to
    specify windowsize.
  NetworkPkg: Define one PCD for PXE to specify MTFTP windowsize.
  NetworkPkg/UefiPxeBcDxe: Use the specified MTFTP windowsize.

 .../Universal/Network/Mtftp4Dxe/Mtftp4Impl.c  |   5 +
 .../Universal/Network/Mtftp4Dxe/Mtftp4Impl.h  |  10 ++
 .../Network/Mtftp4Dxe/Mtftp4Option.c          |  25 +++-
 .../Network/Mtftp4Dxe/Mtftp4Option.h          |   8 +-
 .../Universal/Network/Mtftp4Dxe/Mtftp4Rrq.c   |  55 +++++--
 .../Network/Mtftp4Dxe/Mtftp4Support.c         |   8 +-
 .../Network/Mtftp4Dxe/Mtftp4Support.h         |  13 --
 .../Universal/Network/Mtftp4Dxe/Mtftp4Wrq.c   |   2 +-
 NetworkPkg/Mtftp6Dxe/Mtftp6Impl.h             |  13 +-
 NetworkPkg/Mtftp6Dxe/Mtftp6Option.c           |  22 ++-
 NetworkPkg/Mtftp6Dxe/Mtftp6Option.h           |  14 +-
 NetworkPkg/Mtftp6Dxe/Mtftp6Rrq.c              |  53 +++++--
 NetworkPkg/Mtftp6Dxe/Mtftp6Support.c          |  10 ++
 NetworkPkg/Mtftp6Dxe/Mtftp6Wrq.c              |   2 +-
 NetworkPkg/NetworkPkg.dec                     |   6 +
 NetworkPkg/NetworkPkg.uni                     |   6 +
 NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c           |  10 +-
 NetworkPkg/UefiPxeBcDxe/PxeBcMtftp.c          | 137 +++++++++++++-----
 NetworkPkg/UefiPxeBcDxe/PxeBcMtftp.h          |   6 +-
 NetworkPkg/UefiPxeBcDxe/UefiPxeBcDxe.inf      |   3 +
 .../DynamicCommand/TftpDynamicCommand/Tftp.c  |  65 +++++++--
 .../TftpDynamicCommand/Tftp.uni               |   6 +-
 22 files changed, 371 insertions(+), 108 deletions(-)

-- 
2.17.1.windows.2



             reply	other threads:[~2018-09-25  1:12 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-25  1:11 Jiaxin Wu [this message]
2018-09-25  1:11 ` [PATCH v2 4/5] NetworkPkg: Define one PCD for PXE to specify MTFTP windowsize Jiaxin Wu
2018-09-25 20:12   ` Laszlo Ersek
2018-09-26  0:52     ` Wu, Jiaxin
2018-09-25  1:11 ` [PATCH v2 5/5] NetworkPkg/UefiPxeBcDxe: Use the specified " Jiaxin Wu
2018-09-25 20:23   ` Laszlo Ersek
2018-09-25  1:40 ` [PATCH v2 0/5] Support windowsize to benefit tftp/pxe download performance Fu, Siyuan
2018-09-25  8:40 ` Ye, Ting
2018-09-25 20:07 ` Laszlo Ersek
2018-09-26  0:48   ` Wu, Jiaxin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180925011145.14316-1-Jiaxin.wu@intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox