* [GIT PULL] ARM: kprobes: big-endian support
@ 2014-01-13 12:09 Taras Kondratiuk
2014-02-19 22:59 ` Taras Kondratiuk
0 siblings, 1 reply; 7+ messages in thread
From: Taras Kondratiuk @ 2014-01-13 12:09 UTC (permalink / raw)
To: linux-arm-kernel
Hi Russell,
Please pull fixes for ARM Kprobes big-endian support.
It is reworked initial Ben's series for big endian support [1].
Dropped patches that are not directly related to kprobes.
Current set of patches is enough to have functional BE kprobes.
One ARM kprobe test fails on Cortex-A15 boards (TC2 and Keystone2 EVM),
while it passes on Pandaboard. The issue is not related to this series
and already present in v3.13-rc7.
[1] http://www.spinics.net/lists/arm-kernel/msg285210.html
--
Taras Kondratiuk
--->8
The following changes since commit 7e22e91102c6b9df7c4ae2168910e19d2bb14cd6:
Linux 3.13-rc8 (2014-01-12 17:04:18 +0700)
are available in the git repository at:
git://git.linaro.org/people/taras.kondratiuk/linux.git tags/for_russell/arm-be-kprobes
for you to fetch changes up to 1c2d814c019b63bf0778fc2d2fd54c6e44f1ab5d:
ARM: kprobes-test: Workaround GAS .align bug (2014-01-13 13:58:52 +0200)
----------------------------------------------------------------
Ben Dooks (4):
ARM: kprobes: fix instruction fetch order with <asm/opcodes.h>
ARM: kprobes-test: use <asm/opcodes.h> for instruction accesses
ARM: kprobes-test: use <asm/opcodes.h> for ARM instruction building
ARM: kprobes-test: use <asm/opcodes.h> for Thumb instruction building
Taras Kondratiuk (1):
ARM: kprobes-test: Workaround GAS .align bug
arch/arm/kernel/kprobes-common.c | 19 +-
arch/arm/kernel/kprobes-test-arm.c | 603 +++++++++++++++++-----------------
arch/arm/kernel/kprobes-test-thumb.c | 447 ++++++++++++-------------
arch/arm/kernel/kprobes-test.c | 13 +-
arch/arm/kernel/kprobes-test.h | 2 +-
arch/arm/kernel/kprobes-thumb.c | 20 +-
arch/arm/kernel/kprobes.c | 9 +-
7 files changed, 562 insertions(+), 551 deletions(-)
^ permalink raw reply [flat|nested] 7+ messages in thread* [GIT PULL] ARM: kprobes: big-endian support 2014-01-13 12:09 [GIT PULL] ARM: kprobes: big-endian support Taras Kondratiuk @ 2014-02-19 22:59 ` Taras Kondratiuk 2014-03-26 10:26 ` Taras Kondratiuk 0 siblings, 1 reply; 7+ messages in thread From: Taras Kondratiuk @ 2014-02-19 22:59 UTC (permalink / raw) To: linux-arm-kernel On 01/13/2014 02:09 PM, Taras Kondratiuk wrote: > Hi Russell, > > Please pull fixes for ARM Kprobes big-endian support. > > It is reworked initial Ben's series for big endian support [1]. > Dropped patches that are not directly related to kprobes. > Current set of patches is enough to have functional BE kprobes. > > One ARM kprobe test fails on Cortex-A15 boards (TC2 and Keystone2 EVM), > while it passes on Pandaboard. The issue is not related to this series > and already present in v3.13-rc7. > > [1] http://www.spinics.net/lists/arm-kernel/msg285210.html > Hi Russell, This pull request is based on 3.13-rc8, but it applies cleanly to 3.14-rc3, because kprobes are not touched since then. Do I need to resent a new pull request based on 3.14-rc3 anyway? -- Taras Kondratiuk ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM: kprobes: big-endian support 2014-02-19 22:59 ` Taras Kondratiuk @ 2014-03-26 10:26 ` Taras Kondratiuk 2014-03-28 11:28 ` Russell King - ARM Linux 0 siblings, 1 reply; 7+ messages in thread From: Taras Kondratiuk @ 2014-03-26 10:26 UTC (permalink / raw) To: linux-arm-kernel On 02/20/2014 12:59 AM, Taras Kondratiuk wrote: > On 01/13/2014 02:09 PM, Taras Kondratiuk wrote: >> Hi Russell, >> >> Please pull fixes for ARM Kprobes big-endian support. >> >> It is reworked initial Ben's series for big endian support [1]. >> Dropped patches that are not directly related to kprobes. >> Current set of patches is enough to have functional BE kprobes. >> >> One ARM kprobe test fails on Cortex-A15 boards (TC2 and Keystone2 EVM), >> while it passes on Pandaboard. The issue is not related to this series >> and already present in v3.13-rc7. >> >> [1] http://www.spinics.net/lists/arm-kernel/msg285210.html >> > > Hi Russell, > > This pull request is based on 3.13-rc8, but it applies cleanly to > 3.14-rc3, because kprobes are not touched since then. > Do I need to resent a new pull request based on 3.14-rc3 anyway? > Hi Russell, Is there any issue with this pull request? Should I do something to get it pulled? -- Taras Kondratiuk ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM: kprobes: big-endian support 2014-03-26 10:26 ` Taras Kondratiuk @ 2014-03-28 11:28 ` Russell King - ARM Linux 2014-03-28 18:13 ` Taras Kondratiuk 0 siblings, 1 reply; 7+ messages in thread From: Russell King - ARM Linux @ 2014-03-28 11:28 UTC (permalink / raw) To: linux-arm-kernel On Wed, Mar 26, 2014 at 12:26:34PM +0200, Taras Kondratiuk wrote: > On 02/20/2014 12:59 AM, Taras Kondratiuk wrote: > > On 01/13/2014 02:09 PM, Taras Kondratiuk wrote: > >> Hi Russell, > >> > >> Please pull fixes for ARM Kprobes big-endian support. > >> > >> It is reworked initial Ben's series for big endian support [1]. > >> Dropped patches that are not directly related to kprobes. > >> Current set of patches is enough to have functional BE kprobes. > >> > >> One ARM kprobe test fails on Cortex-A15 boards (TC2 and Keystone2 EVM), > >> while it passes on Pandaboard. The issue is not related to this series > >> and already present in v3.13-rc7. > >> > >> [1] http://www.spinics.net/lists/arm-kernel/msg285210.html > >> > > > > Hi Russell, > > > > This pull request is based on 3.13-rc8, but it applies cleanly to > > 3.14-rc3, because kprobes are not touched since then. > > Do I need to resent a new pull request based on 3.14-rc3 anyway? > > > > Hi Russell, > > Is there any issue with this pull request? > Should I do something to get it pulled? There's nothing wrong, apart from me being far too busy hacking on really crap code to care about reading much email - which means that lots of email simply just gets buried and lost. There is now a problem with this pull - it conflicts very badly with Dave Long's uprobes code, which I've already pulled, so much so that I'm not happy to do the conflict resolution since I know nothing about this code, and it's a feature I don't make any use of. I notice that Dave Long and yourself are both under the Linaro umbrella, but there seems to be no coordination between yourselves, despite working on the same code... -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM: kprobes: big-endian support 2014-03-28 11:28 ` Russell King - ARM Linux @ 2014-03-28 18:13 ` Taras Kondratiuk 2014-03-28 18:22 ` Russell King - ARM Linux 0 siblings, 1 reply; 7+ messages in thread From: Taras Kondratiuk @ 2014-03-28 18:13 UTC (permalink / raw) To: linux-arm-kernel On 03/28/2014 01:28 PM, Russell King - ARM Linux wrote: > On Wed, Mar 26, 2014 at 12:26:34PM +0200, Taras Kondratiuk wrote: >> On 02/20/2014 12:59 AM, Taras Kondratiuk wrote: >> >> Hi Russell, >> >> Is there any issue with this pull request? >> Should I do something to get it pulled? > > There's nothing wrong, apart from me being far too busy hacking on really > crap code to care about reading much email - which means that lots of > email simply just gets buried and lost. > > There is now a problem with this pull - it conflicts very badly with Dave > Long's uprobes code, which I've already pulled, so much so that I'm not > happy to do the conflict resolution since I know nothing about this code, > and it's a feature I don't make any use of. I've rebased it onto your for-next branch. Kprobes are functional in LE and BE. I'll send an updated pull request after verifying uprobes. > I notice that Dave Long and yourself are both under the Linaro umbrella, > but there seems to be no coordination between yourselves, despite working > on the same code... Yeah, we are working in different teams and unfortunately haven't coordinated an upstreaming sequence. -- Taras Kondratiuk ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM: kprobes: big-endian support 2014-03-28 18:13 ` Taras Kondratiuk @ 2014-03-28 18:22 ` Russell King - ARM Linux 2014-03-28 21:43 ` Taras Kondratiuk 0 siblings, 1 reply; 7+ messages in thread From: Russell King - ARM Linux @ 2014-03-28 18:22 UTC (permalink / raw) To: linux-arm-kernel On Fri, Mar 28, 2014 at 08:13:05PM +0200, Taras Kondratiuk wrote: > On 03/28/2014 01:28 PM, Russell King - ARM Linux wrote: > > On Wed, Mar 26, 2014 at 12:26:34PM +0200, Taras Kondratiuk wrote: > >> On 02/20/2014 12:59 AM, Taras Kondratiuk wrote: > >> > >> Hi Russell, > >> > >> Is there any issue with this pull request? > >> Should I do something to get it pulled? > > > > There's nothing wrong, apart from me being far too busy hacking on really > > crap code to care about reading much email - which means that lots of > > email simply just gets buried and lost. > > > > There is now a problem with this pull - it conflicts very badly with Dave > > Long's uprobes code, which I've already pulled, so much so that I'm not > > happy to do the conflict resolution since I know nothing about this code, > > and it's a feature I don't make any use of. > > I've rebased it onto your for-next branch. Kprobes are functional > in LE and BE. I'll send an updated pull request after verifying uprobes. That's not going to work. for-next is an unstable branch - all the merge commits there get regenerated on a fairly regular basis. A better solution would be to base your patches on top of Dave Long's changes, which can be found at git://git.linaro.org/people/dave.long/linux uprobes-v7 and should have an ID of c7edc9e326d5. -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [GIT PULL] ARM: kprobes: big-endian support 2014-03-28 18:22 ` Russell King - ARM Linux @ 2014-03-28 21:43 ` Taras Kondratiuk 0 siblings, 0 replies; 7+ messages in thread From: Taras Kondratiuk @ 2014-03-28 21:43 UTC (permalink / raw) To: linux-arm-kernel On 03/28/2014 08:22 PM, Russell King - ARM Linux wrote: > On Fri, Mar 28, 2014 at 08:13:05PM +0200, Taras Kondratiuk wrote: >> On 03/28/2014 01:28 PM, Russell King - ARM Linux wrote: >>> On Wed, Mar 26, 2014 at 12:26:34PM +0200, Taras Kondratiuk wrote: >>>> On 02/20/2014 12:59 AM, Taras Kondratiuk wrote: >>>> >>>> Hi Russell, >>>> >>>> Is there any issue with this pull request? >>>> Should I do something to get it pulled? >>> >>> There's nothing wrong, apart from me being far too busy hacking on really >>> crap code to care about reading much email - which means that lots of >>> email simply just gets buried and lost. >>> >>> There is now a problem with this pull - it conflicts very badly with Dave >>> Long's uprobes code, which I've already pulled, so much so that I'm not >>> happy to do the conflict resolution since I know nothing about this code, >>> and it's a feature I don't make any use of. >> >> I've rebased it onto your for-next branch. Kprobes are functional >> in LE and BE. I'll send an updated pull request after verifying uprobes. > > That's not going to work. for-next is an unstable branch - all the > merge commits there get regenerated on a fairly regular basis. > > A better solution would be to base your patches on top of Dave Long's > changes, which can be found at > > git://git.linaro.org/people/dave.long/linux uprobes-v7 > > and should have an ID of c7edc9e326d5. Ok I'll base on Dave's branch. -- Taras Kondratiuk ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-03-28 21:43 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-01-13 12:09 [GIT PULL] ARM: kprobes: big-endian support Taras Kondratiuk 2014-02-19 22:59 ` Taras Kondratiuk 2014-03-26 10:26 ` Taras Kondratiuk 2014-03-28 11:28 ` Russell King - ARM Linux 2014-03-28 18:13 ` Taras Kondratiuk 2014-03-28 18:22 ` Russell King - ARM Linux 2014-03-28 21:43 ` Taras Kondratiuk
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox