User Tools

Site Tools


en:developers:documentation:submittingpatches

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
Next revision Both sides next revision
en:developers:documentation:submittingpatches [2017/05/12 08:26]
Kalle Valo SubmittingPatches has moved, point it now to the generated HTML output
en:developers:documentation:submittingpatches [2018/06/14 08:10]
Kalle Valo [More patch work references] Change title, "patch work" is easy to confuse with patchwork (the tool)
Line 28: Line 28:
 Cc: Driver maintainers,​ Other Developers</​code>​ Cc: Driver maintainers,​ Other Developers</​code>​
  
-Where [[en/​developers/​maintainers|Maintainers]] contains the list of maintainers for the piece of code you are patching. //Other Developers//​ in this case can be a list of developers who you think may like to review this patch or who changed the code you are working on recently. ​+Where [[en/​developers/​maintainers|Maintainers]] contains the list of maintainers ​and mailing lists for the piece of code you are patching. //Other Developers//​ in this case can be a list of developers ​(or mailing lists) ​who you think may like to review this patch or who changed the code you are working on recently. ​
  
 +Do note that drivers might have special requirements,​ the best is to check them from the corresponding wiki page. Here's a list for few of them:
  
 +  * [[en/​users/​drivers/​ath10k/​submittingpatches|ath10k]]
 ===== Checking state of patches from patchwork ===== ===== Checking state of patches from patchwork =====
  
