| Message ID | 20110720203703.GA4495@huya.qualcomm.com |
|---|---|
| State | New |
| Headers | show |
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
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
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.
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
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.
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(-)