Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1711966 > unrolled thread

Re: [RFC GIT PULL rcu/next] RCU/hotplug fix

Started byIngo Molnar <mingo@kernel.org>
First post2017-08-15 10:50 +0200
Last post2017-08-24 16:50 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [RFC GIT PULL rcu/next] RCU/hotplug fix Ingo Molnar <mingo@kernel.org> - 2017-08-15 10:50 +0200
    Re: [RFC GIT PULL rcu/next] RCU/hotplug fix Jeffrey Hugo <jhugo@codeaurora.org> - 2017-08-23 18:30 +0200
      Re: [RFC GIT PULL rcu/next] RCU/hotplug fix Ingo Molnar <mingo@kernel.org> - 2017-08-24 10:00 +0200
        Re: [RFC GIT PULL rcu/next] RCU/hotplug fix "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> - 2017-08-24 16:40 +0200
          Re: [RFC GIT PULL rcu/next] RCU/hotplug fix Jeffrey Hugo <jhugo@codeaurora.org> - 2017-08-24 16:50 +0200

#1711966 — Re: [RFC GIT PULL rcu/next] RCU/hotplug fix

FromIngo Molnar <mingo@kernel.org>
Date2017-08-15 10:50 +0200
SubjectRe: [RFC GIT PULL rcu/next] RCU/hotplug fix
Message-ID<ueFHY-7rw-17@gated-at.bofh.it>
* Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:

> Hello, Ingo,
> 
> This pull request is for an RCU change that permits waiting for grace
> periods started by CPUs late in the process of going offline.  Lack of
> this capability is causing failures:
> 
> http://lkml.kernel.org/r/db9c91f6-1b17-6136-84f0-03c3c2581ab4@codeaurora.org
> 
> Although these failures predate v4.12, and thus are in some sense not
> regressions, it would be good for them to be fixed sooner rather than
> later, hence this RFC pull request.
> 
> This commit was posted to LKML:
> 
> 	http://lkml.kernel.org/r/1500933116-11642-2-git-send-email-paulmck@linux.vnet.ibm.com
> 
> This commit has been exposed to 0day test robot and -next testing
> and is available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
> 
> for you to fetch changes up to a58163d8ca2c8d288ee9f95989712f98473a5ac2:
> 
>   rcu: Migrate callbacks earlier in the CPU-offline timeline (2017-07-25 13:03:43 -0700)
> 
> ----------------------------------------------------------------
> Paul E. McKenney (1):
>       rcu: Migrate callbacks earlier in the CPU-offline timeline
> 
>  include/linux/rcupdate.h |   1 +
>  kernel/cpu.c             |   1 +
>  kernel/rcu/tree.c        | 209 +++++++++++++++++++++++++----------------------
>  3 files changed, 115 insertions(+), 96 deletions(-)

Pulled, thanks Paul!

	Ingo

[toc] | [next] | [standalone]


#1718504

FromJeffrey Hugo <jhugo@codeaurora.org>
Date2017-08-23 18:30 +0200
Message-ID<uhGHw-2Qt-29@gated-at.bofh.it>
In reply to#1711966
On 8/15/2017 2:43 AM, Ingo Molnar wrote:
> 
> * Paul E. McKenney <paulmck@linux.vnet.ibm.com> wrote:
> 
>> Hello, Ingo,
>>
>> This pull request is for an RCU change that permits waiting for grace
>> periods started by CPUs late in the process of going offline.  Lack of
>> this capability is causing failures:
>>
>> http://lkml.kernel.org/r/db9c91f6-1b17-6136-84f0-03c3c2581ab4@codeaurora.org
>>
>> Although these failures predate v4.12, and thus are in some sense not
>> regressions, it would be good for them to be fixed sooner rather than
>> later, hence this RFC pull request.
>>
>> This commit was posted to LKML:
>>
>> 	http://lkml.kernel.org/r/1500933116-11642-2-git-send-email-paulmck@linux.vnet.ibm.com
>>
>> This commit has been exposed to 0day test robot and -next testing
>> and is available in the git repository at:
>>
>>    git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
>>
>> for you to fetch changes up to a58163d8ca2c8d288ee9f95989712f98473a5ac2:
>>
>>    rcu: Migrate callbacks earlier in the CPU-offline timeline (2017-07-25 13:03:43 -0700)
>>
>> ----------------------------------------------------------------
>> Paul E. McKenney (1):
>>        rcu: Migrate callbacks earlier in the CPU-offline timeline
>>
>>   include/linux/rcupdate.h |   1 +
>>   kernel/cpu.c             |   1 +
>>   kernel/rcu/tree.c        | 209 +++++++++++++++++++++++++----------------------
>>   3 files changed, 115 insertions(+), 96 deletions(-)
> 
> Pulled, thanks Paul!
> 
> 	Ingo
> 

Ingo,

We've been keeping an eye on this fix, so I'm trying to ascertain it's 
status.  For my information, is this expected to hit Linus' tree in 
4.13, or wait until the 4.14 merge window?

Thanks

-- 
Jeffrey Hugo
Qualcomm Datacenter Technologies as an affiliate of Qualcomm 
Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.

[toc] | [prev] | [next] | [standalone]


#1718935

FromIngo Molnar <mingo@kernel.org>
Date2017-08-24 10:00 +0200
Message-ID<uhVdv-3Ah-1@gated-at.bofh.it>
In reply to#1718504
* Jeffrey Hugo <jhugo@codeaurora.org> wrote:

> > > This commit has been exposed to 0day test robot and -next testing
> > > and is available in the git repository at:
> > > 
> > >    git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
> > > 
> > > for you to fetch changes up to a58163d8ca2c8d288ee9f95989712f98473a5ac2:
> > > 
> > >    rcu: Migrate callbacks earlier in the CPU-offline timeline (2017-07-25 13:03:43 -0700)
> > > 
> > > ----------------------------------------------------------------
> > > Paul E. McKenney (1):
> > >        rcu: Migrate callbacks earlier in the CPU-offline timeline
> > > 
> > >   include/linux/rcupdate.h |   1 +
> > >   kernel/cpu.c             |   1 +
> > >   kernel/rcu/tree.c        | 209 +++++++++++++++++++++++++----------------------
> > >   3 files changed, 115 insertions(+), 96 deletions(-)
> > 
> > Pulled, thanks Paul!
> > 
> > 	Ingo
> > 
> 
> Ingo,
> 
> We've been keeping an eye on this fix, so I'm trying to ascertain it's status.  
> For my information, is this expected to hit Linus' tree in 4.13, or wait until 
> the 4.14 merge window?

It's for v4.14, it's a pretty large (and thus risky) commit that came in 
relatively late. If it's problem-free it can be sent to -stable.

If Paul insists I can try and send it to Linus, but it's getting fairly late
in the cycle.

Thanks,

	Ingo

[toc] | [prev] | [next] | [standalone]


#1719315

From"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Date2017-08-24 16:40 +0200
Message-ID<ui1sB-7MD-1@gated-at.bofh.it>
In reply to#1718935
On Thu, Aug 24, 2017 at 09:53:20AM +0200, Ingo Molnar wrote:
> 
> * Jeffrey Hugo <jhugo@codeaurora.org> wrote:
> 
> > > > This commit has been exposed to 0day test robot and -next testing
> > > > and is available in the git repository at:
> > > > 
> > > >    git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
> > > > 
> > > > for you to fetch changes up to a58163d8ca2c8d288ee9f95989712f98473a5ac2:
> > > > 
> > > >    rcu: Migrate callbacks earlier in the CPU-offline timeline (2017-07-25 13:03:43 -0700)
> > > > 
> > > > ----------------------------------------------------------------
> > > > Paul E. McKenney (1):
> > > >        rcu: Migrate callbacks earlier in the CPU-offline timeline
> > > > 
> > > >   include/linux/rcupdate.h |   1 +
> > > >   kernel/cpu.c             |   1 +
> > > >   kernel/rcu/tree.c        | 209 +++++++++++++++++++++++++----------------------
> > > >   3 files changed, 115 insertions(+), 96 deletions(-)
> > > 
> > > Pulled, thanks Paul!
> > > 
> > > 	Ingo
> > > 
> > 
> > Ingo,
> > 
> > We've been keeping an eye on this fix, so I'm trying to ascertain it's status.  
> > For my information, is this expected to hit Linus' tree in 4.13, or wait until 
> > the 4.14 merge window?
> 
> It's for v4.14, it's a pretty large (and thus risky) commit that came in 
> relatively late. If it's problem-free it can be sent to -stable.
> 
> If Paul insists I can try and send it to Linus, but it's getting fairly late
> in the cycle.

Hugo, I was interpreting your email of August 22nd as meaning that
the problem no longer existed in mainline:

https://marc.info/?l=linux-kernel&m=150343524911301&w=2

If my interpretation is correct, then this commit can wait until the
upcoming merge window.  As Ingo says, I can send patches to -stable
(and backporting is required, BTW).

If my interpretation is incorrect, please let me know what the situation
really is.

							Thanx, Paul

[toc] | [prev] | [next] | [standalone]


#1719325

FromJeffrey Hugo <jhugo@codeaurora.org>
Date2017-08-24 16:50 +0200
Message-ID<ui1Ci-7QZ-15@gated-at.bofh.it>
In reply to#1719315
On 8/24/2017 8:35 AM, Paul E. McKenney wrote:
> On Thu, Aug 24, 2017 at 09:53:20AM +0200, Ingo Molnar wrote:
>>
>> * Jeffrey Hugo <jhugo@codeaurora.org> wrote:
>>
>>>>> This commit has been exposed to 0day test robot and -next testing
>>>>> and is available in the git repository at:
>>>>>
>>>>>     git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git for-mingo
>>>>>
>>>>> for you to fetch changes up to a58163d8ca2c8d288ee9f95989712f98473a5ac2:
>>>>>
>>>>>     rcu: Migrate callbacks earlier in the CPU-offline timeline (2017-07-25 13:03:43 -0700)
>>>>>
>>>>> ----------------------------------------------------------------
>>>>> Paul E. McKenney (1):
>>>>>         rcu: Migrate callbacks earlier in the CPU-offline timeline
>>>>>
>>>>>    include/linux/rcupdate.h |   1 +
>>>>>    kernel/cpu.c             |   1 +
>>>>>    kernel/rcu/tree.c        | 209 +++++++++++++++++++++++++----------------------
>>>>>    3 files changed, 115 insertions(+), 96 deletions(-)
>>>>
>>>> Pulled, thanks Paul!
>>>>
>>>> 	Ingo
>>>>
>>>
>>> Ingo,
>>>
>>> We've been keeping an eye on this fix, so I'm trying to ascertain it's status.
>>> For my information, is this expected to hit Linus' tree in 4.13, or wait until
>>> the 4.14 merge window?
>>
>> It's for v4.14, it's a pretty large (and thus risky) commit that came in
>> relatively late. If it's problem-free it can be sent to -stable.
>>
>> If Paul insists I can try and send it to Linus, but it's getting fairly late
>> in the cycle.
> 
> Hugo, I was interpreting your email of August 22nd as meaning that
> the problem no longer existed in mainline:
> 
> https://marc.info/?l=linux-kernel&m=150343524911301&w=2
> 
> If my interpretation is correct, then this commit can wait until the
> upcoming merge window.  As Ingo says, I can send patches to -stable
> (and backporting is required, BTW).
> 
> If my interpretation is incorrect, please let me know what the situation
> really is.
> 
> 							Thanx, Paul
> 

Sorry for any confusion.  Yes, that interpretation is correct.  I agree, 
it is late in the cycle, and I am not advocating for inclusion into 
4.13.  We have internal processes concerning upstream fixes, and it is 
helpful to identify exactly when a fix is expected to hit Linus' tree - 
none of which is particularly important or relevant to the community, 
I'm just trying to bridge a few gaps.

4.14 merge window is perfectly fine.  I will plan on that.  Thanks for 
the clarification, and a huge thanks for looking at the issue and 
solving it.

-- 
Jeffrey Hugo
Qualcomm Datacenter Technologies as an affiliate of Qualcomm 
Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web