Re: New syscalls

[Andrew Cagney <cagney@redhat.com>]

Sami, Mark,
Nice catch.  That check is doing its job  (Now if only this was all in 
an xml file ;-).
Andrew

Mark Wielaard wrote:
> Hi,
>
> The latest AuditLibs pointed out that there are some new syscalls for
> IA32 and PowerPC. This patch adds them and changes the test to check
> that we know at least the number of syscalls that auditlib knows about.
>
> 2007-02-08  Mark Wielaard  <mark@klomp.org>
>
>     * LinuxIa32Syscall.java (syscallList): Add getcpu and epoll_pwait.
>     * LinuxPowerPCSyscall.java (syscallList): Add move_pages.
>     * TestSyscallsWithAudit.java (syscallTest): Change test to up to
>     at least highest number in auditlib.
>
> It might be a good idea to better integrate the names and numbers as
> reported by auditlib on the users system. Unfortunately auditlib doesn't
> know all the details we need, so we still need to augment the list with
> extra info (like whether or not the call returns, parameter list, etc).
>
> Committed,
>
> Mark
>   
> ------------------------------------------------------------------------
>
> Index: frysk-core/frysk/proc/LinuxIa32Syscall.java
> ===================================================================
> RCS file: /cvs/frysk/frysk-core/frysk/proc/LinuxIa32Syscall.java,v
> retrieving revision 1.5
> diff -u -r1.5 LinuxIa32Syscall.java
> --- frysk-core/frysk/proc/LinuxIa32Syscall.java	4 Dec 2006 20:53:55 -0000	1.5
> +++ frysk-core/frysk/proc/LinuxIa32Syscall.java	8 Feb 2007 11:56:52 -0000
> @@ -440,7 +440,9 @@
>      new Ia32Syscall ("sync_file_range", 314),
>      new Ia32Syscall ("tee", 315),
>      new Ia32Syscall ("vmsplice", 316),
> -    new Ia32Syscall ("move_pages", 317)
> +    new Ia32Syscall ("move_pages", 317),
> +    new Ia32Syscall ("getcpu", 318),
> +    new Ia32Syscall ("epoll_pwait", 319)
>      };
>  
>  
> Index: frysk-core/frysk/proc/LinuxPowerPCSyscall.java
> ===================================================================
> RCS file: /cvs/frysk/frysk-core/frysk/proc/LinuxPowerPCSyscall.java,v
> retrieving revision 1.5
> diff -u -r1.5 LinuxPowerPCSyscall.java
> --- frysk-core/frysk/proc/LinuxPowerPCSyscall.java	4 Dec 2006 20:53:55 -0000	1.5
> +++ frysk-core/frysk/proc/LinuxPowerPCSyscall.java	8 Feb 2007 11:56:52 -0000
> @@ -429,7 +429,8 @@
>      new PowerPCSyscall ("fchmodat", 297),
>      new PowerPCSyscall ("faccessat", 298),
>      new PowerPCSyscall ("get_robust_list", 299),
> -    new PowerPCSyscall ("set_robust_list", 300)
> +    new PowerPCSyscall ("set_robust_list", 300),
> +    new PowerPCSyscall ("move_pages", 301)
>      };
>  
>    static class SocketSubSyscall
> Index: frysk/proc/TestSyscallsWithAudit.java
> ===================================================================
> RCS file: /cvs/frysk/frysk-core/frysk/proc/TestSyscallsWithAudit.java,v
> retrieving revision 1.6
> diff -u -r1.6 TestSyscallsWithAudit.java
> --- frysk/proc/TestSyscallsWithAudit.java	14 Oct 2006 22:25:13 -0000	1.6
> +++ frysk/proc/TestSyscallsWithAudit.java	8 Feb 2007 12:11:03 -0000
> @@ -126,7 +126,8 @@
>      assertNull("MAX_SYSCALL_NUM", AuditLibs.syscallToName(MAX_SYSCALL_NUM,
>  							  machine));
>  
> -    // We should have names up to the highest number auditlib knows about.
> -    assertEquals("max-syscall-num", highestNum, syscallList.length - 1);
> +    // We should have names up to at least the highest number auditlib
> +    // knows about.
> +    assertTrue("max-syscall-num", highestNum <= syscallList.length - 1);
>    }
>  }
>