* [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;
as well as URLs for NNTP newsgroup(s).