Date   

Re: [PATCH v2 000/105] .mailmap: Add mailmap file to have a cleaner git history

Philippe Mathieu-Daudé <philmd@...>
 

On 12/6/19 12:24 PM, Philippe Mathieu-Daude wrote:
[...]
- Another note regarding groups.io 40 posts per 30min limit -
My previous big series [5] involved 79 patches, and groups.io banned
me for some time after the 40th post. On [5] Laszlo suggested to test
the following git-send-email options to see if we can work with big
series and groups.io:
--batch-size=<num>
Some email servers (e.g. smtp.163.com) limit the number
emails to be sent per session (connection) and this will
lead to a failure when sending many messages. With this
option, send-email will disconnect after sending $<num>
messages and wait for a few seconds (see --relogin-delay)
and reconnect, to work around such a limit. You may want
to use some form of credential helper to avoid having to
retype your password every time this happens. Defaults to
the sendemail.smtpBatchSize configuration variable.
--relogin-delay=<int>
Waiting $<int> seconds before reconnecting to SMTP server.
Used together with --batch-size option. Defaults to the
sendemail.smtpReloginDelay configuration variable.
So I'm sending this series using git-publish [7] after setting the
following values:
$ git config sendemail.smtpBatchSize 33
$ git config sendemail.smtpReloginDelay 1848
Replying to self after 66 in an hour to verify that I'm not banned ... :)


Re: [PATCH edk2-platforms 2/2] Platform/ARM/ArmJunoDxe: use PciIo protocol notify to program the MAC

Laszlo Ersek
 

On 12/06/19 12:02, Ard Biesheuvel wrote:
Instead of connecting and thus enumerating the PCIe topology in a
platform driver, just so that we can grab the PciIo protocol that
belongs to the Marvell Yukon NIC and program its MAC address, rely
on a protocol notification handler to do this whenever the core code
decides to enumerate the PCIe.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@...>
---
Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 161 ++++----------------
Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf | 1 -
2 files changed, 30 insertions(+), 132 deletions(-)

diff --git a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
index c0ad7ced2959..ebaf2aa134da 100644
--- a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
+++ b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c
@@ -32,39 +32,8 @@
STATIC CONST EFI_GUID mJunoAcpiTableFile = { 0xa1dd808e, 0x1e95, 0x4399, { 0xab, 0xc0, 0x65, 0x3c, 0x82, 0xe8, 0x53, 0x0c } };
#endif

-typedef struct {
- ACPI_HID_DEVICE_PATH AcpiDevicePath;
- PCI_DEVICE_PATH PciDevicePath;
- EFI_DEVICE_PATH_PROTOCOL EndDevicePath;
-} EFI_PCI_ROOT_BRIDGE_DEVICE_PATH;
-
-STATIC CONST EFI_PCI_ROOT_BRIDGE_DEVICE_PATH mPciRootComplexDevicePath = {
- {
- { ACPI_DEVICE_PATH,
- ACPI_DP,
- { (UINT8) (sizeof (ACPI_HID_DEVICE_PATH)),
- (UINT8) ((sizeof (ACPI_HID_DEVICE_PATH)) >> 8) }
- },
- EISA_PNP_ID (0x0A03),
- 0
- },
- {
- { HARDWARE_DEVICE_PATH,
- HW_PCI_DP,
- { (UINT8) (sizeof (PCI_DEVICE_PATH)),
- (UINT8) ((sizeof (PCI_DEVICE_PATH)) >> 8) }
- },
- 0,
- 0
- },
- {
- END_DEVICE_PATH_TYPE,
- END_ENTIRE_DEVICE_PATH_SUBTYPE,
- { END_DEVICE_PATH_LENGTH, 0 }
- }
-};
-
STATIC EFI_EVENT mAcpiRegistration = NULL;
+STATIC EFI_EVENT mPciIoNotificationRegistration = NULL;
(1) Please use "VOID *" as the type of "mPciIoNotificationRegistration".

