DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP
@ 2020-11-10  0:19 Radha Mohan Chintakuntla
  2020-11-10 18:13 ` Radha Mohan
  0 siblings, 1 reply; 12+ messages in thread
From: Radha Mohan Chintakuntla @ 2020-11-10  0:19 UTC (permalink / raw)
  To: dev, thomas; +Cc: vburru, sburla, jerinj, Radha Mohan Chintakuntla

Replace the maintainers for OcteonTx2 DMA and EP drivers.

Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
---
 MAINTAINERS | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a720cf672e..214515060a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1258,12 +1258,14 @@ F: drivers/raw/dpaa2_cmdif/
 F: doc/guides/rawdevs/dpaa2_cmdif.rst
 
 Marvell OCTEON TX2 DMA
-M: Satha Rao <skoteshwar@marvell.com>
+M: Radha Mohan Chintakuntla <radhac@marvell.com>
+M: Veerasenareddy Burru <vburru@marvell.com>
 F: drivers/raw/octeontx2_dma/
 F: doc/guides/rawdevs/octeontx2_dma.rst
 
 Marvell OCTEON TX2 EP
-M: Mahipal Challa <mchalla@marvell.com>
+M: Radha Mohan Chintakuntla <radhac@marvell.com>
+M: Veerasenareddy Burru <vburru@marvell.com>
 F: drivers/raw/octeontx2_ep/
 F: doc/guides/rawdevs/octeontx2_ep.rst
 
-- 
2.24.1


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-10  0:19 [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP Radha Mohan Chintakuntla
@ 2020-11-10 18:13 ` Radha Mohan
  2020-11-10 18:18   ` [dpdk-dev] [EXT] " Satha Koteswara Rao Kottidi
  2020-11-11  7:56   ` Mahipal Challa
  0 siblings, 2 replies; 12+ messages in thread
From: Radha Mohan @ 2020-11-10 18:13 UTC (permalink / raw)
  To: Radha Mohan Chintakuntla, Satha Koteswara Rao Kottidi, mchalla
  Cc: dev, Thomas Monjalon, vburru, Satananda Burla, Jerin Jacob Kollanukkaran

On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla
<radhac@marvell.com> wrote:
>
> Replace the maintainers for OcteonTx2 DMA and EP drivers.
>
> Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> ---
>  MAINTAINERS | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a720cf672e..214515060a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1258,12 +1258,14 @@ F: drivers/raw/dpaa2_cmdif/
>  F: doc/guides/rawdevs/dpaa2_cmdif.rst
>
>  Marvell OCTEON TX2 DMA
> -M: Satha Rao <skoteshwar@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_dma/
>  F: doc/guides/rawdevs/octeontx2_dma.rst
>
>  Marvell OCTEON TX2 EP
> -M: Mahipal Challa <mchalla@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_ep/
>  F: doc/guides/rawdevs/octeontx2_ep.rst
>
> --
> 2.24.1
>

Adding previous maintainers to ack.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-10 18:13 ` Radha Mohan
@ 2020-11-10 18:18   ` Satha Koteswara Rao Kottidi
  2020-11-11  7:56   ` Mahipal Challa
  1 sibling, 0 replies; 12+ messages in thread
From: Satha Koteswara Rao Kottidi @ 2020-11-10 18:18 UTC (permalink / raw)
  To: Radha Mohan, Radha Chintakuntla, Mahipal Challa
  Cc: dev, Thomas Monjalon, Veerasenareddy Burru, Satananda Burla,
	Jerin Jacob Kollanukkaran



-----Original Message-----
From: Radha Mohan <mohun106@gmail.com> 
Sent: Tuesday, November 10, 2020 11:44 PM
To: Radha Chintakuntla <radhac@marvell.com>; Satha Koteswara Rao Kottidi <skoteshwar@marvell.com>; Mahipal Challa <mchalla@marvell.com>
Cc: dev@dpdk.org; Thomas Monjalon <thomas@monjalon.net>; Veerasenareddy Burru <vburru@marvell.com>; Satananda Burla <sburla@marvell.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>
Subject: [EXT] Re: [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP

External Email

----------------------------------------------------------------------
On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
>
> Replace the maintainers for OcteonTx2 DMA and EP drivers.
>
> Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> ---
>  MAINTAINERS | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS index a720cf672e..214515060a 
> 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1258,12 +1258,14 @@ F: drivers/raw/dpaa2_cmdif/
>  F: doc/guides/rawdevs/dpaa2_cmdif.rst
>
>  Marvell OCTEON TX2 DMA
> -M: Satha Rao <skoteshwar@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_dma/
>  F: doc/guides/rawdevs/octeontx2_dma.rst
>
>  Marvell OCTEON TX2 EP
> -M: Mahipal Challa <mchalla@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_ep/
>  F: doc/guides/rawdevs/octeontx2_ep.rst
>
> --
> 2.24.1
>

Adding previous maintainers to ack.
Acked-by: Satha Rao <skoteshwar@marvell.com>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-10 18:13 ` Radha Mohan
  2020-11-10 18:18   ` [dpdk-dev] [EXT] " Satha Koteswara Rao Kottidi
@ 2020-11-11  7:56   ` Mahipal Challa
  2020-11-13 19:18     ` Radha Mohan
  2020-11-22 11:28     ` Thomas Monjalon
  1 sibling, 2 replies; 12+ messages in thread
