2024-04-29 11:44:59

by Huang Rui

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers and reviewer

I'm happy to add Gautham and Mario as the co-maintainers, Perry as the
reviewer for amd-pstate driver.

Signed-off-by: Huang Rui <[email protected]>
Cc: Gautham R. Shenoy <[email protected]>
Cc: Mario Limonciello <[email protected]>
Cc: Perry Yuan <[email protected]>
Cc: Ananth Narayan <[email protected]>
---

Hi Rafael,

Recently, I was assigned other task of virtio-gpu support for Xen, so
apology not to review the patches timely. After discussing with our AMD
colleagues, we want to add Gautham and Mario as co-maintainers of this
driver from server and client side. If one of the maintainers ack the
amd-pstate patch, then this patch is good to be accepted from AMD
perspective. And also add Perry as reviewer, he is actively contributing
the patches on this driver for a long time.

We will try to keep the patches reviewed on time in future.

Thanks,
Ray

MAINTAINERS | 3 +++
1 file changed, 3 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index ebf03f5f0619..96644624308a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1062,6 +1062,9 @@ F: drivers/gpu/drm/amd/pm/

AMD PSTATE DRIVER
M: Huang Rui <[email protected]>
+M: Gautham R. Shenoy <[email protected]>
+M: Mario Limonciello <[email protected]>
+R: Perry Yuan <[email protected]>
L: [email protected]
S: Supported
F: Documentation/admin-guide/pm/amd-pstate.rst
--
2.25.1



2024-04-29 14:03:03

by Mario Limonciello

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers and reviewer

On 4/29/2024 06:33, Huang Rui wrote:
> I'm happy to add Gautham and Mario as the co-maintainers, Perry as the
> reviewer for amd-pstate driver.
>
> Signed-off-by: Huang Rui <[email protected]>
> Cc: Gautham R. Shenoy <[email protected]>
> Cc: Mario Limonciello <[email protected]>
> Cc: Perry Yuan <[email protected]>
> Cc: Ananth Narayan <[email protected]>
> ---

Thanks Ray! We of course talked about this offline and I'm happy to
step in. Having coverage of two people is also good for any OOO
situation in the future to make sure patches can be reviewed in a timely
fashion.

Acked-by: Mario Limonciello <[email protected]>

>
> Hi Rafael,
>
> Recently, I was assigned other task of virtio-gpu support for Xen, so
> apology not to review the patches timely. After discussing with our AMD
> colleagues, we want to add Gautham and Mario as co-maintainers of this
> driver from server and client side. If one of the maintainers ack the
> amd-pstate patch, then this patch is good to be accepted from AMD
> perspective. And also add Perry as reviewer, he is actively contributing
> the patches on this driver for a long time.
>
> We will try to keep the patches reviewed on time in future.
>
> Thanks,
> Ray
>
> MAINTAINERS | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ebf03f5f0619..96644624308a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1062,6 +1062,9 @@ F: drivers/gpu/drm/amd/pm/
>
> AMD PSTATE DRIVER
> M: Huang Rui <[email protected]>
> +M: Gautham R. Shenoy <[email protected]>
> +M: Mario Limonciello <[email protected]>
> +R: Perry Yuan <[email protected]>
> L: [email protected]
> S: Supported
> F: Documentation/admin-guide/pm/amd-pstate.rst


2024-04-30 10:36:14

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers and reviewer

On Mon, Apr 29, 2024 at 4:02 PM Mario Limonciello
<[email protected]> wrote:
>
> On 4/29/2024 06:33, Huang Rui wrote:
> > I'm happy to add Gautham and Mario as the co-maintainers, Perry as the
> > reviewer for amd-pstate driver.
> >
> > Signed-off-by: Huang Rui <[email protected]>
> > Cc: Gautham R. Shenoy <[email protected]>
> > Cc: Mario Limonciello <[email protected]>
> > Cc: Perry Yuan <[email protected]>
> > Cc: Ananth Narayan <[email protected]>
> > ---
>
> Thanks Ray! We of course talked about this offline and I'm happy to
> step in. Having coverage of two people is also good for any OOO
> situation in the future to make sure patches can be reviewed in a timely
> fashion.
>
> Acked-by: Mario Limonciello <[email protected]>
>
> >
> > Hi Rafael,
> >
> > Recently, I was assigned other task of virtio-gpu support for Xen, so
> > apology not to review the patches timely. After discussing with our AMD
> > colleagues, we want to add Gautham and Mario as co-maintainers of this
> > driver from server and client side. If one of the maintainers ack the
> > amd-pstate patch, then this patch is good to be accepted from AMD
> > perspective. And also add Perry as reviewer, he is actively contributing
> > the patches on this driver for a long time.
> >
> > We will try to keep the patches reviewed on time in future.

Thank you all, patch applied for 6.10.

BTW, patch series "AMD Pstate Driver Core Performance Boost" doesn't
seem to be ready and given that -rc7 is due this week, I'd rather
postpone until the 6.11 cycle.

