diff mbox series

[QEMU,3/3] MAINTAINERS: Add Hyman Huang as maintainer

Message ID 169056029099.1291.9838508091499564970-3@git.sr.ht
State New
Headers show
Series migration: craft the doc comments | expand

Commit Message

~hyman July 26, 2023, 6:10 p.m. UTC
From: Hyman Huang(黄勇) <yong.huang@smartx.com>

I've built interests in dirty-limit and dirty page rate
features and also have been working on projects related
to this subsystem.

Self-recommand myself as a maintainer for this subsystem
so that I can help to improve the dirty-limit algorithm
and review the patches about dirty page rate.

Signed-off-by: Hyman Huang(黄勇) <yong.huang@smartx.com>
---
 MAINTAINERS | 9 +++++++++
 1 file changed, 9 insertions(+)

Comments

Peter Xu July 28, 2023, 6:28 p.m. UTC | #1
On Thu, Jul 27, 2023 at 02:10:09AM +0800, ~hyman wrote:
> From: Hyman Huang(黄勇) <yong.huang@smartx.com>
> 
> I've built interests in dirty-limit and dirty page rate
> features and also have been working on projects related
> to this subsystem.
> 
> Self-recommand myself as a maintainer for this subsystem
> so that I can help to improve the dirty-limit algorithm
> and review the patches about dirty page rate.
> 
> Signed-off-by: Hyman Huang(黄勇) <yong.huang@smartx.com>

Acked-by: Peter Xu <peterx@redhat.com>
Markus Armbruster July 29, 2023, 6:12 a.m. UTC | #2
Recommend to tweak the subject to

    MAINTAINERS: Add section "TITLE"

where TITLE is whatever you use in MAINTAINERS, provided it's short
enough to fit.

~hyman <hyman@git.sr.ht> writes:

> From: Hyman Huang(黄勇) <yong.huang@smartx.com>
>
> I've built interests in dirty-limit and dirty page rate

Recommend to drop the dash.

> features and also have been working on projects related
> to this subsystem.
>
> Self-recommand myself as a maintainer for this subsystem

"Add myself as ..."

> so that I can help to improve the dirty-limit algorithm
> and review the patches about dirty page rate.
>
> Signed-off-by: Hyman Huang(黄勇) <yong.huang@smartx.com>
> ---
>  MAINTAINERS | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 12e59b6b27..d4b1c91096 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3437,6 +3437,15 @@ F: hw/core/clock-vmstate.c
>  F: hw/core/qdev-clock.c
>  F: docs/devel/clocks.rst
>  
> +Dirty-limit and dirty page rate feature

Suggest

   Migration dirty limit and dirty page rate

> +M: Hyman Huang <yong.huang@smartx.com>
> +S: Maintained
> +F: softmmu/dirtylimit.c
> +F: include/sysemu/dirtylimit.h
> +F: migration/dirtyrate.c
> +F: migration/dirtyrate.h
> +F: include/sysemu/dirtyrate.h
> +

Should this go right after section "Migration"?  Or are there other
applications?

>  Usermode Emulation
>  ------------------
>  Overall usermode emulation
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 12e59b6b27..d4b1c91096 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3437,6 +3437,15 @@  F: hw/core/clock-vmstate.c
 F: hw/core/qdev-clock.c
 F: docs/devel/clocks.rst
 
+Dirty-limit and dirty page rate feature
+M: Hyman Huang <yong.huang@smartx.com>
+S: Maintained
+F: softmmu/dirtylimit.c
+F: include/sysemu/dirtylimit.h
+F: migration/dirtyrate.c
+F: migration/dirtyrate.h
+F: include/sysemu/dirtyrate.h
+
 Usermode Emulation
 ------------------
 Overall usermode emulation