This shows you the differences between two versions of the page.
Both sides previous revision Previous revision Next revision | Previous revision Next revision Both sides next revision | ||
civilinfrastructureplatform:cipkernelmaintenance [2017/02/10 16:02] bwhct Grammar fix |
civilinfrastructureplatform:cipkernelmaintenance [2023/07/27 11:58] nobuhiro.iwamatsu [Security fixes] |
||
---|---|---|---|
Line 1: | Line 1: | ||
====== Kernel Maintenance ====== | ====== Kernel Maintenance ====== | ||
- | CIP announced two relevant decisions during the second half of 2016: | + | This wiki page describes the policies and recommendations that the CIP kernel maintainers are following in order to maintain the CIP kernel beyond the current periods established by the enterprise industry, in order to meet Industrial Grade requirements. |
- | * At LinuxCon North America 2016 we announced Ben Hutchings as the CIP kernel maintainer. Ben is currently the Debian Long Term Support kernel maintainer and a reputed kernel hacker, specialised in kernel maintainership. | + | |
- | * At Embedded Linux Conference Europe 2016 we announced that our first Super Long Term Support kernel branch will be based on Linux 4.4. | + | |
- | ===== Maintenance policies ===== | + | ===== Kernel maintenance team ===== |
+ | |||
+ | |||
+ | Nobuhiro Iwamatsu, from Toshiba Corporation. and Pavel Machek, from DENX Software Engineering GmbH are the current CIP kernel maintainer. More developers will join in future. | ||
+ | And when we release the CIP kernel, we will sign it with GnuPG by the maintainers. Each public key can be obtained from the following. | ||
+ | * Nobuhiro Iwamatsu ([[http://keys.gnupg.net/pks/lookup?op=get&search=0x32247FBB40AD1FA6|0x5E629EE5232197357B84CF4332247FBB40AD1FA6]]) | ||
+ | * Pavel Machek ([[http://keys.gnupg.net/pks/lookup?op=get&search=0x30E7F06A95DBFAF2|0x4F7CF3BBAF478086 4D35D2FD30E7F06A95DBFAF2]]) | ||
+ | |||
+ | |||
+ | ===== CIP kernel maintenance policies ===== | ||
==== CIP kernel - SLTS kernel ==== | ==== CIP kernel - SLTS kernel ==== | ||
- | **When does CIP define/label a kernel as SLTS (Super Long Term Support)?** | + | === When does CIP define/label a kernel as SLTS (Super Long Term Support)? === |
- | New major versions of commercial Linux distributions are released at 3-4 year intervals, so that typically only 4 versions need to be supported at one time. Given that CIP's support period is meant to be even longer, it won’t be sustainable to extend every LTS branch, but only to take on a new branch every 2-4 years. | + | New major versions of commercial Linux distributions are released at 3-4 year intervals and maintained over 10-13 years, so that typically only 3-4 versions need to be supported at one time. Given that CIP's support period is meant to be even longer, it won’t be sustainable to extend every LTS branch, but only to take on a new branch every 2-4 years. |
The longer the intervals between new SLTS branches, the greater need there will be for CIP or individual members to backport new hardware support (which carries its own risks). This trade-off is perhaps the most difficult issue to decide. | The longer the intervals between new SLTS branches, the greater need there will be for CIP or individual members to backport new hardware support (which carries its own risks). This trade-off is perhaps the most difficult issue to decide. | ||
- | CIP will measure the effort involved in maintaining the kernel branch we already labelled as SLTS (4.4) and study the technical implications of this work during 2017 in order to define when the next SLTS branch should be started. | + | CIP will measure the effort involved in maintaining the kernel branch we already labelled as SLTS (4.4) and study the technical implications of this work during 2018 in order to define when the next SLTS branch should be started. |
- | **How long will 4.4 be maintained?** | + | === How long will SLTS branches be maintained? === |
- | The use cases CIP project is targeting have a life cycle of between 25 and 50 years. In theory, this is the time in which products shipped with the CIP kernel will be under maintenance. | + | The use cases CIP project is targeting have a life cycle of between 25 and 50 years. In theory, this is the time in which products shipped with the CIP kernel will be under maintenance. However, identifying and backporting relevant fixes becomes increasingly difficult as upstream kernel development diverges further from a stable branch. Any given SLTS branch is unlikely to be maintainable for more than 10-20 years. |
- | The Linux kernel 4.4 was [[http://lkml.iu.edu/hypermail/linux/kernel/1601.1/01592.html|released on January 10th 2016]]. It was [[https://kernel.org/releases.html|declared LTS (Long Term Support) by the stable team]] which means they will maintain it for two years (Feb 2018) following the kernel LTS process. After that period, CIP will maintain it. | + | The Linux kernel 4.4 was [[http://lkml.iu.edu/hypermail/linux/kernel/1601.1/01592.html|released on January 10th 2016]]. It was [[https://kernel.org/releases.html|declared LTS (Long Term Support) by the stable team]] which means Greg Kroah-Hartman was supposed to maintain it for two years (Feb 2018) following the kernel LTS process. In September 2017 it was announced that the 4.4 LTS branch will be maintained during 6 years. Another stable maintainer could extend LTS maintenance for some time beyond that. After that period, CIP will maintain it. |
- | ==== Basic maintenance policies ==== | + | The Linux kernel 4.19 was released on October 22nd, 2018. |
- | Initially, the CIP kernel maintenance team will follow [[https://www.kernel.org/doc/Documentation/stable_kernel_rules.txt|stable_kernel_rules.txt]] as the basis for its work. CIP will release sources, not binaries, except for the CIP platforms once we start to build a more complete platform. | + | ==== Suitable changes ==== |
+ | |||
+ | The policy for which changes to accept is based on [[https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/stable-kernel-rules.rst|stable-kernel-rules.rst]]. For reference, these are the rules with numbering added: | ||
+ | |||
+ | - It must be obviously correct and tested. | ||
+ | - It cannot be bigger than 100 lines, with context. | ||
+ | - It must fix only one thing. | ||
+ | - It must fix a real bug that bothers people (not a, "This could be a problem..." type thing). | ||
+ | - It must fix a problem that causes a build error (but not for things marked ''CONFIG_BROKEN''), an oops, a hang, data corruption, a real security issue, or some "oh, that's not good" issue. In short, something critical. | ||
+ | - Serious issues as reported by a user of a distribution kernel may also be considered if they fix a notable performance or interactivity issue. As these fixes are not as obvious and have a higher risk of a subtle regression they should only be submitted by a distribution kernel maintainer and include an addendum linking to a bugzilla entry if it exists and additional information on the user-visible impact. | ||
+ | - New device IDs and quirks are also accepted. | ||
+ | - No "theoretical race condition" issues, unless an explanation of how the race can be exploited is also provided. | ||
+ | - It cannot contain any "trivial" fixes in it (spelling changes, whitespace cleanups, etc). | ||
+ | - It must follow the Documentation/SubmittingPatches rules. | ||
+ | - It or an equivalent fix must already exist in Linus' tree (upstream). | ||
+ | |||
+ | In practice, exceptions are sometimes made to rules 2, 3, 8 and 9, and we will likewise treat those as //should// rather than //must// rules. | ||
+ | |||
+ | After the end of LTS at kernel.org, the fixes made to a SLTS branch | ||
+ | will be more restricted: | ||
+ | |||
+ | * Architecture and hardware support will be limited to the needs of CIP members; fixes for other hardware will not be applied except where that makes it easier to apply the fixes we do need. For the 4.4 branch, the architectures we've committed to so far are arm (32-bit) and x86_64. | ||
+ | * Some features, particularly those labelled as experimental, may also be excluded from support. We have not done so yet. | ||
+ | |||
+ | Unlike an LTS branch, an SLTS branch may include larger changes to support new hardware, to bridge the gap between SLTS branches. However, this should only be done on the newest SLTS branch. | ||
+ | |||
+ | ==== Releases ==== | ||
+ | |||
+ | CIP will release sources, not binaries, except for the CIP platforms once we start to build a more complete platform. The SLTS kernel maintainer will push a signed tag to the kernel git repository for each release. | ||
+ | |||
+ | Once we have [[civilinfrastructureplatform:ciptesting|automated testing]] in place, a release candidate must be tested on all the platforms that are covered, with no regressions, before it is tagged as a release. | ||
+ | |||
+ | As we will be backporting fixes from mainline, which [[https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/stable-api-nonsense.rst|may change in-kernel APIs]], we will not guarantee API or ABI stability within an SLTS branch. It is the responsibility of members to update any out-of-tree components they use in case they are affected by such a change. However, incompatible changes to user-space APIs or ABIs will not be acceptable. | ||
+ | |||
+ | CIP has released new CIP minor kernel version. This depends on the kernel version and flavor. linux-4.4.y is scheduled to be released once a month, linux-4.19.y is scheduled to be released twice a month, and the RT kernel is scheduled to be released once every two months. | ||
+ | You can find the [[https://git.kernel.org/pub/scm/linux/kernel/git/cip/linux-cip.git/|latest CIP kernel]] version in [[https://git.kernel.org|git.kernel.org]]. Each release will be announced in the cip-dev mailing list and occasionally on the new section of the [[https://www.cip-project.org/news|CIP project]] website. | ||
+ | |||
+ | ==== Maintenance Scope ==== | ||
+ | |||
+ | CIP maintainers will maintain and respond to bug reports from CIP members and their developers, but not system administrators or end users in general, about upstream patches. Bugs should be reported upstream. | ||
+ | |||
+ | The embedded systems that CIP will be used in will also often require out-of-tree drivers and will sometimes include other changes of unknown quality to their kernel. These modifications are in general unsupported. If a bug is found in such a modified kernel, Members will first demonstrate that it exists in the CIP source release in order for the CIP maintainers to act on it. | ||
+ | |||
+ | For specific CIP applied patches, the default communication channel will be the [[https://lists.cip-project.org/mailman/listinfo/cip-dev|CIP-dev]] mailing list. Patches sent to the cip-dev mailing list are tracked on [[https://patchwork.kernel.org/project/cip-dev/list/|patchwork]] since October 12th, 2018. | ||
+ | |||
+ | [[https://gitlab.com/cip-project/cip-kernel/cip-kernel-config|Kernel configuration files]] provided by Members will be used as a reference to limit the maintenance scope. Here is a list of recommendations related to kernel configuration for long-term support: | ||
+ | * [[https://lists.cip-project.org/pipermail/cip-dev/2017-May/000263.html]] | ||
+ | * [[https://lists.cip-project.org/pipermail/cip-dev/2017-July/000387.html]] | ||
+ | * [[https://lists.cip-project.org/pipermail/cip-dev/2018-October/001578.html]] | ||
- | ===== Security ===== | ||
==== Security fixes ==== | ==== Security fixes ==== | ||
- | **Out-of-tree drivers** | + | CIP works towards reducing the window of vulnerability to zero. To achieve this goal, CIP will collaborate with the wider kernel community in initiatives like the [[https://kernsec.org/wiki/index.php/Kernel_Self_Protection_Project|Kernel Self Protection Project]]. |
+ | |||
+ | Although this goal may be achievable for CIP, its members may take much longer to release and deploy binary updates, maybe due to valid concerns about the risk of regression or limited opportunities to deploy updates. In the worst case, they may use CIP as an advertising/compliance point - please don't do that! | ||
+ | |||
+ | To track [[https://cve.mitre.org/about/|CVE]] status in each branch and provide this information either to members or publicly some scripts has been [[https://gitlab.com/cip-project/cip-kernel/cip-kernel-sec|developed and published]] by Masami Ichikawa. | ||
+ | |||
+ | ===== Patch review howto ===== | ||
+ | |||
+ | These recommendations has been written by Ben hutchings. | ||
+ | |||
+ | ==== Find the patches ==== | ||
+ | |||
+ | If you are doing this after a stable release, all the patches will be in the git history of the branch. | ||
+ | |||
+ | If you want to do this during the review period before release, you should be able to receive the patches through the [[http://vger.kernel.org/vger-lists.html#stable|stable@vger.kernel.org]] list. This is a high-traffic list, so you may want to filter by subject. However, at the time of writing, Greg's script that sends patches out for review has a bug that causes patches with non-ASCII characters to be dropped. Greg and most other stable maintainers also publish pending patches in a git repository, either as a quilt patch | ||
+ | series or as a rebased git branch. | ||
+ | |||
+ | Pending patches for stable can be found at: | ||
+ | * Greg Kroah-Hartman [[https://git.kernel.org/pub/scm/linux/kernel/git/stable/stable-queue.git/https://git.kernel.org/pub/scm/linux/kernel/git/stable/stable-queue.git/|queue]]. | ||
+ | * Alexander (Sasha) Levin [[https://git.kernel.org/pub/scm/linux/kernel/git/sashal/linux-stable.git/|queue]]. | ||
+ | |||
+ | ==== Filter the patches ==== | ||
+ | |||
+ | Does the patch touch code that's maintained in the CIP branch? (This should be scripted. If not, ignore it and move on.) | ||
+ | |||
+ | ==== Read the commit message ==== | ||
+ | |||
+ | Does the commit message explain why the change is important, i.e. does the explanation seem to fit the criteria for stable? If not, maybe this doesn't belong in the stable branch. But do consider: | ||
+ | * Sometimes an unimportant commit will be included because it's a dependency of an important fix. Look at the following patches to see if they're related. | ||
+ | * Sometimes a commit that didn't seem important at the time turns out to be important later, e.g. cleaning up a function turned out to fix a corner case that was a security flaw. You can check the stable mailing list archives to see if someone requested it and why. | ||
+ | * If the author is Linus, be aware that he often makes security fixes without describing the security implications. | ||
+ | |||
+ | Look for a ''Fixes:'' tag in the commit message, identifying which commit introduced (or only partly fixed) the bug that's being fixed: | ||
+ | * If it's not present, look to see if the free-form text identifies which commit introduced it. | ||
+ | * Is that commit actually present in the stable branch? Don't forget that it might have been backported as part of another stable update. If not, this probably doesn't belong in the stable branch. | ||
+ | * Find the first release containing a commit: | ||
+ | |||
+ | > ''%%git describe --contains --match 'v*' %%**//hash//**'' | ||
+ | |||
+ | * Check whether a stable branch contains a backported commit: | ||
+ | |||
+ | > ''%%git log --pretty=oneline --grep=%%**//hash//**%% v%%**//version//**..stable/linux-**//version//**.y'' | ||
+ | |||
+ | The ''git-check-in-stable'' script from [[https://anonscm.debian.org/cgit/kernel/kernel-team.git/|Debian kernel/kernel-team.git]] provides a shortcut (and slightly nicer output format) for the second command. | ||
+ | |||
+ | ==== Read the code ==== | ||
+ | |||
+ | Patches only have 3 lines of context around the lines being changed, but in some cases you need to consider the whole of each function (or other definition) being changed. So you will often need to look at the files being patched, not just the patches. | ||
+ | * Does the patched code appear to do what the description says? | ||
+ | * Does the description cover all the code changes, or are there other changes which might not be appropriate for stable? | ||
+ | * Ignore white-space changes - they shouldn't be combined with fixes, but at least they won't do any harm). | ||
+ | * If a file contains multiple similar blocks of code, is the right one being patched? | ||
+ | * If the patch changes a function to acquire and then release a lock or other resource, are there any additional paths through the function where the resource can leak? For example, there might be an extra ''return'' statement in the stable branch, compared to the upstream version. | ||
+ | * If the patch adds a ''return'', ''goto'', or ''break'' statement, again check for leaks. | ||
+ | * Sometimes the upstream commit will have been modified (backported) to create the stable patch, because it depended on new or changed APIs, or because a function that needed to be fixed has been merged or split up in some way. If it has been backported, do the modifications make sense? If someone other than the original author backported it, they should have added an explanation in the new commit message. | ||
+ | * If the patch changes the type or semantics of a function, all its callers probably need to be updated. In the stable branch, there might be additional callers compared to upstream. Have any of them been missed? | ||
+ | |||
+ | ==== When you find a problem ==== | ||
+ | |||
+ | If you find a regression that is specific to the stable branch, it should be fixed by revising or rejecting the patch for the stable branch, or (if it has already been applied and released) by adding another stable-specific fix. | ||
- | The embedded systems that CIP will be used in will also often require out-of-tree drivers and will sometimes include other changes of unknown quality to their kernel. These modifications are in general unsupported. If a bug is found in such a modified kernel, Members will first demonstrate that it exists in the CIP source release in order for the CIP maintainers to act on it. | + | If you find a regression that's also present upstream, it should be fixed upstream first. In fact, that might already have happened: |
+ | * If there is already a fix in Linus's git repository, ask for it to be included in a stable update straight away. | ||
+ | * Otherwise, if the patch that introduces the regression hasn't yet been applied then you should ask for it to be deferred (or never applied, if it was reverted upstream). | ||
- | **Security updates** | + | ==== CIP kernel maintenance status ==== |
+ | * [[linux-4.4.y| linux-4.4.y review]] | ||
+ | * new reviews are done in lts-commit-list repository | ||
+ | * [[linux-4.4-failed-patches| linux-4.4.y failed patches]] | ||
- | CIP works towards reducing the window of vulnerability to zero. To achieve this goal, CIP will collaborate with the wider kernel community in initiatives like the [[https://kernsec.org/wiki/index.php/Kernel_Self_Protection_Project|Kernel Self Protection Project]] and [[https://cve.mitre.org/about/|CVE]]. | + | ==== CIP kernel release tutorial ==== |
+ | * [[civilinfrastructureplatform:cipkernelmaintenance_release_tutorial|CIP kernel release tutorial]] | ||
- | Although this goal is achievable for CIP, its members may take much longer to release and deploy binary updates, maybe due to valid concerns about the risk of regression or limited opportunities to deploy updates. In the worst case, they may use CIP as an advertising/compliance point. | + | [[start|Back to the parent page]] |