2021-12-02 23:59:12

by Atish Patra

Subject: [PATCH v3] MAINTAINERS: Update Atish's email address

I am no longer employed by western digital. Update my email address to
personal one and add entries to .mailmap as well.

Signed-off-by: Atish Patra <[email protected]>
---
.mailmap | 1 +
MAINTAINERS | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.mailmap b/.mailmap
index 6277bb27b4bf..23f6b0a60adf 100644
--- a/.mailmap
+++ b/.mailmap
@@ -50,6 +50,7 @@ Archit Taneja <[email protected]>
Ard Biesheuvel <[email protected]> <[email protected]>
Arnaud Patard <[email protected]>
Arnd Bergmann <[email protected]>
+Atish Patra <[email protected]> <[email protected]> <[email protected]>
Axel Dyks <[email protected]>
Axel Lin <[email protected]>
Bart Van Assche <[email protected]> <[email protected]>
diff --git a/MAINTAINERS b/MAINTAINERS
index 5250298d2817..6c2a34da0314 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10434,7 +10434,7 @@ F: arch/powerpc/kvm/

KERNEL VIRTUAL MACHINE FOR RISC-V (KVM/riscv)
M: Anup Patel <[email protected]>
-R: Atish Patra <[email protected]>
+R: Atish Patra <[email protected]>
L: [email protected]
L: [email protected]
L: [email protected]
--
2.33.1

2021-12-03 12:42:18

by Anup Patel

Subject: Re: [PATCH v3] MAINTAINERS: Update Atish's email address

On Fri, Dec 3, 2021 at 5:29 AM Atish Patra <[email protected]> wrote:
>
> I am no longer employed by western digital. Update my email address to
> personal one and add entries to .mailmap as well.
>
> Signed-off-by: Atish Patra <[email protected]>

I have queued this for 5.17

Thanks,
Anup

> ---
> .mailmap | 1 +
> MAINTAINERS | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/.mailmap b/.mailmap
> index 6277bb27b4bf..23f6b0a60adf 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -50,6 +50,7 @@ Archit Taneja <[email protected]>
> Ard Biesheuvel <[email protected]> <[email protected]>
> Arnaud Patard <[email protected]>
> Arnd Bergmann <[email protected]>
> +Atish Patra <[email protected]> <[email protected]> <[email protected]>
> Axel Dyks <[email protected]>
> Axel Lin <[email protected]>
> Bart Van Assche <[email protected]> <[email protected]>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5250298d2817..6c2a34da0314 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10434,7 +10434,7 @@ F: arch/powerpc/kvm/
>
> KERNEL VIRTUAL MACHINE FOR RISC-V (KVM/riscv)
> M: Anup Patel <[email protected]>
> -R: Atish Patra <[email protected]>
> +R: Atish Patra <[email protected]>
> L: [email protected]
> L: [email protected]
> L: [email protected]
> --
> 2.33.1
>

2021-12-03 16:10:36

by Steven Price

Subject: Re: [PATCH v3] MAINTAINERS: Update Atish's email address

On 02/12/2021 23:58, Atish Patra wrote:
> I am no longer employed by western digital. Update my email address to
> personal one and add entries to .mailmap as well.
>
> Signed-off-by: Atish Patra <[email protected]>
> ---
> .mailmap | 1 +
> MAINTAINERS | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/.mailmap b/.mailmap
> index 6277bb27b4bf..23f6b0a60adf 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -50,6 +50,7 @@ Archit Taneja <[email protected]>
> Ard Biesheuvel <[email protected]> <[email protected]>
> Arnaud Patard <[email protected]>
> Arnd Bergmann <[email protected]>
> +Atish Patra <[email protected]> <[email protected]> <[email protected]>

I don't think this does what you expect. You can't list more than one
email address to replace on the same line. You can use the command "git
check-mailmap" to test what happens, e.g. with this change applied:

$ git check-mailmap "<[email protected]>"
<[email protected]>
$ git check-mailmap "<[email protected]>"
Atish Patra <[email protected]>
$ git check-mailmap "<[email protected]>"
<[email protected]>

So only your @wdc.com address is translated. If you want to translate
the @rivosinc.com address as well you need a second line. As the file says:

# For format details, see "MAPPING AUTHORS" in "man git-shortlog".

Steve

> Axel Dyks <[email protected]>
> Axel Lin <[email protected]>
> Bart Van Assche <[email protected]> <[email protected]>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 5250298d2817..6c2a34da0314 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10434,7 +10434,7 @@ F: arch/powerpc/kvm/
>
> KERNEL VIRTUAL MACHINE FOR RISC-V (KVM/riscv)
> M: Anup Patel <[email protected]>
> -R: Atish Patra <[email protected]>
> +R: Atish Patra <[email protected]>
> L: [email protected]
> L: [email protected]
> L: [email protected]
>

2021-12-04 04:09:36

by Anup Patel

Subject: Re: [PATCH v3] MAINTAINERS: Update Atish's email address