EFI_EVENT happens to work (because it is, regrettably, a typedef, per
spec, to "VOID *").

But semantically we need "VOID *" here -- please see the "Registration"
parameter of EFI_BOOT_SERVICES.RegisterProtocolNotify() and
EFI_BOOT_SERVICES.LocateProtocol().


/**
This function reads PCI ID of the controller.
@@ -99,59 +68,6 @@ ReadMarvellYoukonPciId (
return EFI_SUCCESS;
}

-/**
- This function searches for Marvell Yukon NIC on the Juno
- platform and returns PCI IO protocol handle for the controller.
-
- @param[out] PciIo PCI IO protocol handle
-**/
-STATIC
-EFI_STATUS
-GetMarvellYukonPciIoProtocol (
- OUT EFI_PCI_IO_PROTOCOL **PciIo
- )
-{
- UINTN HandleCount;
- EFI_HANDLE *HandleBuffer;
- UINTN HIndex;
- EFI_STATUS Status;
-
- Status = gBS->LocateHandleBuffer (
- ByProtocol,
- &gEfiPciIoProtocolGuid,
- NULL,
- &HandleCount,
- &HandleBuffer);
- if (EFI_ERROR (Status)) {
- return (Status);
- }
-
- for (HIndex = 0; HIndex < HandleCount; ++HIndex) {
- // If PciIo opened with EFI_OPEN_PROTOCOL_GET_PROTOCOL, the CloseProtocol() is not required
- Status = gBS->OpenProtocol (
- HandleBuffer[HIndex],
- &gEfiPciIoProtocolGuid,
- (VOID **) PciIo,
- NULL,
- NULL,
- EFI_OPEN_PROTOCOL_GET_PROTOCOL);
- if (EFI_ERROR (Status)) {
- continue;
- }
-
- Status = ReadMarvellYoukonPciId (*PciIo, JUNO_MARVELL_YUKON_ID);
- if (EFI_ERROR (Status)) {
- continue;
- } else {
- break;
- }
- }
-
- gBS->FreePool (HandleBuffer);
-
- return Status;
-}
-
/**
This function restore the original controller attributes

@@ -326,18 +242,14 @@ WriteMacAddress (
**/
STATIC
EFI_STATUS
-ArmJunoSetNicMacAddress ()
+ArmJunoSetNicMacAddress (
+ IN EFI_PCI_IO_PROTOCOL *PciIo
+ )
{
UINT64 OldPciAttr;
- EFI_PCI_IO_PROTOCOL* PciIo;
UINT32 PciRegBase;
EFI_STATUS Status;

- Status = GetMarvellYukonPciIoProtocol (&PciIo);
- if (EFI_ERROR (Status)) {
- return Status;
- }
-
PciRegBase = 0;
Status = InitPciDev (PciIo, &PciRegBase, &OldPciAttr);
if (EFI_ERROR (Status)) {
@@ -352,14 +264,8 @@ ArmJunoSetNicMacAddress ()
}

/**
- Notification function of the event defined as belonging to the
- EFI_END_OF_DXE_EVENT_GROUP_GUID event group that was created in
- the entry point of the driver.
-
- This function is called when an event belonging to the
- EFI_END_OF_DXE_EVENT_GROUP_GUID event group is signalled. Such an
- event is signalled once at the end of the dispatching of all
- drivers (end of the so called DXE phase).
+ This function is called when a gEfiPciIoProtocolGuid protocol instance is
+ registered in the protocol database.

@param[in] Event Event declared in the entry point of the driver whose
notification function is being invoked.
@@ -367,33 +273,30 @@ ArmJunoSetNicMacAddress ()
**/
STATIC
VOID
-OnEndOfDxe (
+PciIoNotificationEvent (
IN EFI_EVENT Event,
IN VOID *Context
)
{
- EFI_DEVICE_PATH_PROTOCOL* PciRootComplexDevicePath;
- EFI_HANDLE Handle;
- EFI_STATUS Status;
+ EFI_STATUS Status;
+ EFI_PCI_IO_PROTOCOL *PciIo;

- //
- // PCI Root Complex initialization
- // At the end of the DXE phase, we should get all the driver dispatched.
- // Force the PCI Root Complex to be initialized. It allows the OS to skip
- // this step.
- //
- PciRootComplexDevicePath = (EFI_DEVICE_PATH_PROTOCOL*) &mPciRootComplexDevicePath;
- Status = gBS->LocateDevicePath (&gEfiPciRootBridgeIoProtocolGuid,
- &PciRootComplexDevicePath,
- &Handle);
+ Status = gBS->LocateProtocol (&gEfiPciIoProtocolGuid,
+ mPciIoNotificationRegistration, (VOID **)&PciIo);
+ if (EFI_ERROR (Status)) {
+ return;
+ }

- Status = gBS->ConnectController (Handle, NULL, PciRootComplexDevicePath, FALSE);
- ASSERT_EFI_ERROR (Status);
+ Status = ReadMarvellYoukonPciId (PciIo, JUNO_MARVELL_YUKON_ID);
+ if (EFI_ERROR (Status)) {
+ return;
+ }

- Status = ArmJunoSetNicMacAddress ();
+ Status = ArmJunoSetNicMacAddress (PciIo);
if (EFI_ERROR (Status)) {
DEBUG ((DEBUG_ERROR, "ArmJunoDxe: Failed to set Marvell Yukon NIC MAC address\n"));
}
+ gBS->CloseEvent (Event);
}

EFI_STATUS
@@ -408,7 +311,6 @@ ArmJunoEntryPoint (
CHAR16 *TextDevicePath;
UINTN TextDevicePathSize;
UINT32 JunoRevision;
- EFI_EVENT EndOfDxeEvent;

//
// Register the OHCI and EHCI controllers as non-coherent
@@ -497,20 +399,17 @@ ArmJunoEntryPoint (
PcdSetBoolS (PcdPciDisableBusEnumeration, FALSE);

//
- // Create an event belonging to the "gEfiEndOfDxeEventGroupGuid" group.
- // The "OnEndOfDxe()" function is declared as the call back function.
- // It will be called at the end of the DXE phase when an event of the
- // same group is signalled to inform about the end of the DXE phase.
- // Install the INSTALL_FDT_PROTOCOL protocol.
+ // Create a protocol notification event handler on the PciIo protocol
+ // so we can set the MAC address on the Marvell Yukon as soon as it
+ // appears.
//
- Status = gBS->CreateEventEx (
- EVT_NOTIFY_SIGNAL,
- TPL_CALLBACK,
- OnEndOfDxe,
- NULL,
- &gEfiEndOfDxeEventGroupGuid,
- &EndOfDxeEvent
- );
+ EfiCreateProtocolNotifyEvent (
+ &gEfiPciIoProtocolGuid,
+ TPL_NOTIFY,
(2) I generally prefer the lowest TPL that works, so I'd suggest
TPL_CALLBACK here.

With (1) updated, and (2) optionally updated (if you agree):

Reviewed-by: Laszlo Ersek <lersek@...>

Thanks!
Laszlo


+ PciIoNotificationEvent,
+ NULL,
+ &mPciIoNotificationRegistration
+ );

#ifndef DYNAMIC_TABLES_FRAMEWORK
// Declare the related ACPI Tables
diff --git a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf
index 7c118d9c9c6b..d016967c3c37 100644
--- a/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf
+++ b/Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.inf
@@ -44,7 +44,6 @@ [LibraryClasses]
UefiDriverEntryPoint

[Guids]
- gEfiEndOfDxeEventGroupGuid
gEfiFileInfoGuid

[Protocols]


[PATCH v2 063/105] .mailmap: Add an entry for Mike Wu

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Mike Wu to have his name and email address
displayed properly in the git history.

Cc: Mike Wu <mike.wu@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Mike Wu doesn't give his
approval, by replying to this patch with:
Reviewed-by: Mike Wu <mike.wu@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Mike Wu <mike.wu@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 6f444ad5a3cd..190d2790b103 100644
--- a/.mailmap
+++ b/.mailmap
@@ -122,3 +122,4 @@ Michael Kinney <michael.d.kinney@...> </o=Intel/ou=Americas01/cn=Workers/c
Michael Kubacki <michael.a.kubacki@...>
Michael Kubacki <michael.a.kubacki@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=3c8b0226e75f4ab08d20c151cb7a8a72>
Michael Zimmermann <sigmaepsilon92@...> <[mailto:sigmaepsilon92@...]>
+Mike Wu <mike.wu@...> <mikewuping@6f19259b-4bc3-4df7-8a09-765794883524>
--
2.21.0


[PATCH v2 065/105] .mailmap: Add an entry for Nikolai Saoukh

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Nikolai Saoukh to have his name and email address
displayed properly in the git history.

Cc: Nikolai Saoukh <nms@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Nikolai Saoukh doesn't give his
approval, by replying to this patch with:
Reviewed-by: Nikolai Saoukh <nms@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Nikolai Saoukh <nms@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 9b5351167ca7..a3074feb017c 100644
--- a/.mailmap
+++ b/.mailmap
@@ -124,3 +124,4 @@ Michael Kubacki <michael.a.kubacki@...> </o=Intel/ou=External (FYDIBOHF25S
Michael Zimmermann <sigmaepsilon92@...> <[mailto:sigmaepsilon92@...]>
Mike Wu <mike.wu@...> <mikewuping@6f19259b-4bc3-4df7-8a09-765794883524>
Ming Tan <ming.tan@...>
+Nikolai Saoukh <nms@...>
--
2.21.0


[PATCH v2 064/105] .mailmap: Add an entry for Ming Tan

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Ming Tan to have his name and email address
displayed properly in the git history.

Cc: Ming Tan <ming.tan@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Ming Tan doesn't give his
approval, by replying to this patch with:
Reviewed-by: Ming Tan <ming.tan@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Ming Tan <ming.tan@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 190d2790b103..9b5351167ca7 100644
--- a/.mailmap
+++ b/.mailmap
@@ -123,3 +123,4 @@ Michael Kubacki <michael.a.kubacki@...>
Michael Kubacki <michael.a.kubacki@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=3c8b0226e75f4ab08d20c151cb7a8a72>
Michael Zimmermann <sigmaepsilon92@...> <[mailto:sigmaepsilon92@...]>
Mike Wu <mike.wu@...> <mikewuping@6f19259b-4bc3-4df7-8a09-765794883524>
+Ming Tan <ming.tan@...>
--
2.21.0


[PATCH v2 062/105] .mailmap: Add an entry for Michael Zimmermann

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Michael Zimmermann to have his name and email address
displayed properly in the git history.

Cc: Michael Zimmermann <sigmaepsilon92@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Michael Zimmermann doesn't give his
approval, by replying to this patch with:
Reviewed-by: Michael Zimmermann <sigmaepsilon92@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Michael Zimmermann <sigmaepsilon92@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 822a65a6aad7..6f444ad5a3cd 100644
--- a/.mailmap
+++ b/.mailmap
@@ -121,3 +121,4 @@ Michael Kinney <michael.d.kinney@...> <Michael.d.kinney@...>
Michael Kinney <michael.d.kinney@...> </o=Intel/ou=Americas01/cn=Workers/cn=Kinney, Michael D>
Michael Kubacki <michael.a.kubacki@...>
Michael Kubacki <michael.a.kubacki@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=3c8b0226e75f4ab08d20c151cb7a8a72>
+Michael Zimmermann <sigmaepsilon92@...> <[mailto:sigmaepsilon92@...]>
--
2.21.0


[PATCH v2 061/105] .mailmap: Add an entry for Michael Kubacki

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Michael Kubacki to have his name and email address
displayed properly in the git history.

Cc: Michael Kubacki <michael.a.kubacki@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Michael Kubacki doesn't give his
approval, by replying to this patch with:
Reviewed-by: Michael Kubacki <michael.a.kubacki@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Michael Kubacki <michael.a.kubacki@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 2 ++
1 file changed, 2 insertions(+)

diff --git a/.mailmap b/.mailmap
index a8bffe9d4729..822a65a6aad7 100644
--- a/.mailmap
+++ b/.mailmap
@@ -119,3 +119,5 @@ Michael Kinney <michael.d.kinney@...> <mdkinney@Edk2>
Michael Kinney <michael.d.kinney@...> <mdkinney@...>
Michael Kinney <michael.d.kinney@...> <Michael.d.kinney@...>
Michael Kinney <michael.d.kinney@...> </o=Intel/ou=Americas01/cn=Workers/cn=Kinney, Michael D>
+Michael Kubacki <michael.a.kubacki@...>
+Michael Kubacki <michael.a.kubacki@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=3c8b0226e75f4ab08d20c151cb7a8a72>
--
2.21.0


[PATCH v2 059/105] .mailmap: Add an entry for Michael Johnson

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Michael Johnson to have his name and email address
displayed properly in the git history.

Cc: Michael Johnson <michael.johnson@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Michael Johnson doesn't give his
approval, by replying to this patch with:
Reviewed-by: Michael Johnson <michael.johnson@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Michael Johnson <michael.johnson@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 42a54c6c6f6f..9c1b05fca786 100644
--- a/.mailmap
+++ b/.mailmap
@@ -112,3 +112,4 @@ Mateusz Albecki <mateusz.albecki@...>
Mateusz Albecki <mateusz.albecki@...> </o=Intel/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=Albecki, Mateusz3be>
Maurice Ma <maurice.ma@...>
Maxim Shvecov <Gris87@...>
+Michael Johnson <michael.johnson@...>
--
2.21.0


[PATCH v2 060/105] .mailmap: Add an entry for Michael Kinney

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Michael Kinney to have his name and email address
displayed properly in the git history.

Cc: Michael Kinney <michael.d.kinney@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Michael Kinney doesn't give his
approval, by replying to this patch with:
Reviewed-by: Michael Kinney <michael.d.kinney@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Michael Kinney <michael.d.kinney@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/.mailmap b/.mailmap
index 9c1b05fca786..a8bffe9d4729 100644
--- a/.mailmap
+++ b/.mailmap
@@ -113,3 +113,9 @@ Mateusz Albecki <mateusz.albecki@...> </o=Intel/ou=Exchange Administrative
Maurice Ma <maurice.ma@...>
Maxim Shvecov <Gris87@...>
Michael Johnson <michael.johnson@...>
+Michael Kinney <michael.d.kinney@...>
+Michael Kinney <michael.d.kinney@...> <mdkinney@6f19259b-4bc3-4df7-8a09-765794883524>
+Michael Kinney <michael.d.kinney@...> <mdkinney@Edk2>
+Michael Kinney <michael.d.kinney@...> <mdkinney@...>
+Michael Kinney <michael.d.kinney@...> <Michael.d.kinney@...>
+Michael Kinney <michael.d.kinney@...> </o=Intel/ou=Americas01/cn=Workers/cn=Kinney, Michael D>
--
2.21.0


[PATCH v2 058/105] .mailmap: Add an entry for Maxim Shvecov

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Maxim Shvecov to have his name and email address
displayed properly in the git history.

Cc: Maxim Shvecov <Gris87@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Maxim Shvecov doesn't give his
approval, by replying to this patch with:
Reviewed-by: Maxim Shvecov <Gris87@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Maxim Shvecov <Gris87@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index ce558e39b1b5..42a54c6c6f6f 100644
--- a/.mailmap
+++ b/.mailmap
@@ -111,3 +111,4 @@ Marvin Häuser <mhaeuser@...>
Mateusz Albecki <mateusz.albecki@...>
Mateusz Albecki <mateusz.albecki@...> </o=Intel/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=Albecki, Mateusz3be>
Maurice Ma <maurice.ma@...>
+Maxim Shvecov <Gris87@...>
--
2.21.0


[PATCH v2 056/105] .mailmap: Add an entry for Mateusz Albecki

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Mateusz Albecki to have his name and email address
displayed properly in the git history.

Cc: Mateusz Albecki <mateusz.albecki@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Mateusz Albecki doesn't give his
approval, by replying to this patch with:
Reviewed-by: Mateusz Albecki <mateusz.albecki@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Mateusz Albecki <mateusz.albecki@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 2 ++
1 file changed, 2 insertions(+)

diff --git a/.mailmap b/.mailmap
index d22ce43a6009..4b76c5b50d5c 100644
--- a/.mailmap
+++ b/.mailmap
@@ -108,3 +108,5 @@ Marc W Chen <marc.w.chen@...>
Marvin Häuser <Marvin.Haeuser@...>
Marvin Häuser <Marvin.Haeuser@...> edk2-devel <edk2-devel-bounces@...>
Marvin Häuser <mhaeuser@...>
+Mateusz Albecki <mateusz.albecki@...>
+Mateusz Albecki <mateusz.albecki@...> </o=Intel/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=Albecki, Mateusz3be>
--
2.21.0


[PATCH v2 057/105] .mailmap: Add an entry for Maurice Ma

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Maurice Ma to have his name and email address
displayed properly in the git history.

Cc: Maurice Ma <maurice.ma@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Maurice Ma doesn't give his
approval, by replying to this patch with:
Reviewed-by: Maurice Ma <maurice.ma@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Maurice Ma <maurice.ma@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 4b76c5b50d5c..ce558e39b1b5 100644
--- a/.mailmap
+++ b/.mailmap
@@ -110,3 +110,4 @@ Marvin Häuser <Marvin.Haeuser@...> edk2-devel <edk2-devel-bounces@lists
Marvin Häuser <mhaeuser@...>
Mateusz Albecki <mateusz.albecki@...>
Mateusz Albecki <mateusz.albecki@...> </o=Intel/ou=Exchange Administrative Group (FYDIBOHF23SPDLT)/cn=Recipients/cn=Albecki, Mateusz3be>
+Maurice Ma <maurice.ma@...>
--
2.21.0


[PATCH v2 055/105] .mailmap: Add an entry for Marvin Häuser

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Marvin Häuser to have his name and email address
displayed properly in the git history (in particular, commit 62ec4a5e).

Cc: Marvin Häuser <Marvin.Haeuser@...>
Cc: Marvin Häuser <Marvin.Haeuser@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Marvin Häuser doesn't give his
approval, by replying to this patch with either:
Reviewed-by: Marvin Häuser <Marvin.Haeuser@...>
Reviewed-by: Marvin Häuser <Marvin.Haeuser@...>

If you think this patch is inappropriate, you don't need to justify,
reply with any:
NAcked-by: Marvin Häuser <Marvin.Haeuser@...>
NAcked-by: Marvin Häuser <Marvin.Haeuser@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Note this is a borderline case, due to the generic email used in commit
62ec4a5ef674. Since this is the unique entry, we don't use the
edk2-devel-bounces@... list anymore and we have CI checks,
we shouldn't merge a patch from someone else doing the same mistake
using the same From. So IMHO this patch seems safe.

Regards,

Phil.
---
.mailmap | 3 +++
1 file changed, 3 insertions(+)

diff --git a/.mailmap b/.mailmap
index ca7a730f677f..d22ce43a6009 100644
--- a/.mailmap
+++ b/.mailmap
@@ -105,3 +105,6 @@ Maggie Chu <maggie.chu@...>
Maggie Chu <maggie.chu@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=fe425ca7e5f4401abed22b904fe5d964>
Marc-André Lureau <marcandre.lureau@...> <marcandre.lureau@...>
Marc W Chen <marc.w.chen@...>
+Marvin Häuser <Marvin.Haeuser@...>
+Marvin Häuser <Marvin.Haeuser@...> edk2-devel <edk2-devel-bounces@...>
+Marvin Häuser <mhaeuser@...>
--
2.21.0


[PATCH v2 054/105] .mailmap: Add an entry for Marc W Chen

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Marc W Chen to have his name and email address
displayed properly in the git history.

Cc: Marc W Chen <marc.w.chen@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Marc W Chen doesn't give his
approval, by replying to this patch with:
Reviewed-by: Marc W Chen <marc.w.chen@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Marc W Chen <marc.w.chen@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 2013a48da245..ca7a730f677f 100644
--- a/.mailmap
+++ b/.mailmap
@@ -104,3 +104,4 @@ Maciej Rabeda <maciej.rabeda@...>
Maggie Chu <maggie.chu@...>
Maggie Chu <maggie.chu@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=fe425ca7e5f4401abed22b904fe5d964>
Marc-André Lureau <marcandre.lureau@...> <marcandre.lureau@...>
+Marc W Chen <marc.w.chen@...>
--
2.21.0


[PATCH v2 053/105] .mailmap: Add an entry for Marc-André Lureau

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Marc-André Lureau to have his name and email address
displayed properly in the git history.

Cc: Marc-André Lureau <marcandre.lureau@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Marc-André Lureau doesn't give his
approval, by replying to this patch with:
Reviewed-by: Marc-André Lureau <marcandre.lureau@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Marc-André Lureau <marcandre.lureau@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 66297d4f8943..2013a48da245 100644
--- a/.mailmap
+++ b/.mailmap
@@ -103,3 +103,4 @@ Lu Shifei <shifeix.a.lu@...>
Maciej Rabeda <maciej.rabeda@...>
Maggie Chu <maggie.chu@...>
Maggie Chu <maggie.chu@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=fe425ca7e5f4401abed22b904fe5d964>
+Marc-André Lureau <marcandre.lureau@...> <marcandre.lureau@...>
--
2.21.0


[PATCH v2 052/105] .mailmap: Add an entry for Maggie Chu

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Maggie Chu to have his name and email address
displayed properly in the git history.

Cc: Maggie Chu <maggie.chu@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Maggie Chu doesn't give his
approval, by replying to this patch with:
Reviewed-by: Maggie Chu <maggie.chu@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Maggie Chu <maggie.chu@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 2 ++
1 file changed, 2 insertions(+)

diff --git a/.mailmap b/.mailmap
index 5340600d9d33..66297d4f8943 100644
--- a/.mailmap
+++ b/.mailmap
@@ -101,3 +101,5 @@ Loh Tien Hock <tien.hock.loh@...>
Lubo Zhang <lubo.zhang@...> <luobozhang@Edk2>
Lu Shifei <shifeix.a.lu@...>
Maciej Rabeda <maciej.rabeda@...>
+Maggie Chu <maggie.chu@...>
+Maggie Chu <maggie.chu@...> </o=Intel/ou=External (FYDIBOHF25SPDLT)/cn=Recipients/cn=fe425ca7e5f4401abed22b904fe5d964>
--
2.21.0


[PATCH v2 051/105] .mailmap: Add an entry for Maciej Rabeda

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Maciej Rabeda to have his name and email address
displayed properly in the git history.

Cc: Maciej Rabeda <maciej.rabeda@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Maciej Rabeda doesn't give his
approval, by replying to this patch with:
Reviewed-by: Maciej Rabeda <maciej.rabeda@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Maciej Rabeda <maciej.rabeda@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 51798f876a35..5340600d9d33 100644
--- a/.mailmap
+++ b/.mailmap
@@ -100,3 +100,4 @@ Liming Gao <liming.gao@...> <liming.gao@...>
Loh Tien Hock <tien.hock.loh@...>
Lubo Zhang <lubo.zhang@...> <luobozhang@Edk2>
Lu Shifei <shifeix.a.lu@...>
+Maciej Rabeda <maciej.rabeda@...>
--
2.21.0


[PATCH v2 050/105] .mailmap: Add an entry for Lu Shifei

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Lu Shifei to have his name and email address
displayed properly in the git history.

Cc: Lu Shifei <shifeix.a.lu@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Lu Shifei doesn't give his
approval, by replying to this patch with:
Reviewed-by: Lu Shifei <shifeix.a.lu@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Lu Shifei <shifeix.a.lu@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 67bcbf11d4cb..51798f876a35 100644
--- a/.mailmap
+++ b/.mailmap
@@ -99,3 +99,4 @@ Liming Gao <liming.gao@...> <lgao4@Edk2>
Liming Gao <liming.gao@...> <liming.gao@...>
Loh Tien Hock <tien.hock.loh@...>
Lubo Zhang <lubo.zhang@...> <luobozhang@Edk2>
+Lu Shifei <shifeix.a.lu@...>
--
2.21.0


[PATCH v2 048/105] .mailmap: Add an entry for Loh Tien Hock

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Loh Tien Hock to have his name and email address
displayed properly in the git history.

Cc: Loh Tien Hock <tien.hock.loh@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Loh Tien Hock doesn't give his
approval, by replying to this patch with:
Reviewed-by: Loh Tien Hock <tien.hock.loh@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Loh Tien Hock <tien.hock.loh@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 53aa4fa01e15..6956b68c5e22 100644
--- a/.mailmap
+++ b/.mailmap
@@ -97,3 +97,4 @@ Liming Gao <liming.gao@...> <Gao, Liming liming.gao@...>
Liming Gao <liming.gao@...> <lgao4@6f19259b-4bc3-4df7-8a09-765794883524>
Liming Gao <liming.gao@...> <lgao4@Edk2>
Liming Gao <liming.gao@...> <liming.gao@...>
+Loh Tien Hock <tien.hock.loh@...>
--
2.21.0


[PATCH v2 049/105] .mailmap: Add an entry for Lubo Zhang

Philippe Mathieu-Daudé <philmd@...>
 

We use .mailmap to display contributors email addresses in an
uniform format.

Add an entry for Lubo Zhang to have his name and email address
displayed properly in the git history.

Cc: Lubo Zhang <lubo.zhang@...>
Signed-off-by: Philippe Mathieu-Daude <philmd@...>
---
[Due to MTA restricting the recipient list to 100, I can not Cc all the
named developers in the cover. Therefore I'm adapting the explaination
from the cover in each patch]

This patch won't get merged if Lubo Zhang doesn't give his
approval, by replying to this patch with:
Reviewed-by: Lubo Zhang <lubo.zhang@...>

If you think this patch is inappropriate, you don't need to justify,
reply with:
NAcked-by: Lubo Zhang <lubo.zhang@...>
or simply:
NACK

If your Firstname Lastname order is incorrect, tell me and I will fix it.

You can also ignore this mail, but I might resend it and keep bothering
you.

Regards,

Phil.
---
.mailmap | 1 +
1 file changed, 1 insertion(+)

diff --git a/.mailmap b/.mailmap
index 6956b68c5e22..67bcbf11d4cb 100644
--- a/.mailmap
+++ b/.mailmap
@@ -98,3 +98,4 @@ Liming Gao <liming.gao@...> <lgao4@6f19259b-4bc3-4df7-8a09-765794883524>
Liming Gao <liming.gao@...> <lgao4@Edk2>
Liming Gao <liming.gao@...> <liming.gao@...>
Loh Tien Hock <tien.hock.loh@...>
+Lubo Zhang <lubo.zhang@...> <luobozhang@Edk2>
--
2.21.0