From: Mahipal Challa @ 2020-11-11  7:56 UTC (permalink / raw)
  To: Radha Mohan, Radha Chintakuntla, Satha Koteswara Rao Kottidi
  Cc: dev, Thomas Monjalon, Veerasenareddy Burru, Satananda Burla,
	Jerin Jacob Kollanukkaran



-----Original Message-----
From: Radha Mohan <mohun106@gmail.com> 
Sent: Tuesday, November 10, 2020 11:44 PM
To: Radha Chintakuntla <radhac@marvell.com>; Satha Koteswara Rao Kottidi <skoteshwar@marvell.com>; Mahipal Challa <mchalla@marvell.com>
Cc: dev@dpdk.org; Thomas Monjalon <thomas@monjalon.net>; Veerasenareddy Burru <vburru@marvell.com>; Satananda Burla <sburla@marvell.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>
Subject: [EXT] Re: [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP

External Email

----------------------------------------------------------------------
On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
>
> Replace the maintainers for OcteonTx2 DMA and EP drivers.
>
> Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> ---
>  MAINTAINERS | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS index a720cf672e..214515060a 
> 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1258,12 +1258,14 @@ F: drivers/raw/dpaa2_cmdif/
>  F: doc/guides/rawdevs/dpaa2_cmdif.rst
>
>  Marvell OCTEON TX2 DMA
> -M: Satha Rao <skoteshwar@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_dma/
>  F: doc/guides/rawdevs/octeontx2_dma.rst
>
>  Marvell OCTEON TX2 EP
> -M: Mahipal Challa <mchalla@marvell.com>
> +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> +M: Veerasenareddy Burru <vburru@marvell.com>
>  F: drivers/raw/octeontx2_ep/
>  F: doc/guides/rawdevs/octeontx2_ep.rst
>
> --
> 2.24.1
>

>Adding previous maintainers to ack.
Acked-by: Mahipal Challa <mchalla@marvell.com

Thanks,
Mahipal

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-11  7:56   ` Mahipal Challa
@ 2020-11-13 19:18     ` Radha Mohan
  2020-11-13 22:38       ` Thomas Monjalon
  2020-11-22 11:28     ` Thomas Monjalon
  1 sibling, 1 reply; 12+ messages in thread
From: Radha Mohan @ 2020-11-13 19:18 UTC (permalink / raw)
  To: Mahipal Challa
  Cc: Radha Chintakuntla, Satha Koteswara Rao Kottidi, dev,
	Thomas Monjalon, Veerasenareddy Burru, Satananda Burla,
	Jerin Jacob Kollanukkaran

On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
>
>
>
> -----Original Message-----
> From: Radha Mohan <mohun106@gmail.com>
> Sent: Tuesday, November 10, 2020 11:44 PM
> To: Radha Chintakuntla <radhac@marvell.com>; Satha Koteswara Rao Kottidi <skoteshwar@marvell.com>; Mahipal Challa <mchalla@marvell.com>
> Cc: dev@dpdk.org; Thomas Monjalon <thomas@monjalon.net>; Veerasenareddy Burru <vburru@marvell.com>; Satananda Burla <sburla@marvell.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>
> Subject: [EXT] Re: [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP
>
> External Email
>
> ----------------------------------------------------------------------
> On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> >
> > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> >
> > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > ---
> >  MAINTAINERS | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS index a720cf672e..214515060a
> > 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1258,12 +1258,14 @@ F: drivers/raw/dpaa2_cmdif/
> >  F: doc/guides/rawdevs/dpaa2_cmdif.rst
> >
> >  Marvell OCTEON TX2 DMA
> > -M: Satha Rao <skoteshwar@marvell.com>
> > +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> > +M: Veerasenareddy Burru <vburru@marvell.com>
> >  F: drivers/raw/octeontx2_dma/
> >  F: doc/guides/rawdevs/octeontx2_dma.rst
> >
> >  Marvell OCTEON TX2 EP
> > -M: Mahipal Challa <mchalla@marvell.com>
> > +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> > +M: Veerasenareddy Burru <vburru@marvell.com>
> >  F: drivers/raw/octeontx2_ep/
> >  F: doc/guides/rawdevs/octeontx2_ep.rst
> >
> > --
> > 2.24.1
> >
>
> >Adding previous maintainers to ack.
> Acked-by: Mahipal Challa <mchalla@marvell.com
>
> Thanks,
> Mahipal

Hi Thomas,
Could you please pick this patch ?

regards,
Radha Mohan

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-13 19:18     ` Radha Mohan
@ 2020-11-13 22:38       ` Thomas Monjalon
  2020-11-16 18:28         ` Radha Mohan
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Monjalon @ 2020-11-13 22:38 UTC (permalink / raw)
  To: Mahipal Challa, Radha Mohan
  Cc: Radha Chintakuntla, Satha Koteswara Rao Kottidi, dev,
	Veerasenareddy Burru, Satananda Burla, Jerin Jacob Kollanukkaran

13/11/2020 20:18, Radha Mohan:
> On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> From: Radha Mohan <mohun106@gmail.com>
> Sent: Tuesday, November 10, 2020 11:44 PM
> > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > >
> > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > >
> > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
[...]
> > >Adding previous maintainers to ack.
> > Acked-by: Mahipal Challa <mchalla@marvell.com
> >
> > Thanks,
> > Mahipal
> 
> Hi Thomas,
> Could you please pick this patch ?

This low-priority patch has been sent 4 days ago,
and was acked by only 1 maintainer 3 days ago.

Why pushing? What is your fear exactly?

Why the new maintainers candidates have 0 and 2 contributions
in git history?



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-13 22:38       ` Thomas Monjalon
@ 2020-11-16 18:28         ` Radha Mohan
  2020-11-16 20:28           ` Thomas Monjalon
  0 siblings, 1 reply; 12+ messages in thread
From: Radha Mohan @ 2020-11-16 18:28 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Mahipal Challa, Radha Chintakuntla, Satha Koteswara Rao Kottidi,
	dev, Veerasenareddy Burru, Satananda Burla,
	Jerin Jacob Kollanukkaran

On Fri, Nov 13, 2020 at 2:39 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 13/11/2020 20:18, Radha Mohan:
> > On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> > From: Radha Mohan <mohun106@gmail.com>
> > Sent: Tuesday, November 10, 2020 11:44 PM
> > > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > > >
> > > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > > >
> > > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> [...]
> > > >Adding previous maintainers to ack.
> > > Acked-by: Mahipal Challa <mchalla@marvell.com
> > >
> > > Thanks,
> > > Mahipal
> >
> > Hi Thomas,
> > Could you please pick this patch ?
>
> This low-priority patch has been sent 4 days ago,
> and was acked by only 1 maintainer 3 days ago.
>

It was acked by both the old maintainers Satha Koteshwar and Mahipal
Challa. Maybe both of those emails didn't come on top of each other.

> Why pushing? What is your fear exactly?

Just want to make the transfer of ownership official in the mainline.
There's no fear I just normally wanted to send a reminder to you to
pick as it might be missed inclusion. If there's a guarantee that all
ack'ed patches will go automatically then I won't be sending any
reminder emails in future.

>
> Why the new maintainers candidates have 0 and 2 contributions
> in git history?
>
These drivers requires some maintenance and likely expanded to support
future chips from Marvell. So you will not see contributions right
away.
So to become a maintainer one has to have prior contributions ? Kind
of makes odd sense as these drivers are specific to our chip and we
are changing ownership for maintaining them.
I do not understand why you have an issue here? You don't like
reminding then I understand.

regards,
Radha Mohan

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-16 18:28         ` Radha Mohan
@ 2020-11-16 20:28           ` Thomas Monjalon
  2020-11-18  4:15             ` Radha Mohan
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Monjalon @ 2020-11-16 20:28 UTC (permalink / raw)
  To: Radha Mohan, Jerin Jacob Kollanukkaran
  Cc: Mahipal Challa, Radha Chintakuntla, Satha Koteswara Rao Kottidi,
	dev, Veerasenareddy Burru, Satananda Burla

16/11/2020 19:28, Radha Mohan:
> On Fri, Nov 13, 2020 at 2:39 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > 13/11/2020 20:18, Radha Mohan:
> > > On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> > > From: Radha Mohan <mohun106@gmail.com>
> > > Sent: Tuesday, November 10, 2020 11:44 PM
> > > > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > > > >
> > > > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > > > >
> > > > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > [...]
> > > > >Adding previous maintainers to ack.
> > > > Acked-by: Mahipal Challa <mchalla@marvell.com
> > > >
> > > > Thanks,
> > > > Mahipal
> > >
> > > Hi Thomas,
> > > Could you please pick this patch ?
> >
> > This low-priority patch has been sent 4 days ago,
> > and was acked by only 1 maintainer 3 days ago.
> >
> 
> It was acked by both the old maintainers Satha Koteshwar and Mahipal
> Challa. Maybe both of those emails didn't come on top of each other.

I missed one.

> > Why pushing? What is your fear exactly?
> 
> Just want to make the transfer of ownership official in the mainline.
> There's no fear I just normally wanted to send a reminder to you to
> pick as it might be missed inclusion. If there's a guarantee that all
> ack'ed patches will go automatically then I won't be sending any
> reminder emails in future.

It's OK to send reminder, but please wait at least a week.

> > Why the new maintainers candidates have 0 and 2 contributions
> > in git history?
> >
> These drivers requires some maintenance and likely expanded to support
> future chips from Marvell. So you will not see contributions right
> away.
> So to become a maintainer one has to have prior contributions ?

Yes: http://doc.dpdk.org/guides/contributing/patches.html#maintainers-and-sub-trees
"
Maintainers should have demonstrated a reasonable level of contributions or reviews to the component area. The maintainer should be confirmed by an ack from an established contributor.
"

> Kind of makes odd sense as these drivers are specific to our chip and we
> are changing ownership for maintaining them.
> I do not understand why you have an issue here?
> You don't like reminding then I understand.

I don't like how you push new unconfirmed maintainers.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-16 20:28           ` Thomas Monjalon
@ 2020-11-18  4:15             ` Radha Mohan
  2020-11-18  9:15               ` Thomas Monjalon
  0 siblings, 1 reply; 12+ messages in thread
From: Radha Mohan @ 2020-11-18  4:15 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Jerin Jacob Kollanukkaran, Mahipal Challa, Radha Chintakuntla,
	Satha Koteswara Rao Kottidi, dev, Veerasenareddy Burru,
	Satananda Burla

On Mon, Nov 16, 2020 at 12:28 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 16/11/2020 19:28, Radha Mohan:
> > On Fri, Nov 13, 2020 at 2:39 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > 13/11/2020 20:18, Radha Mohan:
> > > > On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> > > > From: Radha Mohan <mohun106@gmail.com>
> > > > Sent: Tuesday, November 10, 2020 11:44 PM
> > > > > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > > > > >
> > > > > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > > > > >
> > > > > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > > [...]
> > > > > >Adding previous maintainers to ack.
> > > > > Acked-by: Mahipal Challa <mchalla@marvell.com
> > > > >
> > > > > Thanks,
> > > > > Mahipal
> > > >
> > > > Hi Thomas,
> > > > Could you please pick this patch ?
> > >
> > > This low-priority patch has been sent 4 days ago,
> > > and was acked by only 1 maintainer 3 days ago.
> > >
> >
> > It was acked by both the old maintainers Satha Koteshwar and Mahipal
> > Challa. Maybe both of those emails didn't come on top of each other.
>
> I missed one.
>
> > > Why pushing? What is your fear exactly?
> >
> > Just want to make the transfer of ownership official in the mainline.
> > There's no fear I just normally wanted to send a reminder to you to
> > pick as it might be missed inclusion. If there's a guarantee that all
> > ack'ed patches will go automatically then I won't be sending any
> > reminder emails in future.
>
> It's OK to send reminder, but please wait at least a week.
>
> > > Why the new maintainers candidates have 0 and 2 contributions
> > > in git history?
> > >
> > These drivers requires some maintenance and likely expanded to support
> > future chips from Marvell. So you will not see contributions right
> > away.
> > So to become a maintainer one has to have prior contributions ?
>
> Yes: http://doc.dpdk.org/guides/contributing/patches.html#maintainers-and-sub-trees
> "
> Maintainers should have demonstrated a reasonable level of contributions or reviews to the component area. The maintainer should be confirmed by an ack from an established contributor.
> "

How was this done without prior contributions?
http://git.dpdk.org/dpdk/commit/MAINTAINERS?id=238e3167ca869abf44fa50ead022d7fc3b99605b

The commit log says that he is a "new developer". So am I technically
to the community.

Also the thing that puzzles me most is its a driver specific to
Marvell and both the previous maintainers are ok with the transfer but
a strange rule that probably applies to generic things of dpdk code
(which really makes sense there) comes as a blocker.
And how does new driver go?
Lets say if either myself or veerasena have pushed a new driver now
who has to ack it for inclusion ? Isn't that like a chicken and egg
problem.

I am looking for clarifications as things are different in other open
source communities.

>
> > Kind of makes odd sense as these drivers are specific to our chip and we
> > are changing ownership for maintaining them.
> > I do not understand why you have an issue here?
> > You don't like reminding then I understand.
>
> I don't like how you push new unconfirmed maintainers.

Could you please explain above commit made into new maintainer?

>
>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-18  4:15             ` Radha Mohan
@ 2020-11-18  9:15               ` Thomas Monjalon
  2020-11-18 21:31                 ` Radha Mohan
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Monjalon @ 2020-11-18  9:15 UTC (permalink / raw)
  To: Radha Mohan
  Cc: Jerin Jacob Kollanukkaran, Mahipal Challa, Radha Chintakuntla,
	Satha Koteswara Rao Kottidi, dev, Veerasenareddy Burru,
	Satananda Burla, david.marchand

18/11/2020 05:15, Radha Mohan:
> On Mon, Nov 16, 2020 at 12:28 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > 16/11/2020 19:28, Radha Mohan:
> > > On Fri, Nov 13, 2020 at 2:39 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > > 13/11/2020 20:18, Radha Mohan:
> > > > > On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> > > > > From: Radha Mohan <mohun106@gmail.com>
> > > > > Sent: Tuesday, November 10, 2020 11:44 PM
> > > > > > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > > > > > >
> > > > > > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > > > > > >
> > > > > > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > > > [...]
> > > > > > >Adding previous maintainers to ack.
> > > > > > Acked-by: Mahipal Challa <mchalla@marvell.com
> > > > > >
> > > > > > Thanks,
> > > > > > Mahipal
> > > > >
> > > > > Hi Thomas,
> > > > > Could you please pick this patch ?
> > > >
> > > > This low-priority patch has been sent 4 days ago,
> > > > and was acked by only 1 maintainer 3 days ago.
> > > >
> > >
> > > It was acked by both the old maintainers Satha Koteshwar and Mahipal
> > > Challa. Maybe both of those emails didn't come on top of each other.
> >
> > I missed one.
> >
> > > > Why pushing? What is your fear exactly?
> > >
> > > Just want to make the transfer of ownership official in the mainline.
> > > There's no fear I just normally wanted to send a reminder to you to
> > > pick as it might be missed inclusion. If there's a guarantee that all
> > > ack'ed patches will go automatically then I won't be sending any
> > > reminder emails in future.
> >
> > It's OK to send reminder, but please wait at least a week.
> >
> > > > Why the new maintainers candidates have 0 and 2 contributions
> > > > in git history?
> > > >
> > > These drivers requires some maintenance and likely expanded to support
> > > future chips from Marvell. So you will not see contributions right
> > > away.
> > > So to become a maintainer one has to have prior contributions ?
> >
> > Yes: http://doc.dpdk.org/guides/contributing/patches.html#maintainers-and-sub-trees
> > "
> > Maintainers should have demonstrated a reasonable level of contributions or reviews to the component area. The maintainer should be confirmed by an ack from an established contributor.
> > "
> 
> How was this done without prior contributions?
> http://git.dpdk.org/dpdk/commit/MAINTAINERS?id=238e3167ca869abf44fa50ead022d7fc3b99605b

I asked a confirmation:
http://inbox.dpdk.org/dev/1985242.AXpOyGb66D@thomas/


> The commit log says that he is a "new developer". So am I technically
> to the community.
> 
> Also the thing that puzzles me most is its a driver specific to
> Marvell and both the previous maintainers are ok with the transfer but
> a strange rule that probably applies to generic things of dpdk code
> (which really makes sense there) comes as a blocker.

Did I say it is a blocker?

> And how does new driver go?

For new drivers, the new maintainers arrive by contributing new code.

> Lets say if either myself or veerasena have pushed a new driver now
> who has to ack it for inclusion ? Isn't that like a chicken and egg
> problem.

Nobody has to ack for a new driver.

> I am looking for clarifications as things are different in other open
> source communities.
> 
> > > Kind of makes odd sense as these drivers are specific to our chip and we
> > > are changing ownership for maintaining them.
> > > I do not understand why you have an issue here?
> > > You don't like reminding then I understand.
> >
> > I don't like how you push new unconfirmed maintainers.
> 
> Could you please explain above commit made into new maintainer?

Listen: I say I don't like how pushy you are, that's all.
In general it's very good to have new contributors.
But please think how you can help instead of just requesting.
We have a lot more important things to care at the moment
to close the big release 20.11. If you want to help, you are welcome.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-18  9:15               ` Thomas Monjalon
@ 2020-11-18 21:31                 ` Radha Mohan
  0 siblings, 0 replies; 12+ messages in thread
From: Radha Mohan @ 2020-11-18 21:31 UTC (permalink / raw)
  To: Thomas Monjalon
  Cc: Jerin Jacob Kollanukkaran, Mahipal Challa, Radha Chintakuntla,
	Satha Koteswara Rao Kottidi, dev, Veerasenareddy Burru,
	Satananda Burla, David Marchand

On Wed, Nov 18, 2020 at 1:15 AM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> 18/11/2020 05:15, Radha Mohan:
> > On Mon, Nov 16, 2020 at 12:28 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > 16/11/2020 19:28, Radha Mohan:
> > > > On Fri, Nov 13, 2020 at 2:39 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> > > > > 13/11/2020 20:18, Radha Mohan:
> > > > > > On Tue, Nov 10, 2020 at 11:57 PM Mahipal Challa <mchalla@marvell.com> wrote:
> > > > > > From: Radha Mohan <mohun106@gmail.com>
> > > > > > Sent: Tuesday, November 10, 2020 11:44 PM
> > > > > > > On Mon, Nov 9, 2020 at 4:20 PM Radha Mohan Chintakuntla <radhac@marvell.com> wrote:
> > > > > > > >
> > > > > > > > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> > > > > > > >
> > > > > > > > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > > > > [...]
> > > > > > > >Adding previous maintainers to ack.
> > > > > > > Acked-by: Mahipal Challa <mchalla@marvell.com
> > > > > > >
> > > > > > > Thanks,
> > > > > > > Mahipal
> > > > > >
> > > > > > Hi Thomas,
> > > > > > Could you please pick this patch ?
> > > > >
> > > > > This low-priority patch has been sent 4 days ago,
> > > > > and was acked by only 1 maintainer 3 days ago.
> > > > >
> > > >
> > > > It was acked by both the old maintainers Satha Koteshwar and Mahipal
> > > > Challa. Maybe both of those emails didn't come on top of each other.
> > >
> > > I missed one.
> > >
> > > > > Why pushing? What is your fear exactly?
> > > >
> > > > Just want to make the transfer of ownership official in the mainline.
> > > > There's no fear I just normally wanted to send a reminder to you to
> > > > pick as it might be missed inclusion. If there's a guarantee that all
> > > > ack'ed patches will go automatically then I won't be sending any
> > > > reminder emails in future.
> > >
> > > It's OK to send reminder, but please wait at least a week.
> > >
> > > > > Why the new maintainers candidates have 0 and 2 contributions
> > > > > in git history?
> > > > >
> > > > These drivers requires some maintenance and likely expanded to support
> > > > future chips from Marvell. So you will not see contributions right
> > > > away.
> > > > So to become a maintainer one has to have prior contributions ?
> > >
> > > Yes: http://doc.dpdk.org/guides/contributing/patches.html#maintainers-and-sub-trees
> > > "
> > > Maintainers should have demonstrated a reasonable level of contributions or reviews to the component area. The maintainer should be confirmed by an ack from an established contributor.
> > > "
> >
> > How was this done without prior contributions?
> > http://git.dpdk.org/dpdk/commit/MAINTAINERS?id=238e3167ca869abf44fa50ead022d7fc3b99605b
>
> I asked a confirmation:
> http://inbox.dpdk.org/dev/1985242.AXpOyGb66D@thomas/

Right. But in this case I got different.

>
>
> > The commit log says that he is a "new developer". So am I technically
> > to the community.
> >
> > Also the thing that puzzles me most is its a driver specific to
> > Marvell and both the previous maintainers are ok with the transfer but
> > a strange rule that probably applies to generic things of dpdk code
> > (which really makes sense there) comes as a blocker.
>
> Did I say it is a blocker?

Maybe I got that impression from your comments.

>
> > And how does new driver go?
>
> For new drivers, the new maintainers arrive by contributing new code.
>
> > Lets say if either myself or veerasena have pushed a new driver now
> > who has to ack it for inclusion ? Isn't that like a chicken and egg
> > problem.
>
> Nobody has to ack for a new driver.
>
> > I am looking for clarifications as things are different in other open
> > source communities.
> >
> > > > Kind of makes odd sense as these drivers are specific to our chip and we
> > > > are changing ownership for maintaining them.
> > > > I do not understand why you have an issue here?
> > > > You don't like reminding then I understand.
> > >
> > > I don't like how you push new unconfirmed maintainers.
> >
> > Could you please explain above commit made into new maintainer?
>
> Listen: I say I don't like how pushy you are, that's all.
> In general it's very good to have new contributors.
> But please think how you can help instead of just requesting.
> We have a lot more important things to care at the moment
> to close the big release 20.11. If you want to help, you are welcome.
>

ok fair enough. For my defence i wasn't being pushy. So hopefully this
updation can be taken once you are able to do after 20.11.

>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [PATCH] maintainers: Update for OcteonTx2 DMA and EP
  2020-11-11  7:56   ` Mahipal Challa
  2020-11-13 19:18     ` Radha Mohan
@ 2020-11-22 11:28     ` Thomas Monjalon
  1 sibling, 0 replies; 12+ messages in thread
From: Thomas Monjalon @ 2020-11-22 11:28 UTC (permalink / raw)
  To: Radha Mohan, Radha Chintakuntla, Veerasenareddy Burru
  Cc: Satha Koteswara Rao Kottidi, dev, Satananda Burla,
	Jerin Jacob Kollanukkaran, Mahipal Challa

> > Replace the maintainers for OcteonTx2 DMA and EP drivers.
> >
> > Signed-off-by: Radha Mohan Chintakuntla <radhac@marvell.com>
> > ---
> >  Marvell OCTEON TX2 DMA
> > -M: Satha Rao <skoteshwar@marvell.com>
> > +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> > +M: Veerasenareddy Burru <vburru@marvell.com>
> >  F: drivers/raw/octeontx2_dma/
> >  F: doc/guides/rawdevs/octeontx2_dma.rst
> >
> >  Marvell OCTEON TX2 EP
> > -M: Mahipal Challa <mchalla@marvell.com>
> > +M: Radha Mohan Chintakuntla <radhac@marvell.com>
> > +M: Veerasenareddy Burru <vburru@marvell.com>
> >  F: drivers/raw/octeontx2_ep/
> >  F: doc/guides/rawdevs/octeontx2_ep.rst
> >
> Acked-by: Satha Rao <skoteshwar@marvell.com>
> Acked-by: Mahipal Challa <mchalla@marvell.com

Applied



^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2020-11-22 11:28 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-10  0:19 [dpdk-dev] [PATCH] maintainers: Update for OcteonTx2 DMA and EP Radha Mohan Chintakuntla
2020-11-10 18:13 ` Radha Mohan
2020-11-10 18:18   ` [dpdk-dev] [EXT] " Satha Koteswara Rao Kottidi
2020-11-11  7:56   ` Mahipal Challa
2020-11-13 19:18     ` Radha Mohan
2020-11-13 22:38       ` Thomas Monjalon
2020-11-16 18:28         ` Radha Mohan
2020-11-16 20:28           ` Thomas Monjalon
2020-11-18  4:15             ` Radha Mohan
2020-11-18  9:15               ` Thomas Monjalon
2020-11-18 21:31                 ` Radha Mohan
2020-11-22 11:28     ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git