-All wireless patches are tracked in [[https://​patchwork.kernel.org/​project/​linux-wireless/​list/​|linux-wireless patchwork project]] ​(only exception being ath10k which has its own [[https://​patchwork.kernel.org/​project/​ath10k/​list/?​state=*|ath10k patchwork project]]). From patchwork you can check the state of the patch and to whom it is assigned. Here's a quick link to see all the patches, no matter what's the state:+All wireless patches are tracked in [[https://​patchwork.kernel.org/​project/​linux-wireless/​list/​|linux-wireless patchwork project]]. From patchwork you can check the state of the patch and to whom it is assigned. Here's a quick link to see all the patches, no matter what's the state:
  
 [[https://​patchwork.kernel.org/​project/​linux-wireless/​list/?​state=*]] [[https://​patchwork.kernel.org/​project/​linux-wireless/​list/?​state=*]]
  
-Always avoid contacting maintainers directly, they get way too much email already. Instead use the link above to find your patch and see the status. Only in last resort contact the maintainers,​ and do that by replying to your own patch and ask for status.+Always avoid contacting maintainers directly, they get way too much email already. Instead use the link above to find your patch and see the status. Only in last resort contact the maintainers,​ and do that by replying to your own patch and ask for status. ​Do not top post!
  
 Different patchwork states and their meanings: Different patchwork states and their meanings:
Line 109: Line 111:
 ===== Format of patches ===== ===== Format of patches =====
  
-We prefer patches to be inline-text at the end of the body of the e-mail. ​You can use git-diff or the like to generate ​the patch. Additionally note that we prefer to apply patches with -p1. A header as follows is then acceptable: ​+We prefer patches to be inline-text at the end of the body of the e-mail. ​It's strongly recommended to use git-format-patch and git-send-email tools to submit patches as they use the correct format automatically. Additionally note that we prefer to apply patches with git-am (using the -p1 diff format). A header as follows is then acceptable: ​
  
  
Line 230: Line 232:
 ==== Changelog missing ==== ==== Changelog missing ====
  
-When sending a new version of a patch or patchset you should **always** add a changelog so that maintainer can easily what has changed.+When sending a new version of a patch or patchset you should **always** add a changelog so that maintainer can easily ​see what has changed.
  
 If you have just one patch you can add the changelog after the ''​%%---%%''​ (three dashes) line. If you have just one patch you can add the changelog after the ''​%%---%%''​ (three dashes) line.
  
-If you have multiples patches (called a patchset) add the changelog to the cover letter. You can create the cover letter with switch ''​--cover-letter'':​+If you have multiples patches (called a patchset) add the changelog to the cover letter. You can create the cover letter with the switch ''​--cover-letter'':​
  
   git format-patch --subject-prefix="​PATCH v2" --cover-letter   git format-patch --subject-prefix="​PATCH v2" --cover-letter
 +  ​
 +==== Signed-off-by missing ====
 +
 +Read [[https://​www.kernel.org/​doc/​html/​latest/​process/​submitting-patches.html#​sign-your-work-the-developer-s-certificate-of-origin|Developer'​s Certificate of Origin]] and add Signed-off-by to the commit log.
 +
 +==== Format issues ====
 +
 +Patch is somehow whitespace damaged, for example tabs converted to spaces, extra new lines or other modifications which prevent applying the patch without manual fixing. Or the mail is in HTML format which most of the mailing lists even block silently.
 +
 +The best way to avoid all formatting issues is to use [[https://​www.kernel.org/​pub/​software/​scm/​git/​docs/​git-send-email.html|git send-email]]. See [[en/​developers/​Documentation/​git-guide|linux-wireless git guide]] for more information.
 +
 +==== Fixes line is incorrect ====
 +
 +The correct format for the commit references in Fixes line is the 12 initial digits
 +of the SHA1_ID of the commit, followed by a space, followed by the
 +commit log message header line text enclosed in parentheses and
 +double quotes with no line breaks whatsoever.
 +
 +Example:
 +
 +  Fixes: c742e623e941 ("​mwifiex:​ sdio card reset enhancement"​)
 +
 +==== Commit reference is wrong ====
 +
 +The correct format for the commit references in commit logs is to start with the string "​commit",​ followed by a space, followed 12 initial digits
 +of the SHA1_ID of the commit, followed by a space and followed by the
 +commit log message header line text enclosed in parentheses.
 +
 +Example:
 +
 +<​code>​
 +commit f99a6abe59e096cc2c753e667c19f22022e3bef4
 +Author: Sara Sharon <​sara.sharon@intel.com>​
 +Date:   Sun Mar 5 18:35:02 2017 +0200
 +
 +    iwlwifi: mvm: memset binding before setting values
 +    ​
 +    The changes in commit 9415af7f306b ("​iwlwifi:​ mvm: support new binding
 +    API") assigned values that were later memset to 0.  Move the memset
 +    earlier.
 +    ​
 +    Fixes: 9415af7f306b ("​iwlwifi:​ mvm: support new binding API")
 +    Signed-off-by:​ Sara Sharon <​sara.sharon@intel.com>​
 +    Signed-off-by:​ Luca Coelho <​luciano.coelho@intel.com>​
 +</​code>​
 +
 +==== Commit title is wrong ====
 +
 +The correc tformat for the commit title is name of driver, followed by a colon, followed by a space and then followed by the actual title. You can use ''​git log''​ to check older commits and see what prefix was used:
 +
 +<​code>​
 +$ git log --oneline --follow --no-merges -20 drivers/​net/​wireless/​marvell/​mwifiex/​11ac.c
 +277b024e5e3d mwifiex: move under marvell vendor directory
 +65da33f5557f mwifiex: update Copyright to 2014
 +cf831ffe4473 mwifiex: fix IE parsing issues
 +d51246481c7f mwifiex: save and copy AP's VHT capability info correctly
 +5f6d5983394f mwifiex: add VHT support for TDLS
 +9ed230bcbab7 mwifiex: pass ieee80211_vht_cap to mwifiex_fill_vht_cap_tlv
 +406d702b47a2 mwifiex: improve readability in 11ac mcsmap to maxrate conversion
 +89467d8ca21b mwifiex: make 11ac mcs rate tables global and const
 +7abf4129e6df mwifiex: make use of IEEE80211_VHT_MCS_NOT_SUPPORTED
 +0648f3a4b0e9 mwifiex: correct bss_mode check while appending vht operation IE
 +2b6254dacfe6 mwifiex: use separate AMPDU tx/rx window sizes in 11ac networks
 +83c78da983d6 mwifiex: add support to configure VHT for AP mode
 +a5f390562a37 mwifiex: add 802.11AC support
 +</​code>​
 +
 +==== Too many patches ====
 +
 +The recommend size is 10-12 patches per patchset. More than that it gets difficult for reviewers and maintainers. Of course there'​s no hard rule, for simple patches more than that might be ok but then again for more complex patches even 10 patches per patchset might be too much.
 +
 +==== Resubmit the whole patchset =====
 +
 +Even if just one patch has changed in a patch series resubmit the whole patchset (and remember to increase the version number), do not just resubmit that one changed patch. The reason is that it's difficult to apply patches in correct order when some of them are submitted separately.
 +
 +==== Commit log does not answer "​Why?"​ ====
 +
 +The commit log should //always// answer the question "​Why?"​ and describe the reason what
 +motivated to implement the patch. This is the most important part of the commit log as this helps maintainers,​ backports, distros etc to make decisions if the patch is important for them or not and to what release it should go.
 +
 +The commit log needs to tell why you wrote the patch. If you fixed a bug give a short summary of the bug (can be a long one as well, of course) from user's point of view, and if there'​s a publically available bug report include a link to that. If you are fixing a warning from a compiler or a static checker add the warning from tool. Or if it's just code cleanup or fixing a theoretical issue, and does not have practical user visible changes, mention that also.
 +
 +==== Do not top post and edit your quotes ====
 +
 +Top posting makes following email threads hard to follow and also it makes use of patchwork more difficult, which gets the maintainers grumpy as you are making their work more difficult. So do not top post and instead edit your quotes properly.
 +
 +<​code>​A:​ Because it messes up the order in which people normally read text.
 +Q: Why is top-posting such a bad thing?
 +A: Top-posting.
 +Q: What is the most annoying thing in e-mail?
 +
 +A: No.
 +Q: Should I include quotations after my reply?
 +</​code>​
 +
 +More info: http://​www.idallen.com/​topposting.html
 +
 +==== Use RFC or RFT for patches not ready ====
 +
 +If the patches are not yet ready to be applied by the maintainer mark them as RFC (Request For Comments) or RFC (Request For Test), this way the maintainer can easily see that the patch should not be applied yet. This saves a lot of maintainer'​s time.
 +
  
-===== More patch work references =====+===== More references =====
  
 Here is a list of links to help you write better patches ​ Here is a list of links to help you write better patches ​
Line 246: Line 349:
   * [[http://​linux.yyz.us/​patch-format.html|http://​linux.yyz.us/​patch-format.html]] ​   * [[http://​linux.yyz.us/​patch-format.html|http://​linux.yyz.us/​patch-format.html]] ​
   * [[https://​www.ozlabs.org/​~akpm/​stuff/​tpp.txt|Andrew Morton'​s ''​The perfect patch''​]] ​   * [[https://​www.ozlabs.org/​~akpm/​stuff/​tpp.txt|Andrew Morton'​s ''​The perfect patch''​]] ​
 +  * [[http://​lkml.kernel.org/​r/​20171026223701.GA25649@bhelgaas-glaptop.roam.corp.google.com|Make Bjorn'​s life easier (and grease the path of your patch)]]
en/developers/documentation/submittingpatches.txt ยท Last modified: 2024/02/01 20:12 by Jeff Johnson