> On Feb 9, 2021, at 5:31 PM, Stefano Stabellini <sstabell...@kernel.org> wrote: > > On Tue, 9 Feb 2021, Ian Jackson wrote: >> Jan Beulich writes ("Re: [PATCH v2] xen/arm: fix gnttab_need_iommu_mapping"): >>> On 08.02.2021 21:24, Stefano Stabellini wrote: >> ... >>>> For these cases, I would just follow a simple rule of thumb: >>>> - is the submitter willing to provide the backport? >>>> - is the backport low-risk? >>>> - is the underlying bug important? >>>> >>>> If the answer to all is "yes" then I'd go with it. >>> >>> Personally I disagree, for the very simple reason of the question >>> going to become "Where do we draw the line?" The only non-security >>> backports that I consider acceptable are low-risk changes to allow >>> building with newer tool chains. I know other backports have >>> occurred in the past, and I did voice my disagreement with this >>> having happened. >> >> I think I take a more relaxed view than Jan, but still a much more >> firm line than Stefano. My opinion is that we should make exceptions >> for only bugs of exceptional severity. >> >> I don't think I have seen an argument that this bug is exceptionally >> severe. >> >> For me the fact that you can only experience this bug if you upgrade >> the hardware or significantly change the configuration, means that >> this isn't so serious a bug. > > Yeah, I think that's really the core of this issue. If somebody is > already using 4.12 happily, there is really no reason for them to take > the fix. If somebody is about to use 4.12, then it is a severe issue. > > The view of the group is that nobody should be switching to 4.12 now > because there are newer releases out there. I don't know if that is > true.
I don’t think we have to tell people what to do; rather, we need to tell people what *we* are going to do. We’ve told people that we’ll provide normal bug-fixes until 2020-10-02, and security bugfixes until 2022-04-02. Anyone is welcome to use 4.12 whenever they want to; but if they do, they take the risk that there will be a “normal” bug that doesn’t get fixed by upstream. The patch is available, anyone who wants to can apply it themselves (at their own risk, of course). We don’t seem to have anything explicitly written down anywhere, but whenever someone has come to us using some obsolete version of Xen, we’ve recommended they use the most recent stable release. The one reason someone might use 4.12 over a newer version is if that’s still the one packaged by their distro. In that case, the distro should either update to a newer version, or take on the effort of supporting the old version by backporting bugfixes. If such a distro chooses not to do so, then a user should consider switching to a different distro. The purpose of having such a policy is to avoid needing to have this discussion about every single patch that comes up; and the number was chosen to balance effort for testing and maintainers against value to downstreams. Certainly backposting Just This One patch won’t be a huge amount of work; but I completely sympathize with Jan’s point of view that if we start to make exceptions, people will begin to expect them. We’ve said what we’re going to do, we should just stick with it. Now maybe the 18-month “full-support" window is too small. We seem to have lots of downstreams (at least SuSE and XenServer) who support much older versions; coordinating on versions to designate LTS releases to avoid duplication of effort and share the benefits of that effort with the wider community might be something worth considering. But that’s a completely different discussion. -George