public inbox for linux-arch@vger.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH 0/4] kprobes: split blacklist into common and arch
       [not found] <1364977734-32267-1-git-send-email-oskar.andero@sonymobile.com>
@ 2013-04-04 11:32 ` Vineet Gupta
  2013-04-04 11:41   ` Oskar Andero
  0 siblings, 1 reply; 3+ messages in thread
From: Vineet Gupta @ 2013-04-04 11:32 UTC (permalink / raw)
  To: oskar.andero
  Cc: linux-kernel, masami.hiramatsu.pt, davem, anil.s.keshavamurthy,
	ananth, radovan.lekanovic, bjorn.davidsson,
	linux-arch@vger.kernel.org

Hi Oskar,

On 04/03/2013 01:58 PM, oskar.andero@sonymobile.com wrote:
> Hi,
> 
> This is a slight rework of the following patches which I posted earlier:
> [PATCH] Kprobes blacklist: Conditionally add x86-specific symbols
> [PATCH] delay blacklist symbol lookup until we actually need it
> 
> This serie includes a patch that moves the architecture dependent black points
> to the arch/ directory and the x86 patch was modified to follow that patch.
> 
> -Oskar
> 
> Björn Davidsson (1):
>   kprobes: move x86-specific blacklist symbols to arch directory
> 
> Oskar Andero (2):
>   kprobes: split blacklist into common and arch
>   kprobes: replace printk with pr_-functions
> 
> Toby Collett (1):
>   kprobes: delay blacklist symbol lookup until we actually need it
> 
>  arch/arc/kernel/kprobes.c      |   3 +
>  arch/arm/kernel/kprobes.c      |   2 +
>  arch/avr32/kernel/kprobes.c    |   3 +
>  arch/ia64/kernel/kprobes.c     |   3 +
>  arch/mips/kernel/kprobes.c     |   3 +
>  arch/mn10300/kernel/kprobes.c  |   2 +
>  arch/powerpc/kernel/kprobes.c  |   3 +
>  arch/s390/kernel/kprobes.c     |   3 +
>  arch/sh/kernel/kprobes.c       |   3 +
>  arch/sparc/kernel/kprobes.c    |   3 +
>  arch/x86/kernel/kprobes/core.c |   7 ++
>  kernel/kprobes.c               | 149 ++++++++++++++++++++++++++---------------
>  12 files changed, 130 insertions(+), 54 deletions(-)
> 

For any arch changes, however trivial they might be, you need to CC the respective
maintainers, or atleast post the patches on linux-arch so we know what's going on !


Thx,
-Vineet

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/4] kprobes: split blacklist into common and arch
  2013-04-04 11:32 ` [PATCH 0/4] kprobes: split blacklist into common and arch Vineet Gupta
@ 2013-04-04 11:41   ` Oskar Andero
  2013-04-04 11:41     ` Oskar Andero
  0 siblings, 1 reply; 3+ messages in thread
From: Oskar Andero @ 2013-04-04 11:41 UTC (permalink / raw)
  To: Vineet Gupta
  Cc: linux-kernel@vger.kernel.org, masami.hiramatsu.pt@hitachi.com,
	davem@davemloft.net, anil.s.keshavamurthy@intel.com,
	ananth@in.ibm.com, Lekanovic, Radovan, Davidsson, Björn,
	linux-arch@vger.kernel.org

On 13:32 Thu 04 Apr     , Vineet Gupta wrote:
> Hi Oskar,
> 
> On 04/03/2013 01:58 PM, oskar.andero@sonymobile.com wrote:
> > Hi,
> > 
> > This is a slight rework of the following patches which I posted earlier:
> > [PATCH] Kprobes blacklist: Conditionally add x86-specific symbols
> > [PATCH] delay blacklist symbol lookup until we actually need it
> > 
> > This serie includes a patch that moves the architecture dependent black points
> > to the arch/ directory and the x86 patch was modified to follow that patch.
> > 
> > -Oskar
> > 
> > Björn Davidsson (1):
> >   kprobes: move x86-specific blacklist symbols to arch directory
> > 
> > Oskar Andero (2):
> >   kprobes: split blacklist into common and arch
> >   kprobes: replace printk with pr_-functions
> > 
> > Toby Collett (1):
> >   kprobes: delay blacklist symbol lookup until we actually need it
> > 
> >  arch/arc/kernel/kprobes.c      |   3 +
> >  arch/arm/kernel/kprobes.c      |   2 +
> >  arch/avr32/kernel/kprobes.c    |   3 +
> >  arch/ia64/kernel/kprobes.c     |   3 +
> >  arch/mips/kernel/kprobes.c     |   3 +
> >  arch/mn10300/kernel/kprobes.c  |   2 +
> >  arch/powerpc/kernel/kprobes.c  |   3 +
> >  arch/s390/kernel/kprobes.c     |   3 +
> >  arch/sh/kernel/kprobes.c       |   3 +
> >  arch/sparc/kernel/kprobes.c    |   3 +
> >  arch/x86/kernel/kprobes/core.c |   7 ++
> >  kernel/kprobes.c               | 149 ++++++++++++++++++++++++++---------------
> >  12 files changed, 130 insertions(+), 54 deletions(-)
> > 
> 
> For any arch changes, however trivial they might be, you need to CC the respective
> maintainers, or atleast post the patches on linux-arch so we know what's going on !

Thanks Vineet! I will make sure to add linux-arch on CC for version 2.

-Oskar

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/4] kprobes: split blacklist into common and arch
  2013-04-04 11:41   ` Oskar Andero
