From patchwork Wed May 16 08:26:57 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jani Nikula X-Patchwork-Id: 10403015 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 97590602C2 for ; Wed, 16 May 2018 08:24:36 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8817E24B5B for ; Wed, 16 May 2018 08:24:36 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7B508287CC; Wed, 16 May 2018 08:24:36 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00, MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED, TVD_PH_BODY_ACCOUNTS_PRE autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 9F8A024B5B for ; Wed, 16 May 2018 08:24:35 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1B0176E350; Wed, 16 May 2018 08:24:34 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5DA326E350; Wed, 16 May 2018 08:24:33 +0000 (UTC) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 May 2018 01:24:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,404,1520924400"; d="scan'208";a="55654552" Received: from jnikula-mobl2.fi.intel.com (HELO localhost) ([10.237.72.62]) by fmsmga001.fm.intel.com with ESMTP; 16 May 2018 01:24:30 -0700 From: Jani Nikula To: intel-gfx@lists.freedesktop.org Subject: [PATCH] doc: how to become a drm-intel committer Date: Wed, 16 May 2018 11:26:57 +0300 Message-Id: <20180516082657.25711-1-jani.nikula@intel.com> X-Mailer: git-send-email 2.11.0 Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: dim-tools@lists.freedesktop.org, jani.nikula@intel.com, dri-devel@lists.freedesktop.org, Rodrigo Vivi MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP Until now, the drm-intel commit access have been handed out ad hoc, without transparency, consistency, or fairness. With pressure to add more committers, this is no longer tenable, if it ever was. Document the requirements and expectations around becoming a drm-intel committer. The drm-intel maintainers believe that a reasonable level of experience and track record of working on the driver, as well as actively engaging in the community upstream, are necessary before becoming a committer. While the requirements outlined here may seem strict in contrast with many projects, it seems easier to start strict and relax the requirements later on as needed than the other way round. v2: Address some of the concerns brought up by Daniel, and try to align the structure with the proposed igt rules. v3: Update commit message. Cc: Gustavo Padovan Cc: Maarten Lankhorst Cc: Sean Paul Cc: Dave Airlie Cc: Daniel Vetter Cc: dim-tools@lists.freedesktop.org Cc: dri-devel@lists.freedesktop.org Cc: intel-gfx@lists.freedesktop.org Acked-by: Dave Airlie Acked-by: Daniel Vetter Signed-off-by: Jani Nikula Signed-off-by: Joonas Lahtinen Signed-off-by: Rodrigo Vivi --- commit-access.rst | 91 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ index.rst | 1 + 2 files changed, 92 insertions(+) create mode 100644 commit-access.rst diff --git a/commit-access.rst b/commit-access.rst new file mode 100644 index 000000000000..54a7402714cd --- /dev/null +++ b/commit-access.rst @@ -0,0 +1,91 @@ +=============== + Commit Access +=============== + +The drm-misc and drm-intel repositories operate in a maintainer/committer model +with a large pool committers who can push patches in accordance with the stated +merge criteria, and maintainers handling pull requests, topic branches, merges, +and so on. + +This document outlines the requirements for becoming a committer. + +drm-misc +-------- + +See :ref:`drm-misc`. + +drm-intel +--------- + +Criteria +~~~~~~~~ + +Commit rights will be granted to anyone who requests them and fulfills the +following criteria: + +- Has contributed at least 25 patches to i915 driver that have already been + merged upstream. Most of the patches must be non-trivial, not just simple + spelling or style fixes or code movement. + +- Has reviewed at least 25 patches from other developers to i915 driver that + have already been merged upstream. Again, most of the reviewed patches must be + non-trivial. + +- Are actively participating in discussions about their work and areas of + expertise on the project communication channels (the intel-gfx mailing list, + #intel-gfx freenode IRC channel, and freedesktop.org bugzilla). + +- Has been active in the past year (at least some commits or reviews on i915 + driver). + +- Will be regularly contributing further patches. This includes regular + contributors to other parts of the open source graphics stack who only do the + occasional patch within i915 itself. + +- Agrees to use their commit rights in accordance with the documented merge + criteria, tools, and processes. + +The above criteria are in place to encourage and require committers are actively +and broadly engaged upstream, and that they are acquainted and comfortable with +the open collaboration model we have. To ensure the committers have enough +experience to gauge reasonably well how much review a patch needs, and whether +it needs acks from domain experts or maintainers before pushing. + +Access Request +~~~~~~~~~~~~~~ + +Apply for an account (and any other account change requests, including commit +rights if you already have an account) through + +https://www.freedesktop.org/wiki/AccountRequests/ + +Maintainer acks are required to confirm commit rights. Please ping the +maintainers if your request is stuck. + +Maintainers may rate limit adding new committers to ensure there's enough +bandwidth to properly support ramp-up on the tools and processes. In this case, +the maintainers will pledge to add at least two new committers per month, +loosely prioritized based on commits, reviews, and in-flight patches. + +Committers are encouraged to request their commit rights get removed when they +no longer contribute to the project. Commit rights will be automatically revoked +after a year of inactivity (no commits or reviews). Commit rights will be +reinstated when they come back to the project. + +Maintainers and committers should encourage contributors to request commit +rights. + +Code of Conduct +~~~~~~~~~~~~~~~ + +Please be aware the freedesktop.org Code of Conduct also applies to i915: + +https://www.freedesktop.org/wiki/CodeOfConduct/ + +See the MAINTAINERS file for contact details of the i915 maintainers. + +Abuse of commit rights, like engaging in commit fights or willfully pushing +patches that violate the documented merge criteria or process, will also be +handled through the Code of Conduct enforcement process. Violations may lead to +temporary or permanent account or commit rights suspension according to +freedesktop.org umbrella rules. diff --git a/index.rst b/index.rst index d1fd4a14bde0..088b4869dd63 100644 --- a/index.rst +++ b/index.rst @@ -26,6 +26,7 @@ Contents: drm-tip drm-misc drm-intel + commit-access getting-started dim qf