From: Huajing Li
Change "dmpstore" to show name of known variable vendor GUID.
The name is got from ShellProtocol.GetGuidName().
Cc: Jaben Carsey
Reviewed-by: Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Huajing Li
---
ShellPkg/Library/UefiShellDebug1Comman
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Guo Mang
---
Vlv2TbltDevicePkg/PlatformPkgConfig.dsc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Vlv2TbltDevicePkg/PlatformPkgConfig.dsc
b/Vlv2TbltDevicePkg/PlatformPkgConfig.dsc
index f172f8f..b6df2a
Reviewed-by: Liming Gao
>-Original Message-
>From: Leif Lindholm [mailto:leif.lindh...@linaro.org]
>Sent: Saturday, September 16, 2017 3:34 PM
>To: Ard Biesheuvel
>Cc: edk2-devel@lists.01.org; Gao, Liming ; Zhu,
>Yonghong
>Subject: Re: [PATCH] BaseTools/tools_def AARCH64: enable frame p
Hi karunakar,
You can verify the DNS device path with HTTP boot feature. After the successful
HTTP boot DNS parsing, the device path should be like:
//../Mac(...)[/Vlan(...)][/Wi-Fi(...)]/IPv4(...)[/Dns(...)]/Uri(...).
That is recommend way for the verification.
Besides, you can also draf
Paulo,
Could you please paste a "map -r" output on a CDROM which
contains Eltorito volume?
I want to confirm that the result is expected.
Thanks/Ray
> -Original Message-
> From: Paulo Alcantara [mailto:pca...@zytor.com]
> Sent: Sunday, September 17, 2017 9:13 PM
> To: edk2-devel@lists.01.
On 17 September 2017 at 21:07, Udit Kumar wrote:
> Hi Ard,
>
>> Thanks for the patch. But please consider carefully what you're doing:
>> My point is really that you can just remove lines 124 - 189 instead.
>
> We want, OS to re-use firmware volume area. Therefore making firmware volume
> visibl
We met a case that use two microcode files in the Microcode.inf file,
one is .mcb file, another is .txt file. then it cause build failure
because the SourceFileList include the .txt file's output file, while
this output file is still not be generated, so it cause
GetFileDependency report failure.
Hi Jiaxin,
Thank you very much for your info, Yes it works fine for manual configuration.
And also could you please provide steps to verify "Add DNS device path node"
feature.
Thanks,
karunakar
-Original Message-
From: Wu, Jiaxin [mailto:jiaxin...@intel.com]
Sent: Monday, September 1
Hi Ard,
> Thanks for the patch. But please consider carefully what you're doing:
> My point is really that you can just remove lines 124 - 189 instead.
We want, OS to re-use firmware volume area. Therefore making firmware volume
visible.
You are right, removing lines 124-189 will do the same
Hi Ting Ye,
Thanks for your reply.
[Here the bug details]
Bug 710 - IPv6 support condition check for HTTP and ISCSI
https://bugzilla.tianocore.org/show_bug.cgi?id=710
Thanks,
karunakar
-Original Message-
From: Ye, Ting [mailto:ting...@intel.com]
Sent: Monday, September 18, 2017 8:23 AM
>From CpuDxe driver perspective, it doesn't update GCD memory attributes
from current page table setup during its initialization. So the memory
attributes in GCD might not reflect all memory attributes in real world.
Cc: Jiewen Yao
Cc: Star Zeng
Cc: Laszlo Ersek
Cc: Michael Kinney
Suggested-b
>From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept
page related attributes. That means users cannot use it to change page
attributes, and have to turn to CPU arch protocol to do it, which is not
be allowed by PI spec.
Cc: Jiewen Yao
Cc: Star Zeng
Cc: Laszlo Ersek
Cc: Mi
There're two issues here actually.
>From GCD perspective, its SetMemorySpaceAttributes() method doesn't accept
page related attributes. That means users cannot use it to change page
attributes, and have to turn to CPU arch protocol to do it, which is not
be allowed by PI spec.
>From CpuDxe driver
Hi Karunakar,
Thanks for capturing this. We agree that it's reasonable to add similar check
to HTTP and ISCSI. Could you please help to file a bugzilla tracker for this?
Thanks,
Ting Ye
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
Karunakar
Hi Karunakar,
According the UEFI Spec, the Ip4Config2DataTypeManualAddress,
Ip4Config2DataTypeGateway and Ip4Config2DataTypeDnsServer configuration data
are not
allowed to set via SetData() if the policy is DHCP. So, the clear feature is
only for the manual configuration. This is our design pu
Reviewed-by: Chasel Chiu
-Original Message-
From: Zeng, Star
Sent: Thursday, September 14, 2017 7:29 PM
To: edk2-devel@lists.01.org
Cc: Zeng, Star ; Yao, Jiewen ; Chiu,
Chasel
Subject: [PATCH] MdeModulePkg SmbiosMeasurementDxe: Skip measurement for OEM
type
The generic driver has no
Paulo,
I checked carefully of patch #1, 50% carefully of #2, 20% carefully of #3.
I could only give comments from the EDKII coding style perspective.
I do provide some other comments, but please understand they are from
a person that knows very little about UDF. (I know the concept of Volume.
But j
Paulo,
With the change in partition driver, I suppose UdfDxe driver
only needs to take care of area covered by the partition descriptor.
But why StartMainVolumeDescriptorSequence() still reads
LVD, TD, and PD?
I thought the UdfDxe driver's logic can be simplified a lot.
There should be no duplicat
Paulo,
Several comments:
1. Can below logic be removed in PartitionDxe/Udf.c?
while (!IsDevicePathEnd (DevicePathNode)) {
//
// Do not allow checking for UDF file systems in CDROM "El Torito"
// partitions, and skip duplicate installation of UDF file system child
// nodes.
//
Continue reading the patch #2, I think we can change IS_PD to:
#define IS_UDF_PD(Tag) ((Tag)->TagIdentifier == 5)
Using the above way, we can avoid caller to supply an invalid buffer.
Thanks/Ray
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
#define _GET_TAG_ID(_Pointer) \
(((UDF_DESCRIPTOR_TAG *)(_Pointer))->TagIdentifier)
#define IS_PD(_Pointer) \
((BOOLEAN)(_GET_TAG_ID (_Pointer) == 5))
#define IS_LVD(_Pointer) \
((BOOLEAN)(_GET_TAG_ID (_Pointer) == 6))
#define IS_TD(_Pointer) \
((BOOLEAN)(_GET_TAG_ID (_Pointer) == 8))
#de
Hi Jiewen,
On 16/09/2017 20:52, Yao, Jiewen wrote:
Thank you Paulo, to provide a fix for this driver.
No problem! I'm trying to do the best I can in my very short time.
I do not have comment for this specific patch. I would defer the review
work to Star and Ruiyu.
OK.
I do have some g
Thank you Laszlo. You analysis is great!
Thank you
Yao Jiewen
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo
Ersek
Sent: Sunday, September 17, 2017 6:07 PM
To: Yao, Jiewen ; Paulo Alcantara
Cc: Ni, Ruiyu ; Dong, Eric ;
edk2-devel@lists.01.org; Gao, Liming ; Kinne
Do not use entire block device size for the UDF logical partition,
instead reserve the appropriate space (LVD space) for it.
Cc: Eric Dong
Cc: Ruiyu Ni
Cc: Star Zeng
Cc: Laszlo Ersek
Reported-by: Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Paulo Alcantara
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=707
Hi,
This patchset fixes a bug in Partition driver that UDF partitions
occupied the entire disk space instead of using LVD space only.
BTW, I've only tested it under OVMF and built it with GCC only. That
would be appreciable if someone could
This patch reworks the driver to support Partition driver changes.
Cc: Eric Dong
Cc: Paulo Alcantara
Cc: Ruiyu Ni
Cc: Star Zeng
Cc: Laszlo Ersek
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Paulo Alcantara
---
MdeModulePkg/Universal/Disk/UdfDxe/File.c
This patch adds a fewe more UDF structures in order to detect Logical
Volume and Partition descriptors during Main Volume Descriptor Sequence
in Partition driver.
Cc: Michael D Kinney
Cc: Liming Gao
Cc: Laszlo Ersek
Cc: Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-of
Hi Jiewen,
I agree these are important questions; even earlier I noticed the
following remark in "PartitionDxe.inf":
# Caution: This module requires additional review when modified.
# This driver will have external input - disk partition.
# This external input must be validated carefully to av
28 matches
Mail list logo