2024-04-30 16:43:18

by Mario Limonciello

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers and reviewer

On 4/30/2024 05:35, Rafael J. Wysocki wrote:
> On Mon, Apr 29, 2024 at 4:02 PM Mario Limonciello
> <[email protected]> wrote:
>>
>> On 4/29/2024 06:33, Huang Rui wrote:
>>> I'm happy to add Gautham and Mario as the co-maintainers, Perry as the
>>> reviewer for amd-pstate driver.
>>>
>>> Signed-off-by: Huang Rui <[email protected]>
>>> Cc: Gautham R. Shenoy <[email protected]>
>>> Cc: Mario Limonciello <[email protected]>
>>> Cc: Perry Yuan <[email protected]>
>>> Cc: Ananth Narayan <[email protected]>
>>> ---
>>
>> Thanks Ray! We of course talked about this offline and I'm happy to
>> step in. Having coverage of two people is also good for any OOO
>> situation in the future to make sure patches can be reviewed in a timely
>> fashion.
>>
>> Acked-by: Mario Limonciello <[email protected]>
>>
>>>
>>> Hi Rafael,
>>>
>>> Recently, I was assigned other task of virtio-gpu support for Xen, so
>>> apology not to review the patches timely. After discussing with our AMD
>>> colleagues, we want to add Gautham and Mario as co-maintainers of this
>>> driver from server and client side. If one of the maintainers ack the
>>> amd-pstate patch, then this patch is good to be accepted from AMD
>>> perspective. And also add Perry as reviewer, he is actively contributing
>>> the patches on this driver for a long time.
>>>
>>> We will try to keep the patches reviewed on time in future.
>
> Thank you all, patch applied for 6.10.
>
> BTW, patch series "AMD Pstate Driver Core Performance Boost" doesn't
> seem to be ready and given that -rc7 is due this week, I'd rather
> postpone until the 6.11 cycle.

Yes; I agree with this approach too.

2024-05-06 08:58:08

by Yuan, Perry

[permalink] [raw]
Subject: RE: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers and reviewer

[AMD Official Use Only - General]

Hi Rafael

> -----Original Message-----
> From: Limonciello, Mario <[email protected]>
> Sent: Wednesday, May 1, 2024 12:43 AM
> To: Rafael J. Wysocki <[email protected]>; Huang, Ray
> <[email protected]>; Shenoy, Gautham Ranjal
> <[email protected]>
> Cc: Rafael J . Wysocki <[email protected]>; [email protected];
> [email protected]; Meng, Li (Jassmine) <[email protected]>; Du,
> Xiaojian <[email protected]>; Yuan, Perry <[email protected]>;
> Narayan, Ananth <[email protected]>
> Subject: Re: [PATCH] MAINTAINERS: cpufreq: amd-pstate: Add co-maintainers
> and reviewer
>
> On 4/30/2024 05:35, Rafael J. Wysocki wrote:
> > On Mon, Apr 29, 2024 at 4:02 PM Mario Limonciello
> > <[email protected]> wrote:
> >>
> >> On 4/29/2024 06:33, Huang Rui wrote:
> >>> I'm happy to add Gautham and Mario as the co-maintainers, Perry as
> >>> the reviewer for amd-pstate driver.
> >>>
> >>> Signed-off-by: Huang Rui <[email protected]>
> >>> Cc: Gautham R. Shenoy <[email protected]>
> >>> Cc: Mario Limonciello <[email protected]>
> >>> Cc: Perry Yuan <[email protected]>
> >>> Cc: Ananth Narayan <[email protected]>
> >>> ---
> >>
> >> Thanks Ray! We of course talked about this offline and I'm happy to
> >> step in. Having coverage of two people is also good for any OOO
> >> situation in the future to make sure patches can be reviewed in a
> >> timely fashion.
> >>
> >> Acked-by: Mario Limonciello <[email protected]>
> >>
> >>>
> >>> Hi Rafael,
> >>>
> >>> Recently, I was assigned other task of virtio-gpu support for Xen,
> >>> so apology not to review the patches timely. After discussing with
> >>> our AMD colleagues, we want to add Gautham and Mario as
> >>> co-maintainers of this driver from server and client side. If one of
> >>> the maintainers ack the amd-pstate patch, then this patch is good to
> >>> be accepted from AMD perspective. And also add Perry as reviewer, he
> >>> is actively contributing the patches on this driver for a long time.
> >>>
> >>> We will try to keep the patches reviewed on time in future.
> >
> > Thank you all, patch applied for 6.10.
> >
> > BTW, patch series "AMD Pstate Driver Core Performance Boost" doesn't
> > seem to be ready and given that -rc7 is due this week, I'd rather
> > postpone until the 6.11 cycle.
>
> Yes; I agree with this approach too.

Yeah, 6.11 is fine to me, the CPB patch set still there are a few questions to be addressed, I need more time to finalize the solution.
Thank you, Rafael.

Perry.