On Fri, Dec 3, 2021 at 9:40 PM Steven Price <[email protected]> wrote:
>
> On 02/12/2021 23:58, Atish Patra wrote:
> > I am no longer employed by western digital. Update my email address to
> > personal one and add entries to .mailmap as well.
> >
> > Signed-off-by: Atish Patra <[email protected]>
> > ---
> > .mailmap | 1 +
> > MAINTAINERS | 2 +-
> > 2 files changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/.mailmap b/.mailmap
> > index 6277bb27b4bf..23f6b0a60adf 100644
> > --- a/.mailmap
> > +++ b/.mailmap
> > @@ -50,6 +50,7 @@ Archit Taneja <[email protected]>
> > Ard Biesheuvel <[email protected]> <[email protected]>
> > Arnaud Patard <[email protected]>
> > Arnd Bergmann <[email protected]>
> > +Atish Patra <[email protected]> <[email protected]> <[email protected]>
>
> I don't think this does what you expect. You can't list more than one
> email address to replace on the same line. You can use the command "git
> check-mailmap" to test what happens, e.g. with this change applied:
>
> $ git check-mailmap "<[email protected]>"
> <[email protected]>
> $ git check-mailmap "<[email protected]>"
> Atish Patra <[email protected]>
> $ git check-mailmap "<[email protected]>"
> <[email protected]>
>
> So only your @wdc.com address is translated. If you want to translate
> the @rivosinc.com address as well you need a second line. As the file says:

Thanks Steve for noticing this. Even, I realized this while queuing the patch.

I have removed @rivosinc.com email address from the patch in my queue. I
believe Atish currently uses both personal and @rivosinc.com email addresses
on LKML.

Refer,
https://github.com/kvm-riscv/linux/commit/2255b100410179bb8151e99e8396debddea0ef1d
https://github.com/kvm-riscv/linux/commits/riscv_kvm_queue

Regards,
Anup

>
> # For format details, see "MAPPING AUTHORS" in "man git-shortlog".
>
> Steve
>
> > Axel Dyks <[email protected]>
> > Axel Lin <[email protected]>
> > Bart Van Assche <[email protected]> <[email protected]>
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 5250298d2817..6c2a34da0314 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -10434,7 +10434,7 @@ F: arch/powerpc/kvm/
> >
> > KERNEL VIRTUAL MACHINE FOR RISC-V (KVM/riscv)
> > M: Anup Patel <[email protected]>
> > -R: Atish Patra <[email protected]>
> > +R: Atish Patra <[email protected]>
> > L: [email protected]
> > L: [email protected]
> > L: [email protected]
> >
>

2021-12-04 22:26:07

by Atish Patra

Subject: Re: [PATCH v3] MAINTAINERS: Update Atish's email address

On Fri, Dec 3, 2021 at 8:09 PM Anup Patel <[email protected]> wrote:
>
> On Fri, Dec 3, 2021 at 9:40 PM Steven Price <[email protected]> wrote:
> >
> > On 02/12/2021 23:58, Atish Patra wrote:
> > > I am no longer employed by western digital. Update my email address to
> > > personal one and add entries to .mailmap as well.
> > >
> > > Signed-off-by: Atish Patra <[email protected]>
> > > ---
> > > .mailmap | 1 +
> > > MAINTAINERS | 2 +-
> > > 2 files changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/.mailmap b/.mailmap
> > > index 6277bb27b4bf..23f6b0a60adf 100644
> > > --- a/.mailmap
> > > +++ b/.mailmap
> > > @@ -50,6 +50,7 @@ Archit Taneja <[email protected]>
> > > Ard Biesheuvel <[email protected]> <[email protected]>
> > > Arnaud Patard <[email protected]>
> > > Arnd Bergmann <[email protected]>
> > > +Atish Patra <[email protected]> <[email protected]> <[email protected]>
> >
> > I don't think this does what you expect. You can't list more than one
> > email address to replace on the same line. You can use the command "git
> > check-mailmap" to test what happens, e.g. with this change applied:
> >
> > $ git check-mailmap "<[email protected]>"
> > <[email protected]>
> > $ git check-mailmap "<[email protected]>"
> > Atish Patra <[email protected]>
> > $ git check-mailmap "<[email protected]>"
> > <[email protected]>
> >
> > So only your @wdc.com address is translated. If you want to translate
> > the @rivosinc.com address as well you need a second line. As the file says:
>
> Thanks Steve for noticing this. Even, I realized this while queuing the patch.
>
> I have removed @rivosinc.com email address from the patch in my queue. I
> believe Atish currently uses both personal and @rivosinc.com email addresses
> on LKML.
>

Thanks!

> Refer,
> https://github.com/kvm-riscv/linux/commit/2255b100410179bb8151e99e8396debddea0ef1d
> https://github.com/kvm-riscv/linux/commits/riscv_kvm_queue
>
> Regards,
> Anup
>
> >
> > # For format details, see "MAPPING AUTHORS" in "man git-shortlog".
> >
> > Steve
> >
> > > Axel Dyks <[email protected]>
> > > Axel Lin <[email protected]>
> > > Bart Van Assche <[email protected]> <[email protected]>
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 5250298d2817..6c2a34da0314 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -10434,7 +10434,7 @@ F: arch/powerpc/kvm/
> > >
> > > KERNEL VIRTUAL MACHINE FOR RISC-V (KVM/riscv)
> > > M: Anup Patel <[email protected]>
> > > -R: Atish Patra <[email protected]>
> > > +R: Atish Patra <[email protected]>
> > > L: [email protected]
> > > L: [email protected]
> > > L: [email protected]
> > >
> >



--
Regards,
Atish