mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Chuck Ebbert <76306.1226@compuserve.com>
To: Stephane Eranian <eranian@frankl.hpl.hp.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	Stephane Eranian <eranian@hpl.hp.com>
Subject: Re: [PATCH 12/16] 2.6.17-rc6 perfmon2 patch for review: modified i386 files
Date: Thu, 15 Jun 2006 23:00:01 -0400	[thread overview]
Message-ID: <200606152301_MC3-1-C28E-ADE0@compuserve.com> (raw)

In-Reply-To: <200606150907.k5F97g7B008214@frankl.hpl.hp.com>

On Thu, 15 Jun 2006 02:07:42 -0700, Stephane Eranian wrote:

> This patch contains the modified i386 files
>
> <...>
>
> diff -ur linux-2.6.17-rc6.orig/arch/i386/kernel/apic.c linux-2.6.17-rc6/arch/i386/kernel/apic.c
> --- linux-2.6.17-rc6.orig/arch/i386/kernel/apic.c     2006-06-08 01:42:30.000000000 -0700
> +++ linux-2.6.17-rc6/arch/i386/kernel/apic.c  2006-06-08 01:49:22.000000000 -0700
> @@ -27,6 +27,7 @@
>  #include <linux/sysdev.h>
>  #include <linux/cpu.h>
>  #include <linux/module.h>
> +#include <linux/perfmon.h>
>  
>  #include <asm/atomic.h>
>  #include <asm/smp.h>
> @@ -1179,6 +1180,8 @@
>       update_process_times(user_mode_vm(regs));
>  #endif
>  
> +     pfm_handle_switch_timeout();
> +
>       /*
>        * We take the 'long' return path, and there every subsystem
>        * grabs the apropriate locks (kernel lock/ irq lock).

Please add '-p' to your diff options.  It makes it easier to see what is
happening.

>
> <...>
>
> diff -ur linux-2.6.17-rc6.orig/arch/i386/kernel/syscall_table.S linux-2.6.17-rc6/arch/i386/kernel/syscall_table.S
> --- linux-2.6.17-rc6.orig/arch/i386/kernel/syscall_table.S    2006-06-08 01:42:30.000000000 -0700
> +++ linux-2.6.17-rc6/arch/i386/kernel/syscall_table.S 2006-06-08 01:50:27.000000000 -0700
> @@ -316,3 +316,15 @@
>       .long sys_sync_file_range
>       .long sys_tee                   /* 315 */
>       .long sys_vmsplice
> +             .long sys_pfm_create_context
> +             .long sys_pfm_write_pmcs
> +             .long sys_pfm_write_pmds
> +             .long sys_pfm_read_pmds         /* 320 */
> +             .long sys_pfm_load_context
> +             .long sys_pfm_start
> +             .long sys_pfm_stop
> +             .long sys_pfm_restart
> +             .long sys_pfm_create_evtsets    /* 325 */
> +             .long sys_pfm_getinfo_evtsets
> +             .long sys_pfm_delete_evtsets
> +     .long sys_pfm_unload_context

I think there are seven spaces plus a tab here for the first 11 new
syscalls? (You won't be able to tell from my quote because my mail
program mangles quoted text.)

>
> <...>
>
> --- linux-2.6.17-rc6.orig/include/asm-i386/unistd.h   2006-06-08 01:42:35.000000000 -0700
> +++ linux-2.6.17-rc6/include/asm-i386/unistd.h        2006-06-08 01:49:22.000000000 -0700
> @@ -322,8 +322,19 @@
>  #define __NR_sync_file_range 314
>  #define __NR_tee             315
>  #define __NR_vmsplice                316
> +#define __NR_pfm_create_context      317
> +#define __NR_pfm_write_pmcs  (__NR_pfm_create_context+1)
> +#define __NR_pfm_write_pmds  (__NR_pfm_create_context+2)
> +#define __NR_pfm_read_pmds   (__NR_pfm_create_context+3)
> +#define __NR_pfm_load_context        (__NR_pfm_create_context+4)
> +#define __NR_pfm_start               (__NR_pfm_create_context+5)
> +#define __NR_pfm_stop                (__NR_pfm_create_context+6)
> +#define __NR_pfm_restart     (__NR_pfm_create_context+7)
> +#define __NR_pfm_create_evtsets      (__NR_pfm_create_context+8)
> +#define __NR_pfm_getinfo_evtsets (__NR_pfm_create_context+9)
> +#define __NR_pfm_delete_evtsets (__NR_pfm_create_context+10)
>  
> -#define NR_syscalls 317
> +#define NR_syscalls 329
>  
>  /*
>   * user-visible error numbers are in the range -1 - -128: see

You missed __NR_pfm_unload_context.


-- 
Chuck


             reply	other threads:[~2006-06-16  3:04 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-16  3:00 Chuck Ebbert [this message]
2006-06-16  7:47 ` Stephane Eranian
  -- strict thread matches above, loose matches on Subject: below --
2006-06-15  9:07 Stephane Eranian

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200606152301_MC3-1-C28E-ADE0@compuserve.com \
    --to=76306.1226@compuserve.com \
    --cc=eranian@frankl.hpl.hp.com \
    --cc=eranian@hpl.hp.com \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®