On Monday 25 February 2008 11:50:24 am Dave Hansen wrote:
> On Mon, 2008-02-25 at 11:27 +0530, srinivasa wrote:
> > This patch prohibits user from probing preempt_schedule(). One way of
> > prohibiting the user from probing functions is by marking such
> > functions with __kprobes. But this method
Kprobes makes use of preempt_disable(),preempt_enable_noresched() and these
functions inturn call add/sub_preempt_count(). So we need to refuse user from
inserting probe in to these functions.
This patch disallows user from probing add/sub_preempt_count().
Signed-off-by: Srinivasa DS <[EM
arded)
init functions harder to read on some archs, but we already have that
issue for modules and noone has complained.
Thanks Rusty, This patch fixes my problem.
Tested-by: Srinivasa DS <[EMAIL PROTECTED]>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel&q
er to read on some archs.
I think it is better to make use of new function than sacrificing
__init function symbol information in backtrace.
Thanks
Srinivasa DS
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL
This patch makes use of persistent_kernel_text_address() to avoid
probing __init functions.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
kernel/kprobes.c |4 ++--
1 file changed, 2 insertions(+), 2 deletio
persistent_kernel_text_address() to identify
"non_init" text area.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
include/linux/kernel.h |2 ++
kernel/extable.c | 16 +++-
2 files
that.
This patch creates non_init_kernel_text_address() to identify
non_init text area.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
include/linux/kernel.h |2 ++
kernel/extable.c | 16 +
This patch makes use of regular_kernel_text_address() to avoid
probing __init functions.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
kernel/kprobes.c |4 ++--
1 file changed, 2 insertions(+), 2 deletio
non_init_kernel_text_address() to identify
non_init text area.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
include/linux/kernel.h |2 ++
kernel/extable.c | 16 +++-
2 files changed, 17 inse
This patch makes use of non_init_kernel_text_address() to avoid
probing __init functions.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
kernel/kprobes.c |4 ++--
1 file changed, 2 insertions(+), 2 deletio
suggestions for a better functionname.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Ananth N Mavinakayanahalli <[EMAIL PROTECTED]>
---
include/linux/kernel.h |2 ++
kernel/extable.c | 16 +++-
2 files changed, 17 insertions(+), 1 deleti
@ long arch_ptrace(struct task_struct *child, long request,
long addr, long data)
break;
}
Thanks
Srinivasa DS
Linux Technology Centre
IBM.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED
ted first(because it took a lot time
to return).
So only thing to match the entry_handler() with its return_handler() is
return probe instance(ri)'s address, which user has to take care explicitly
(Hence I feel sol a) would be nice).
Thanks
Srinivasa DS
-
To unsubscribe from this list: s
Abhishek Sagar wrote:
On Nov 13, 2007 12:09 AM, Abhishek Sagar <[EMAIL PROTECTED]> wrote:
Whoops...sry for the repeated email..emailer trouble.
Expecting this one to makes it to the list. Summary again:
This patch introduces a provision to specify a user-defined callback
to run at function en
call_entry+0x141/0x174
[] tracesys+0xdc/0xe1
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Suzuki K P <[EMAIL PROTECTED]>
Signed-off-by: Prasanna S Panchamukhi <[EMAIL PROTECTED]>
Index: linux-2.6.23-rc1/ar
indicates that vunmap() is not receving the page-aligned address in
text_poke(). So the below attached patch will address this issue. Please let
me know your comments.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Suzuki K P <[EMAIL PROTECTED]>
Index: linux-2.6
On Friday 01 June 2007 12:57:53 Srinivasa Ds wrote:
I have modified the patch little bit, Thanks Vatsa for input and hence
resending it. Please let me know your comments on this.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
---
kernel/cpuset.c | 10 ++
1 file chang
=
This is because "npids"(represnets number of pids in that cpuset) in
"cpu_task_open" is zero and it tries allocate 0 bytes through kmalloc.
Below patch fixes this problem. Please let me know your comments on this.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]&
Christoph Lameter wrote:
> On Tue, 8 May 2007, Srinivasa Ds wrote:
>
>> Looks like there is a bug in SLUB implementaion for ppc64 in 2.6.21-mm1.
>> I unmarked CONFIG_SLUB and build the kernel, its booting cleary now.
>>
>> Thanks
>> Srinivasa DS
>
> Th
> msr: 80009032
>
Looks like there is a bug in SLUB implementaion for ppc64 in 2.6.21-mm1.
I unmarked CONFIG_SLUB and build the kernel, its booting cleary now.
Thanks
Srinivasa DS
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body o
=
Since code is not compatible with struct sk_buff change, we have this error.
Below patch should fix this problem. Please let me know your comments on
this.
Signed-off-by: Srinivasa Ds <[EMAIL PROTECTED]>
---
drivers/net/ehea/ehea_main.c |6 +++---
1 file changed, 3 insertions(+), 3 del
Paul Mackerras wrote:
> Srinivasa Ds writes:
>
>> +} else {\
>> +char dot_name[KSYM_NAME_LEN+1]; \
>> +dot_name[0] = '.';
ep do_fork
c006283c T .do_fork
==
User needs add "." to "do_fork"
kp.symbol_name = ".do_fork";
This makes kprobe modules unportable. Below patch fixes the problem.
Signed-off-by: Srinivasa Ds <
declaration of
function ‘sysdev_register’
make[1]: *** [arch/powerpc/sysdev/timer.o] Error 1
make: *** [arch/powerpc/sysdev] Error 2
===
Below patch fixes this problem. please let me know comments on this.
Signed-off-by: Srinivasa Ds <[EMAIL PROTECTED]>
---
Andrew Morton wrote:
> On Wed, 07 Feb 2007 10:55:23 +0530
> Srinivasa Ds <[EMAIL PROTECTED]> wrote:
>
>> --- linux-2.6.20.orig/fs/debugfs/inode.c
>> +++ linux-2.6.20/fs/debugfs/inode.c
>> @@ -25,6 +25,7 @@
>>
>> if (retval)
>&
Frederik Deweerdt wrote:
> Hi,
>
> Comments below,
>
> On Tue, Feb 06, 2007 at 08:17:06PM +0530, Srinivasa Ds wrote:
>> Srinivasa Ds wrote:
>>> Christoph Hellwig wrote:
>>>> On Tue, Feb 06, 2007 at 03:25:35PM +0530, Srinivasa Ds wrote:
>>>>&
Srinivasa Ds wrote:
> Christoph Hellwig wrote:
>> On Tue, Feb 06, 2007 at 03:25:35PM +0530, Srinivasa Ds wrote:
>>> Hi folks
>>>
>>> I have developed a patch, that lists all active probes in the
>>> system. I have done this through /proc int
Christoph Hellwig wrote:
> On Tue, Feb 06, 2007 at 03:25:35PM +0530, Srinivasa Ds wrote:
>> Hi folks
>>
>> I have developed a patch, that lists all active probes in the
>> system. I have done this through /proc interface. Currently list is
>> availa
Andrew Morton wrote:
> On Tue, 06 Feb 2007 15:25:35 +0530 Srinivasa Ds <[EMAIL PROTECTED]> wrote:
>
>> I have developed a patch, that lists all active probes in the
>> system.
>
> userspace added the probes, so userspace should know where they all are?
Irres
s4_dentry_release+0x0 autofs4
c00275d0 k kretprobe_trampoline+0x0
c00bfd18 k .vfs_write+0x0
Please let me know your comments on this.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
fs/proc/root.c |4 ++
include/linux/kp
envioronment.
It happens in 32-bit compatibility mode.
Problem is in calculation of apicid's and delivery of IPI's.
I saw a oops,when I do cpuhotplug operations on it.
If you want any further information,please free to ask.
Thanks
Srinivasa Ds
thanks,
suresh
On Mon, Jan 22, 2007 at 01:42:
think there is no concept of apic clusters
if there are only 2 cpus). Hence it makes me to conclude that problem is
in delivery of IPI's.
So Iam completely stuck here. Iam not able to move forward in debugging.
So could someone(may be intel folks) please throw some light on this.
Thanks
ode... ".
So Iam reproposing my patch(taken against latest kernel) here. Please
let me know your comments on this.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
Signed-off-by: Alasdair G Kergon <[EMAIL PROTECTED]>
Cc: Ingo Molnar <[EMAIL PROTECTED]>
Cc: Eric Sandeen <[EMAIL P
nd it needs to cover the wider
semantics correctly.. and soon you have a semaphore again.
Let true semaphores be semaphores, and make all real mutexes mutexes.
But lets not make actual semaphores use mutex code... ".
So Iam reproposing my patch(taken against latest kernel) here. Please
let m
acl(tid, ip, dip);
if (rc)
goto out3;
===
So below patch should fix this problem,please test this. Let me know
your comments on this.
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
namei.c |4 ++--
1 file changed, 2 inser
ch solves this problem. Please let me
know your comments.
====
Signed-off-by: Srinivasa DS <[EMAIL PROTECTED]>
super.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.19-rc6/fs/gfs2/super.c
===
--- lin
rce (the TSC is probably safe, but
not necessarily).
thanks
-john
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.o
37 matches
Mail list logo