@ 2013-04-04 11:41     ` Oskar Andero
  0 siblings, 0 replies; 3+ messages in thread
From: Oskar Andero @ 2013-04-04 11:41 UTC (permalink / raw)
  To: Vineet Gupta
  Cc: linux-kernel@vger.kernel.org, masami.hiramatsu.pt@hitachi.com,
	davem@davemloft.net, anil.s.keshavamurthy@intel.com,
	ananth@in.ibm.com, Lekanovic, Radovan, Davidsson, Björn,
	linux-arch@vger.kernel.org

On 13:32 Thu 04 Apr     , Vineet Gupta wrote:
> Hi Oskar,
> 
> On 04/03/2013 01:58 PM, oskar.andero@sonymobile.com wrote:
> > Hi,
> > 
> > This is a slight rework of the following patches which I posted earlier:
> > [PATCH] Kprobes blacklist: Conditionally add x86-specific symbols
> > [PATCH] delay blacklist symbol lookup until we actually need it
> > 
> > This serie includes a patch that moves the architecture dependent black points
> > to the arch/ directory and the x86 patch was modified to follow that patch.
> > 
> > -Oskar
> > 
> > Björn Davidsson (1):
> >   kprobes: move x86-specific blacklist symbols to arch directory
> > 
> > Oskar Andero (2):
> >   kprobes: split blacklist into common and arch
> >   kprobes: replace printk with pr_-functions
> > 
> > Toby Collett (1):
> >   kprobes: delay blacklist symbol lookup until we actually need it
> > 
> >  arch/arc/kernel/kprobes.c      |   3 +
> >  arch/arm/kernel/kprobes.c      |   2 +
> >  arch/avr32/kernel/kprobes.c    |   3 +
> >  arch/ia64/kernel/kprobes.c     |   3 +
> >  arch/mips/kernel/kprobes.c     |   3 +
> >  arch/mn10300/kernel/kprobes.c  |   2 +
> >  arch/powerpc/kernel/kprobes.c  |   3 +
> >  arch/s390/kernel/kprobes.c     |   3 +
> >  arch/sh/kernel/kprobes.c       |   3 +
> >  arch/sparc/kernel/kprobes.c    |   3 +
> >  arch/x86/kernel/kprobes/core.c |   7 ++
> >  kernel/kprobes.c               | 149 ++++++++++++++++++++++++++---------------
> >  12 files changed, 130 insertions(+), 54 deletions(-)
> > 
> 
> For any arch changes, however trivial they might be, you need to CC the respective
> maintainers, or atleast post the patches on linux-arch so we know what's going on !

Thanks Vineet! I will make sure to add linux-arch on CC for version 2.

-Oskar

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-04-04 11:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1364977734-32267-1-git-send-email-oskar.andero@sonymobile.com>
2013-04-04 11:32 ` [PATCH 0/4] kprobes: split blacklist into common and arch Vineet Gupta
2013-04-04 11:41   ` Oskar Andero
2013-04-04 11:41     ` Oskar Andero

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox