Patchwork [GIT,PULL] MSM fix

login
register
mail settings
Submitter David Brown
Date July 20, 2011, 8:37 p.m.
Message ID <20110720203703.GA4495@huya.qualcomm.com>
Download mbox
Permalink /patch/105824/
State New
Headers show

Pull-request

git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix

Comments

David Brown - July 20, 2011, 8:37 p.m.
I seem to have missed this fix when it was sent out in April.

The following changes since commit 620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc:

  Linux 3.0-rc7 (2011-07-11 16:51:52 -0700)

are available in the git repository at:
  git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix

Jeff Ohlstein (1):
      msm: platsmp: actually read core count in get_core_count

 arch/arm/mach-msm/platsmp.c |   11 +++++++++--
 1 files changed, 9 insertions(+), 2 deletions(-)
Arnd Bergmann - July 20, 2011, 9:06 p.m.
On Wednesday 20 July 2011 13:37:03 David Brown wrote:
> I seem to have missed this fix when it was sent out in April.
> 
> The following changes since commit 620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc:
> 
>   Linux 3.0-rc7 (2011-07-11 16:51:52 -0700)
> 
> are available in the git repository at:
>   git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix
> 
> Jeff Ohlstein (1):
>       msm: platsmp: actually read core count in get_core_count
> 

Hmm, this one seems like it really applies to older kernels as well, so
we can just as well add a cc:stable@kernel.org tag and put it into the
arm-soc/next/fixes branch, right?

Linus has just pulled the 'last arm-soc fixes for 3.0' branch, and I don't
really want to follow up with 'last fix, this time for real' pull request
unless it's for a serious regression.

On a related topic, what's the deal with the msm-core and msm-move-gpio
branches in the same tree? I have the impression that they should be
in arm-soc.git, but I never received a pull request.

The msm-move-gpio branch looks like it wasn't merged into the gpio
tree either, which would be the other logical place, and the msm-core
branch is currently in linux-next but hasn't been updated since April.

	Arnd
David Brown - July 20, 2011, 10:50 p.m.
On Wed, Jul 20, 2011 at 11:06:15PM +0200, Arnd Bergmann wrote:
> On Wednesday 20 July 2011 13:37:03 David Brown wrote:
> > I seem to have missed this fix when it was sent out in April.
> > 
> > The following changes since commit 620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc:
> > 
> >   Linux 3.0-rc7 (2011-07-11 16:51:52 -0700)
> > 
> > are available in the git repository at:
> >   git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix
> > 
> > Jeff Ohlstein (1):
> >       msm: platsmp: actually read core count in get_core_count
> > 
> 
> Hmm, this one seems like it really applies to older kernels as well, so
> we can just as well add a cc:stable@kernel.org tag and put it into the
> arm-soc/next/fixes branch, right?

It looks like it would apply to 2.6.38 and 39 as well.  I'm not sure
it qualifies under the "It must fix a real bug that bothers people".
It would only apply if someone had set NR_CPUS to more than 2.

> Linus has just pulled the 'last arm-soc fixes for 3.0' branch, and I don't
> really want to follow up with 'last fix, this time for real' pull request
> unless it's for a serious regression.

It's not serious, so 3.1 is fine.

> On a related topic, what's the deal with the msm-core and msm-move-gpio
> branches in the same tree? I have the impression that they should be
> in arm-soc.git, but I never received a pull request.

I had sent a pull request for msm-move-gpio to Grant, but apparently
that didn't happen.  If more appropriate, I can send one to you for
it, or resend to Grant.

The msm-core stuff happened before the snafu about ARM, and I was
holding off on these things.  The authors of these patches say they
are working on cleaned up versions.

I'm not sure we want the dummy clocks code in the kernel.  It's
intended as a placeholder.  Stephen Boyd has a 26-series patch that
adds better clock support.  Given the size (13047 insertions), this
may need some cleanup or even device-tree work to include.

Unfortunately, a lot of the Qualcomm development seems to have moved
back to private trees.

BTW, I am planning on being at the Linaro Connect in August.

David
Russell King - ARM Linux - July 20, 2011, 10:54 p.m.
On Wed, Jul 20, 2011 at 03:50:10PM -0700, David Brown wrote:
> On Wed, Jul 20, 2011 at 11:06:15PM +0200, Arnd Bergmann wrote:
> > On Wednesday 20 July 2011 13:37:03 David Brown wrote:
> > > I seem to have missed this fix when it was sent out in April.
> > > 
> > > The following changes since commit 620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc:
> > > 
> > >   Linux 3.0-rc7 (2011-07-11 16:51:52 -0700)
> > > 
> > > are available in the git repository at:
> > >   git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix
> > > 
> > > Jeff Ohlstein (1):
> > >       msm: platsmp: actually read core count in get_core_count
> > > 
> > 
> > Hmm, this one seems like it really applies to older kernels as well, so
> > we can just as well add a cc:stable@kernel.org tag and put it into the
> > arm-soc/next/fixes branch, right?
> 
> It looks like it would apply to 2.6.38 and 39 as well.  I'm not sure
> it qualifies under the "It must fix a real bug that bothers people".
> It would only apply if someone had set NR_CPUS to more than 2.

Err, this makes no sense.  MSM doesn't have a get_core_count function
and as far as I can see never has done in mainline.

And where is the patch?

We shouldn't be pulling git requests into arm-soc without having seen
the patches on LAKML.
David Brown - July 20, 2011, 11:44 p.m.
On Wed, Jul 20, 2011 at 11:54:06PM +0100, Russell King - ARM Linux wrote:
> On Wed, Jul 20, 2011 at 03:50:10PM -0700, David Brown wrote:
> > On Wed, Jul 20, 2011 at 11:06:15PM +0200, Arnd Bergmann wrote:
> > > On Wednesday 20 July 2011 13:37:03 David Brown wrote:
> > > > I seem to have missed this fix when it was sent out in April.
> > > > 
> > > > The following changes since commit 620917de59eeb934b9f8cf35cc2d95c1ac8ed0fc:
> > > > 
> > > >   Linux 3.0-rc7 (2011-07-11 16:51:52 -0700)
> > > > 
> > > > are available in the git repository at:
> > > >   git://codeaurora.org/quic/kernel/davidb/linux-msm.git msm-fix
> > > > 
> > > > Jeff Ohlstein (1):
> > > >       msm: platsmp: actually read core count in get_core_count
> > > > 
> > > 
> > > Hmm, this one seems like it really applies to older kernels as well, so
> > > we can just as well add a cc:stable@kernel.org tag and put it into the
> > > arm-soc/next/fixes branch, right?
> > 
> > It looks like it would apply to 2.6.38 and 39 as well.  I'm not sure
> > it qualifies under the "It must fix a real bug that bothers people".
> > It would only apply if someone had set NR_CPUS to more than 2.
> 
> Err, this makes no sense.  MSM doesn't have a get_core_count function
> and as far as I can see never has done in mainline.
> 
> And where is the patch?
> 
> We shouldn't be pulling git requests into arm-soc without having seen
> the patches on LAKML.

It was sent back in April.

https://patchwork.kernel.org/patch/693881/

David
Russell King - ARM Linux - July 21, 2011, 3:41 p.m.
On Thu, Jul 21, 2011 at 04:49:39PM +0200, Arnd Bergmann wrote:
> On Thursday 21 July 2011, Russell King - ARM Linux wrote:
> > > It looks like it would apply to 2.6.38 and 39 as well.  I'm not sure
> > > it qualifies under the "It must fix a real bug that bothers people".
> > > It would only apply if someone had set NR_CPUS to more than 2.
> > 
> > Err, this makes no sense.  MSM doesn't have a get_core_count function
> > and as far as I can see never has done in mainline.
> 
> I've committed the version below now, with a clarified title.
> The patch looks reasonable to me, but there may be some issue
> I'm missing of course. The other platforms that used to have
> a get_core_count function no longer have that because you changed
> them to use scu_get_core_count, but that doesn't seem to apply
> to mach-msm.

That patch description helps a lot to clarify what's going on.