These here patches are the result of chasing a bug in the TSX Force Abort
patches.
The first patch is simple and fixes the issue and should be marked /urgent and
backported.
The rest is the result of having a hard look at how all that event scheduling
stuff worked while trying to figure out wth was going wrong.
Stephane, since you were involved with writing much of that code, can you
please have a good look? I _think_ I got it, but... :-)
On Thu, Mar 14, 2019 at 6:11 AM Peter Zijlstra <[email protected]> wrote:
>
> These here patches are the result of chasing a bug in the TSX Force Abort
> patches.
>
> The first patch is simple and fixes the issue and should be marked /urgent and
> backported.
>
> The rest is the result of having a hard look at how all that event scheduling
> stuff worked while trying to figure out wth was going wrong.
>
> Stephane, since you were involved with writing much of that code, can you
> please have a good look? I _think_ I got it, but... :-)
>
Will do today.
But you did not send me PATCH 1/8 apparently.
On Fri, Mar 15, 2019 at 12:15 AM Stephane Eranian <[email protected]> wrote:
>
> On Thu, Mar 14, 2019 at 6:11 AM Peter Zijlstra <[email protected]> wrote:
> >
> > These here patches are the result of chasing a bug in the TSX Force Abort
> > patches.
> >
> > The first patch is simple and fixes the issue and should be marked /urgent and
> > backported.
> >
> > The rest is the result of having a hard look at how all that event scheduling
> > stuff worked while trying to figure out wth was going wrong.
> >
> > Stephane, since you were involved with writing much of that code, can you
> > please have a good look? I _think_ I got it, but... :-)
> >
> Will do today.
> But you did not send me PATCH 1/8 apparently.
Correction : I got it. Mail client had just moved it somewhere else!
On Fri, Mar 15, 2019 at 12:15:55AM -0700, Stephane Eranian wrote:
> On Fri, Mar 15, 2019 at 12:15 AM Stephane Eranian <[email protected]> wrote:
> >
> > On Thu, Mar 14, 2019 at 6:11 AM Peter Zijlstra <[email protected]> wrote:
> > >
> > > These here patches are the result of chasing a bug in the TSX Force Abort
> > > patches.
> > >
> > > The first patch is simple and fixes the issue and should be marked /urgent and
> > > backported.
> > >
> > > The rest is the result of having a hard look at how all that event scheduling
> > > stuff worked while trying to figure out wth was going wrong.
> > >
> > > Stephane, since you were involved with writing much of that code, can you
> > > please have a good look? I _think_ I got it, but... :-)
> > >
> > Will do today.
> > But you did not send me PATCH 1/8 apparently.
>
> Correction : I got it. Mail client had just moved it somewhere else!
Also please look at all 8, I'm reasonably sure 1/8 is correct, but it is
rather more ugly than I'd wish. So the remaining 7 clean it all up (and
effectively revert 1 again) but are much more invasive.