diff --git a/MAINTAINERS.yaml b/MAINTAINERS.yaml index c8b8957496..02a8b3eb7b 100644 --- a/MAINTAINERS.yaml +++ b/MAINTAINERS.yaml @@ -1,23 +1,22 @@ # See CONTRIBUTING.md for details of gem5's contribution process. # # This file contains a list of gem5's subsystems and their -# maintainers. The key used to identifity a subsystem should be used -# as a tag in commit messages targetting that subsystem. At least one -# (not all) of these maintainers must review the patch before it can -# be pushed. These people will automatically be emailed when you -# upload the patch to Gerrit (https://gem5-review.googlesource.com). -# These subsystem keys mostly follow the directory structure. +# maintainers. The key used to identify a subsystem should be used +# as a tag in commit messages targeting that subsystem. Via our GitHub +# Pull Request system (https://github.com/gem5/gem5/pulls) a maintainer +# of the subsystem impacted by a pull request contribution will be added +# as an assignee to that pull request. Their role is be to referee the +# contribution (add a review, assign reviewers, suggest changes, etc.), then +# merge the contribution into the gem5 develop branch when they are satisfied +# with the change. # -# Maintainers have the following responsibilities: -# 1. That at least one maintainer of each subsystem reviews all -# changes to that subsystem (they will be automatically tagged and -# emailed on each new change). -# 2. They will complete your reviews in a timely manner (within a few -# business days). -# 3. They pledge to uphold gem5's community standards and its code of -# conduct by being polite and professional in their code -# reviews. See CODE-OF-CONDUCT.md. +# Maintainers assigned to a pull request are expected to acknowledge their +# assignment in 2 business days and to fully begin refereeing the contribution +# within a business week. # +# Maintainers pledge to uphold gem5's community standards and its code of +# conduct by being polite and professional in their interactions with +# contributors. See CODE-OF-CONDUCT.md. # # Entries in this file have the following format: # key: @@ -27,94 +26,73 @@ # maintainers: # - John Doe # - Jane Doe -# +# experts: +# - Jack Doe +# - Jill Doe # # The status field should have one of the following values: # - maintained: The component has an active maintainer. # - orphaned: The component is looking for a new owner. - - -pmc: - desc: >- - PMC Members (general maintainers): - status: maintained - maintainers: - - Andreas Sandberg - - Brad Beckmann - - David Wood - - Gabe Black - - Giacomo Travaglini - - Jason Lowe-Power (chair) - - Matt Sinclair - - Tony Gutierrez - - Steve Reinhardt +# +# The experts field is optional and used to identify people who are +# knowledgeable about the subsystem but are not responsible for it. Those +# listed as an expert are typically good to add as a reviewer for pull requests +# targeting that subsystem. arch: desc: >- General architecture-specific components - status: maintained - maintainers: - - Gabe Black + status: orphaned arch-arm: status: maintained maintainers: - - Andreas Sandberg - Giacomo Travaglini + - Andreas Sandberg arch-gcn3: status: maintained maintainers: - - Matt Poremba - Matt Sinclair + - Matt Porema arch-vega: status: maintained maintainers: - - Matt Poremba - Matt Sinclair + - Matt Porema arch-mips: status: orphaned arch-power: - status: maintained - maintainers: - - Boris Shingarov + status: orphaned arch-riscv: status: orphaned arch-sparc: - status: maintained - maintainers: - - Gabe Black + status: orphaned arch-x86: - status: maintained - maintainers: - - Gabe Black + status: orphaned base: - status: maintained - maintainers: - - Bobby Bruce - - Daniel Carvalho + status: orphaned base-stats: status: orphaned configs: - status: maintained - maintainers: + status: orphaned + experts: - Jason Lowe-Power cpu: desc: >- General changes to all CPU models (e.g., BaseCPU) - status: maintained - maintainers: - - Gabe Black + status: orphaned + experts: - Jason Lowe-Power cpu-kvm: @@ -123,23 +101,18 @@ cpu-kvm: - Andreas Sandberg cpu-minor: - status: maintained - maintainers: - - Zhengrong Wang + status: orphaned cpu-o3: status: orphaned cpu-simple: - status: maintained - maintainers: + status: orphaned + experts: - Jason Lowe-Power - - Gabe Black dev: - status: maintained - maintainers: - - Gabe Black + status: orphaned dev-hsa: status: maintained @@ -159,47 +132,41 @@ dev-virtio: dev-arm: status: maintained maintainers: - - Andreas Sandberg - Giacomo Travaglini + - Andreas Sandberg doc: - status: maintained - maintainers: - - Bobby Bruce + status: orphaned ext: desc: >- Components external to gem5 - status: maintained - maintainers: - - Bobby Bruce + status: orphaned + experts: - Jason Lowe-Power ext-testlib: - status: maintained - maintainers: - - Bobby Bruce - - Hoa Nguyen + status: orphaned + experts: + - Bobby R. Bruce fastmodel: desc: >- Changes relating to ARM Fast Models - status: maintained - maintainers: - - Gabe Black + status: orphaned gpu-compute: status: maintained maintainers: - Matt Poremba - - Matt Sinclair learning-gem5: desc: >- The code and configs for the Learning gem5 book - status: maintained - maintainers: + status: orphaned + experts: - Jason Lowe-Power + - Bobby R. Bruce stdlib: desc: >- @@ -211,74 +178,64 @@ stdlib: mem: desc: >- General memory system (e.g., XBar, Packet) - status: maintained - maintainers: - - Nikos Nikoleris + status: orphaned mem-cache: desc: >- Classic caches and coherence - status: maintained - maintainers: - - Nikos Nikoleris - - Daniel Carvalho + status: orphaned mem-dram: - status: maintained - maintainers: - - Nikos Nikoleris + status: orphaned mem-garnet: desc: >- Garnet subcomponent of Ruby - status: maintained - maintainers: - - Srikant Bharadwaj + status: orphaned mem-ruby: desc: >- Ruby structures and protocols status: maintained maintainers: - - Jason Lowe-Power - Matt Sinclair + experts: + - Jason Lowe-Power misc: desc: >- Anything outside of the other categories - status: maintained - maintainers: - - Bobby Bruce + status: orphaned + experts: - Jason Lowe-Power python: desc: >- Python SimObject wrapping and infrastructure - status: maintained - maintainers: - - Andreas Sandberg + status: orphaned + experts: - Jason Lowe-Power + - Andreas Sandberg resources: desc: >- The gem5-resources repo with auxiliary resources for simulation status: maintained maintainers: - - Bobby Bruce + - Bobby R. Bruce + experts: - Jason Lowe-Power scons: desc: >- Build system - status: maintained - maintainers: - - Gabe Black + status: orphaned sim: desc: >- General simulation components - status: maintained - maintainers: + status: orphaned + experts: - Jason Lowe-Power sim-se: @@ -289,48 +246,40 @@ sim-se: system-arm: status: maintained maintainers: - - Andreas Sandberg - Giacomo Travaglini + - Andreas Sandberg systemc: desc: >- Code for the gem5 SystemC implementation and interface - status: maintained - maintainers: - - Gabe Black + status: orphaned tests: desc: >- testing changes status: maintained maintainers: - - Bobby Bruce + - Bobby R. Bruce util: - status: maintained - maintainers: - - Gabe Black + status: orphaned util-docker: status: maintained maintainers: - - Bobby Bruce + - Bobby R. Bruce util-m5: - status: maintained - maintainers: - - Gabe Black + status: orphaned util-gem5art: - status: maintained - maintainers: - - Bobby Bruce - - Jason Lowe-Power + status: orphaned website: desc: >- The gem5-website repo which contains the gem5.org site status: maintained maintainers: - - Bobby Bruce - - Hoa Nguyen + - Bobby R. Bruce + experts: + - Jason Lowe-Power diff --git a/util/git-commit-msg.py b/util/git-commit-msg.py index 12baad8c19..c12b0cd215 100755 --- a/util/git-commit-msg.py +++ b/util/git-commit-msg.py @@ -100,8 +100,7 @@ def _validateTags(commit_header): maintainer_dict = maintainers.Maintainers.from_file() valid_tags = [tag for tag, _ in maintainer_dict] - # Remove non-tag 'pmc' and add special tags not in MAINTAINERS.yaml - valid_tags.remove("pmc") + # Add special tags not in MAINTAINERS.yaml valid_tags.extend(["RFC", "WIP"]) tags = "".join(commit_header.split(":")[0].split()).split(",")