Message ID | 20230327185532.2354250-1-Liam.Howlett@oracle.com (mailing list archive) |
---|---|
Headers | show |
Series | Fix VMA tree modification under mmap read lock | expand |
On Mon, 27 Mar 2023 14:55:24 -0400 "Liam R. Howlett" <Liam.Howlett@oracle.com> wrote: > These patches have been in -next since next-20230301, and have received > intensive testing in Android as part of the RCU page fault patchset. > They were also sent as part of the "Per-VMA locks" v4 patch series. > Patches 1 to 7 are bug fixes for RCU mode of the tree and patch 8 enables > RCU mode for the tree. What's happening here? I assume you've decided that the first 8 patches of the "Per-VMA locks v4" series should be fast-tracked into 6.3-rcX and backported? And we retain the rest of that series for 6.4-rc1? Patch [3/8] hasn't come through to me, to linux-mm or to linux-kernel.
* Andrew Morton <akpm@linux-foundation.org> [230327 15:35]: > On Mon, 27 Mar 2023 14:55:24 -0400 "Liam R. Howlett" <Liam.Howlett@oracle.com> wrote: > > > These patches have been in -next since next-20230301, and have received > > intensive testing in Android as part of the RCU page fault patchset. > > They were also sent as part of the "Per-VMA locks" v4 patch series. > > Patches 1 to 7 are bug fixes for RCU mode of the tree and patch 8 enables > > RCU mode for the tree. > > What's happening here? I assume you've decided that the first 8 > patches of the "Per-VMA locks v4" series should be fast-tracked into > 6.3-rcX and backported? And we retain the rest of that series for > 6.4-rc1? Yes, they need to be backported and fast tracked to fix the issue syzbot found. > > Patch [3/8] hasn't come through to me, to linux-mm or to linux-kernel. Should arrive shortly, I received it from one of the ML.
On 3/27/23 21:48, Liam R. Howlett wrote: > * Andrew Morton <akpm@linux-foundation.org> [230327 15:35]: >> On Mon, 27 Mar 2023 14:55:24 -0400 "Liam R. Howlett" <Liam.Howlett@oracle.com> wrote: >> >> > These patches have been in -next since next-20230301, and have received >> > intensive testing in Android as part of the RCU page fault patchset. >> > They were also sent as part of the "Per-VMA locks" v4 patch series. >> > Patches 1 to 7 are bug fixes for RCU mode of the tree and patch 8 enables >> > RCU mode for the tree. >> >> What's happening here? I assume you've decided that the first 8 >> patches of the "Per-VMA locks v4" series should be fast-tracked into >> 6.3-rcX and backported? And we retain the rest of that series for >> 6.4-rc1? > > Yes, they need to be backported and fast tracked to fix the issue syzbot > found. Stable usually wants the "mainline first" which means fast tracking first, then once it's in mainline, they pick it and annotate with mainline commit id. One question is how Linus would feel about this now for rc5. Another question is if we should really deviate in the patch 8/8 backport just because it's not necessary for stable. Generally they would also prefer not to deviate, unless there's a strong reason. >> >> Patch [3/8] hasn't come through to me, to linux-mm or to linux-kernel. > > Should arrive shortly, I received it from one of the ML. > >
* Vlastimil Babka <vbabka@suse.cz> [230328 05:11]: > On 3/27/23 21:48, Liam R. Howlett wrote: > > * Andrew Morton <akpm@linux-foundation.org> [230327 15:35]: > >> On Mon, 27 Mar 2023 14:55:24 -0400 "Liam R. Howlett" <Liam.Howlett@oracle.com> wrote: > >> > >> > These patches have been in -next since next-20230301, and have received > >> > intensive testing in Android as part of the RCU page fault patchset. > >> > They were also sent as part of the "Per-VMA locks" v4 patch series. > >> > Patches 1 to 7 are bug fixes for RCU mode of the tree and patch 8 enables > >> > RCU mode for the tree. > >> > >> What's happening here? I assume you've decided that the first 8 > >> patches of the "Per-VMA locks v4" series should be fast-tracked into > >> 6.3-rcX and backported? And we retain the rest of that series for > >> 6.4-rc1? > > > > Yes, they need to be backported and fast tracked to fix the issue syzbot > > found. > > Stable usually wants the "mainline first" which means fast tracking first, > then once it's in mainline, they pick it and annotate with mainline commit id. Right. I meant these patches won't cleanly apply to 6.1/6.2 and will need more than just a cherry-pick due to the vma iterator changes. I have those modified patches ready to go as well. > > One question is how Linus would feel about this now for rc5. > > Another question is if we should really deviate in the patch 8/8 backport > just because it's not necessary for stable. Generally they would also prefer > not to deviate, unless there's a strong reason. Just to clarify, the change is to remove something that isn't necessary at all. > > >> > >> Patch [3/8] hasn't come through to me, to linux-mm or to linux-kernel. > > > > Should arrive shortly, I received it from one of the ML. > > > > > > > -- > maple-tree mailing list > maple-tree@lists.infradead.org > https://lists.infradead.org/mailman/listinfo/maple-tree
* Liam R. Howlett <Liam.Howlett@Oracle.com> [230328 09:02]: > * Vlastimil Babka <vbabka@suse.cz> [230328 05:11]: > > On 3/27/23 21:48, Liam R. Howlett wrote: > > > * Andrew Morton <akpm@linux-foundation.org> [230327 15:35]: > > >> On Mon, 27 Mar 2023 14:55:24 -0400 "Liam R. Howlett" <Liam.Howlett@oracle.com> wrote: > > >> > > >> > These patches have been in -next since next-20230301, and have received > > >> > intensive testing in Android as part of the RCU page fault patchset. > > >> > They were also sent as part of the "Per-VMA locks" v4 patch series. > > >> > Patches 1 to 7 are bug fixes for RCU mode of the tree and patch 8 enables > > >> > RCU mode for the tree. > > >> > > >> What's happening here? I assume you've decided that the first 8 > > >> patches of the "Per-VMA locks v4" series should be fast-tracked into > > >> 6.3-rcX and backported? And we retain the rest of that series for > > >> 6.4-rc1? > > > > > > Yes, they need to be backported and fast tracked to fix the issue syzbot > > > found. > > > > Stable usually wants the "mainline first" which means fast tracking first, > > then once it's in mainline, they pick it and annotate with mainline commit id. > > Right. I meant these patches won't cleanly apply to 6.1/6.2 and will > need more than just a cherry-pick due to the vma iterator changes. I > have those modified patches ready to go as well. > > > > > One question is how Linus would feel about this now for rc5. > > > > Another question is if we should really deviate in the patch 8/8 backport > > just because it's not necessary for stable. Generally they would also prefer > > not to deviate, unless there's a strong reason. > > Just to clarify, the change is to remove something that isn't necessary > at all. > Andrew, I just wanted to know where we stand with these patches? I understand that it's late in the cycle, but this is a bug that affects 6.1, 6.2, 6.3-rc5 and can be triggered from userspace. I'm asking because the LTS 6.1 is starting to be picked up by distributions, although I don't know the scale of the install, and getting these upstream will allow for the backported fixes to be picked up by stable quicker. Thanks, Liam
On Mon, 3 Apr 2023 15:44:43 -0400 "Liam R. Howlett" <Liam.Howlett@Oracle.com> wrote:
> I just wanted to know where we stand with these patches?
They're in the mm.git hotfixes queue. I'll be asking Linus
to pull them this week.