Hi Abner,
REST as a more generic network API and could possibly be placed in NetworkPkg.
My concerns are:
Is REST in UEFI going to be HTTP-based or is it planned to be tied to drivers
outside of NetworkPkg?
If it is going to be more generic, I do not think that it is a
good idea to tie it with other network drivers within NetworkPkg
Is REST currently consumed only by Redfish or is going to be extensively used
outside that package?
Even if it is HTTP-based, question of maintenance arises. If Redfish is the
only consumer of REST API, I would leave it within RedfishPkg so that
Redfish+REST could be treated as a whole solution by the package maintainer.
In case REST is planned to be used outside of RedfishPkg, it could be added to
NetworkPkg with some extra work.
I believe that, by default, REST driver build will be controlled by a separate
flag and disabled by default (no need for it in regular network stack use cases
like PXE/iSCSI/HTTP boot).
Additionally, if REST DXE build is requested, one would have to ensure that
dependant drivers like HttpDxe will be included and built within NetworkPkg and
outside of it (by consumers like OvmfPkg).
For example: there is a preprocessor mechanism used in
NetworkPkg/NetworkDefines.dsc.inc to throw a compiler error in case TLS is not
included if HTTPS support is requested.
Jiaxin,
What do you think?
Thanks,
Maciej
<Please disregard the legal disclaimer below. I'm working on it.>
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Abner Chang
Sent: Thursday, November 14, 2019 17:10
To: devel@edk2.groups.io
Subject: [edk2-devel] Which is the proper edk2 package the EFI REST Structure
DXE driver should stay with?
Hi all,
I would like to get your suggestion with regard to the suitable edk2 package
for EFI REST Structure DXE driver. We have POC code for this protocol (refer to
UEFI spec 2.8, section 29.7.3) and currently this driver is implemented in
RedfishPkg. However, this driver is not for Redfish only. Either NetowrkPkg or
MdeModulePkg is better than RedfishPkg. I prefer to put this driver in
NetworkPkg, but it's fine to me if you have other opinions.
Thanks
Abner
--------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Slowackiego 173 | 80-298 Gdansk | Sad Rejonowy Gdansk Polnoc | VII Wydzial
Gospodarczy Krajowego Rejestru Sadowego - KRS 101882 | NIP 957-07-52-316 |
Kapital zakladowy 200.000 PLN.
Ta wiadomosc wraz z zalacznikami jest przeznaczona dla okreslonego adresata i
moze zawierac informacje poufne. W razie przypadkowego otrzymania tej
wiadomosci, prosimy o powiadomienie nadawcy oraz trwale jej usuniecie;
jakiekolwiek
przegladanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole
use of the intended recipient(s). If you are not the intended recipient, please
contact the sender and delete all copies; any review or distribution by
others is strictly prohibited.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#50828): https://edk2.groups.io/g/devel/message/50828
Mute This Topic: https://groups.io/mt/57795634/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
