blob: 0180641be3aa5ee7e1db730ac26bbfa847c8ef4d [file] [log] [blame]
Daniel Vetterc15e6932014-10-28 10:54:32 +01001Patches to intel-gpu-tools are very much welcome, we really want this to be the
2universal set of low-level tools and testcases for the Intel kernel gfx driver
3on Linux and similar platforms. So please bring on porting patches, bugfixes,
4improvements for documentation and new tools and testcases.
5
6A short list of contribution guidelines:
7
8- Please submit patches formatted with git send-email/git format-patch or
Rhys Kidddbf5b6f2018-01-22 23:43:56 -05009 equivalent to:
10
11 Development mailing list for IGT GPU Tools <igt-dev@lists.freedesktop.org>
12
13 For a transition period patches are accepted on both the igt-dev mailing list
14 and the former intel-gfx mailing list (with the appropriate patch
15 subjectprefix, see below).
16
17 During the transition period, feel free to send patches to both lists if you
18 have a need to do so, as they will get deduplicated so they only appear and
19 are tested in one Patchwork instance.
Daniel Vetterc15e6932014-10-28 10:54:32 +010020
21 Intel GFX discussion <intel-gfx@lists.freedesktop.org>
22
Rhys Kidddbf5b6f2018-01-22 23:43:56 -050023 Please use --subject-prefix="PATCH i-g-t" so IGT patches are easily
Daniel Vetterc15e6932014-10-28 10:54:32 +010024 identified in the massive amount mails on intel-gfx. To ensure this is always
Daniel Vetter07c33172018-01-11 09:47:52 +010025 done, meson.sh (and autogen.sh) will run:
Daniel Vetterc15e6932014-10-28 10:54:32 +010026
27 git config format.subjectprefix "PATCH i-g-t"
28
Eric Anholteeeb3042017-06-07 16:51:21 -070029 on its first invocation.
Daniel Vetterc15e6932014-10-28 10:54:32 +010030
Sean Pauld4e5b772018-01-08 10:55:20 -050031- intel-gpu-tools is MIT licensed and we require contributions to follow the
Daniel Vetterc15e6932014-10-28 10:54:32 +010032 developer's certificate of origin: http://developercertificate.org/
33
34- When submitting new testcases please follow the naming conventions documented
Sean Pauld4e5b772018-01-08 10:55:20 -050035 in the generated documentation. Also please make full use of all the helpers
36 and convenience macros provided by the igt library. The semantic patch
37 lib/igt.cocci can help with the more automatic conversions.
Daniel Vetterc15e6932014-10-28 10:54:32 +010038
Daniel Vetterc42ab4d2017-07-18 18:00:20 +020039- Patches need to be reviewed on the mailing list. Exceptions only apply for
40 testcases and tooling for drivers with just a single contributor (e.g. vc4).
41 In this case patches must still be submitted to the mailing list first.
Rhys Kidd2db12192018-01-22 23:43:53 -050042 Testcase should preferably be cross-reviewed by the same people who write and
Daniel Vetterc42ab4d2017-07-18 18:00:20 +020043 review the kernel feature itself.
Daniel Vetterc15e6932014-10-28 10:54:32 +010044
45- When patches from new contributors (without commit access) are stuck, for
46 anything related to the regular releases, issues with packaging and
Thomas Woode80922e2014-12-05 14:07:53 +000047 integrating platform support or any other intel-gpu-tools issues, please
48 contact one of the maintainers (listed in the MAINTAINERS file) and cc the
Rhys Kidddbf5b6f2018-01-22 23:43:56 -050049 igt-dev mailing list.
Daniel Vetterc15e6932014-10-28 10:54:32 +010050
Rhys Kidd2db12192018-01-22 23:43:53 -050051- Changes to the testcases are automatically tested. Take the results into
52 account before merging.
Daniel Vetterc15e6932014-10-28 10:54:32 +010053
54Happy hacking!