From patchwork Thu Oct 24 00:09:29 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Omair Javaid X-Patchwork-Id: 21237 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-pa0-f70.google.com (mail-pa0-f70.google.com [209.85.220.70]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id 0251A20F5A for ; Thu, 24 Oct 2013 00:09:51 +0000 (UTC) Received: by mail-pa0-f70.google.com with SMTP id fb1sf2366804pad.5 for ; Wed, 23 Oct 2013 17:09:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:delivered-to:mime-version:from:date:message-id :subject:to:cc:x-original-sender:x-original-authentication-results :precedence:mailing-list:list-id:list-post:list-help:list-archive :list-unsubscribe:content-type; bh=eVk5Ac8DXBIoU2KFfrE36Rhr2j72xk1w2bOoCwYHXk4=; b=Cc/RmBfiPlgCnEpm38Pp9CweuOMih0CLuxTQpb5fvDifFa9UuWN+UMbVBAv60Jtiu2 tdMMD0CbZh3P3IAOiNBqotWTizBGfrIr4sWVdIjKpVsoYzKnjlOC8nV+SQKLOyGm/Sp3 1iBthP8dkqw8B9ZYAP458yKlWPwVXwdCnvBobOK8K5rSSg+GR+0V2zHS2Y+9jhL0BLLz /XUdfx9CoY3pbqkh9ES3iO5zHwr6g+tVTlL6PXRTHG5odqjKqzBsm/dCidgKG4UYPtHL SfftcPP5aUL9cjZn4kfqLC2uei/iMvEPFhhWXuELIwCg78AAK22150G2weDigK047XYI q68A== X-Received: by 10.66.240.4 with SMTP id vw4mr197386pac.10.1382573390961; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) X-BeenThere: patchwork-forward@linaro.org Received: by 10.49.13.9 with SMTP id d9ls704288qec.26.gmail; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) X-Received: by 10.52.229.234 with SMTP id st10mr3892vdc.34.1382573390702; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) Received: from mail-ve0-f176.google.com (mail-ve0-f176.google.com [209.85.128.176]) by mx.google.com with ESMTPS id st7si7831943vdc.146.2013.10.23.17.09.50 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 23 Oct 2013 17:09:50 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.128.176 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) client-ip=209.85.128.176; Received: by mail-ve0-f176.google.com with SMTP id jx11so864924veb.21 for ; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) X-Gm-Message-State: ALoCoQkEv7XmLIdTblCRt/2/JPDO9yGtp0XlveSxcazhzXth1Qwzq8JPXb1QWz3WmyzERQOwXGiF X-Received: by 10.58.255.233 with SMTP id at9mr6227ved.20.1382573390592; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patches@linaro.org Received: by 10.220.174.196 with SMTP id u4csp349530vcz; Wed, 23 Oct 2013 17:09:50 -0700 (PDT) X-Received: by 10.60.131.232 with SMTP id op8mr41400oeb.75.1382573389968; Wed, 23 Oct 2013 17:09:49 -0700 (PDT) Received: from mail-oa0-f50.google.com (mail-oa0-f50.google.com [209.85.219.50]) by mx.google.com with ESMTPS id m9si6404028oep.56.2013.10.23.17.09.49 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 23 Oct 2013 17:09:49 -0700 (PDT) Received-SPF: neutral (google.com: 209.85.219.50 is neither permitted nor denied by best guess record for domain of omair.javaid@linaro.org) client-ip=209.85.219.50; Received: by mail-oa0-f50.google.com with SMTP id j6so1030339oag.37 for ; Wed, 23 Oct 2013 17:09:49 -0700 (PDT) X-Received: by 10.182.55.8 with SMTP id n8mr41141obp.100.1382573389577; Wed, 23 Oct 2013 17:09:49 -0700 (PDT) MIME-Version: 1.0 Received: by 10.182.137.135 with HTTP; Wed, 23 Oct 2013 17:09:29 -0700 (PDT) From: Omair Javaid Date: Thu, 24 Oct 2013 05:09:29 +0500 Message-ID: Subject: [PATCH 2/2] GDB process record and reverse debugging improvements for arm*-linux* To: gdb-patches@sourceware.org Cc: Patch Tracking X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: omair.javaid@linaro.org X-Original-Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.128.176 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Precedence: list Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org List-ID: X-Google-Group-Id: 836684582541 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , This patch adds support for process record/replay system call recording for arm targets. gdb: 2013-10-24 Omair Javaid * arm-linux-tdep.c (struct arm_linux_record_tdep): Declare. (arm_canonicalize_syscall): New function. (arm_all_but_pc_registers_record): New function. (arm_linux_syscall_record): New function. (arm_linux_init_abi): Update. * arm-tdep.h (enum gdb_regnum): Update. (gdbarch_tdep): Update. /* Structures used for displaced stepping. */ Index: gdb/arm-linux-tdep.c =================================================================== RCS file: /cvs/src/src/gdb/arm-linux-tdep.c,v retrieving revision 1.97 diff -u -p -r1.97 arm-linux-tdep.c --- gdb/arm-linux-tdep.c 16 Sep 2013 11:28:53 -0000 1.97 +++ gdb/arm-linux-tdep.c 23 Oct 2013 23:10:11 -0000 @@ -44,6 +44,9 @@ #include "gdbthread.h" #include "symfile.h" +#include "record-full.h" +#include "linux-record.h" + #include "cli/cli-utils.h" #include "stap-probe.h" #include "parser-defs.h" @@ -1231,6 +1234,120 @@ arm_stap_parse_special_token (struct gdb return 1; } +/* ARM process record-replay constructs; syscall, signal etc. */ + +struct linux_record_tdep arm_linux_record_tdep; + +/* arm_canonicalize_syscall maps from the native arm Linux set + of syscall ids into a canonical set of syscall ids used by + process record. */ + +static enum gdb_syscall +arm_canonicalize_syscall (int syscall) +{ + enum { sys_process_vm_writev = 377 }; + + if (syscall <= gdb_sys_sched_getaffinity) + { + return syscall; + } + else if (syscall >= 243 && syscall <= 247 ) + { + return syscall + 2; + } + else if (syscall >= 248 && syscall <= 253 ) + { + return syscall + 4; + } + + return -1; +} + +/* Record all registers but PC register for process-record. */ + +static int +arm_all_but_pc_registers_record (struct regcache *regcache) +{ + if (record_full_arch_list_add_reg (regcache, ARM_R0_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R1_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R2_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R3_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R4_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R5_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R6_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R7_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R8_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R9_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R10_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R11_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R12_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R13_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_R14_REGNUM)) + return -1; + if (record_full_arch_list_add_reg (regcache, ARM_PS_REGNUM)) + return -1; + + return 0; +} + +/* Handler for arm system call instruction and recording. */ + +static int +arm_linux_syscall_record (struct regcache *regcache, unsigned long svc_number) +{ + int ret = 0; + enum gdb_syscall syscall_gdb; + + syscall_gdb = arm_canonicalize_syscall (svc_number); + + if (syscall_gdb < 0) + { + printf_unfiltered (_("Process record and replay target doesn't " + "support syscall number %s\n"), + plongest (svc_number)); + return -1; + } + + if (syscall_gdb == gdb_sys_sigreturn + || syscall_gdb == gdb_sys_rt_sigreturn) + { + if (arm_all_but_pc_registers_record (regcache)) + return -1; + return 0; + } + + ret = record_linux_system_call (syscall_gdb, regcache, + &arm_linux_record_tdep); + if (ret) + return ret; + + /* Record the return value of the system call. */ + if (record_full_arch_list_add_reg (regcache, ARM_R0_REGNUM)) + return -1; + /* Record LR. */ + if (record_full_arch_list_add_reg (regcache, ARM_LR_REGNUM)) + return -1; + /* Record CPSR. */ + if (record_full_arch_list_add_reg (regcache, ARM_PS_REGNUM)) + return -1; + + return 0; +} + static void arm_linux_init_abi (struct gdbarch_info info, struct gdbarch *gdbarch) @@ -1350,7 +1467,164 @@ arm_linux_init_abi (struct gdbarch_info set_gdbarch_get_syscall_number (gdbarch, arm_linux_get_syscall_number); /* Syscall record. */ - tdep->arm_swi_record = NULL; + tdep->arm_syscall_record = arm_linux_syscall_record; + + /* Initialize the arm_linux_record_tdep. */ + /* These values are the size of the type that will be used in a system + call. They are obtained from Linux Kernel source. */ + arm_linux_record_tdep.size_pointer + = gdbarch_ptr_bit (gdbarch) / TARGET_CHAR_BIT; + arm_linux_record_tdep.size__old_kernel_stat = 32; + arm_linux_record_tdep.size_tms = 16; + arm_linux_record_tdep.size_loff_t = 8; + arm_linux_record_tdep.size_flock = 16; + arm_linux_record_tdep.size_oldold_utsname = 45; + arm_linux_record_tdep.size_ustat = 20; + arm_linux_record_tdep.size_old_sigaction = 140; + arm_linux_record_tdep.size_old_sigset_t = 128; + arm_linux_record_tdep.size_rlimit = 8; + arm_linux_record_tdep.size_rusage = 72; + arm_linux_record_tdep.size_timeval = 8; + arm_linux_record_tdep.size_timezone = 8; + arm_linux_record_tdep.size_old_gid_t = 2; + arm_linux_record_tdep.size_old_uid_t = 2; + arm_linux_record_tdep.size_fd_set = 128; + arm_linux_record_tdep.size_dirent = 268; + arm_linux_record_tdep.size_dirent64 = 276; + arm_linux_record_tdep.size_statfs = 64; + arm_linux_record_tdep.size_statfs64 = 84; + arm_linux_record_tdep.size_sockaddr = 16; + arm_linux_record_tdep.size_int + = gdbarch_int_bit (gdbarch) / TARGET_CHAR_BIT; + arm_linux_record_tdep.size_long + = gdbarch_long_bit (gdbarch) / TARGET_CHAR_BIT; + arm_linux_record_tdep.size_ulong + = gdbarch_long_bit (gdbarch) / TARGET_CHAR_BIT; + arm_linux_record_tdep.size_msghdr = 28; + arm_linux_record_tdep.size_itimerval = 16; + arm_linux_record_tdep.size_stat = 88; + arm_linux_record_tdep.size_old_utsname = 325; + arm_linux_record_tdep.size_sysinfo = 64; + arm_linux_record_tdep.size_msqid_ds = 88; + arm_linux_record_tdep.size_shmid_ds = 84; + arm_linux_record_tdep.size_new_utsname = 390; + arm_linux_record_tdep.size_timex = 128; + arm_linux_record_tdep.size_mem_dqinfo = 24; + arm_linux_record_tdep.size_if_dqblk = 68; + arm_linux_record_tdep.size_fs_quota_stat = 68; + arm_linux_record_tdep.size_timespec = 8; + arm_linux_record_tdep.size_pollfd = 8; + arm_linux_record_tdep.size_NFS_FHSIZE = 32; + arm_linux_record_tdep.size_knfsd_fh = 132; + arm_linux_record_tdep.size_TASK_COMM_LEN = 16; + arm_linux_record_tdep.size_sigaction = 140; + arm_linux_record_tdep.size_sigset_t = 8; + arm_linux_record_tdep.size_siginfo_t = 128; + arm_linux_record_tdep.size_cap_user_data_t = 12; + arm_linux_record_tdep.size_stack_t = 12; + arm_linux_record_tdep.size_off_t = arm_linux_record_tdep.size_long; + arm_linux_record_tdep.size_stat64 = 96; + arm_linux_record_tdep.size_gid_t = 2; + arm_linux_record_tdep.size_uid_t = 2; + arm_linux_record_tdep.size_PAGE_SIZE = 4096; + arm_linux_record_tdep.size_flock64 = 24; + arm_linux_record_tdep.size_user_desc = 16; + arm_linux_record_tdep.size_io_event = 32; + arm_linux_record_tdep.size_iocb = 64; + arm_linux_record_tdep.size_epoll_event = 12; + arm_linux_record_tdep.size_itimerspec + = arm_linux_record_tdep.size_timespec * 2; + arm_linux_record_tdep.size_mq_attr = 32; + arm_linux_record_tdep.size_siginfo = 128; + arm_linux_record_tdep.size_termios = 36; + arm_linux_record_tdep.size_termios2 = 44; + arm_linux_record_tdep.size_pid_t = 4; + arm_linux_record_tdep.size_winsize = 8; + arm_linux_record_tdep.size_serial_struct = 60; + arm_linux_record_tdep.size_serial_icounter_struct = 80; + arm_linux_record_tdep.size_hayes_esp_config = 12; + arm_linux_record_tdep.size_size_t = 4; + arm_linux_record_tdep.size_iovec = 8; + + /* These values are the second argument of system call "sys_ioctl". + They are obtained from Linux Kernel source. */ + arm_linux_record_tdep.ioctl_TCGETS = 0x5401; + arm_linux_record_tdep.ioctl_TCSETS = 0x5402; + arm_linux_record_tdep.ioctl_TCSETSW = 0x5403; + arm_linux_record_tdep.ioctl_TCSETSF = 0x5404; + arm_linux_record_tdep.ioctl_TCGETA = 0x5405; + arm_linux_record_tdep.ioctl_TCSETA = 0x5406; + arm_linux_record_tdep.ioctl_TCSETAW = 0x5407; + arm_linux_record_tdep.ioctl_TCSETAF = 0x5408; + arm_linux_record_tdep.ioctl_TCSBRK = 0x5409; + arm_linux_record_tdep.ioctl_TCXONC = 0x540A; + arm_linux_record_tdep.ioctl_TCFLSH = 0x540B; + arm_linux_record_tdep.ioctl_TIOCEXCL = 0x540C; + arm_linux_record_tdep.ioctl_TIOCNXCL = 0x540D; + arm_linux_record_tdep.ioctl_TIOCSCTTY = 0x540E; + arm_linux_record_tdep.ioctl_TIOCGPGRP = 0x540F; + arm_linux_record_tdep.ioctl_TIOCSPGRP = 0x5410; + arm_linux_record_tdep.ioctl_TIOCOUTQ = 0x5411; + arm_linux_record_tdep.ioctl_TIOCSTI = 0x5412; + arm_linux_record_tdep.ioctl_TIOCGWINSZ = 0x5413; + arm_linux_record_tdep.ioctl_TIOCSWINSZ = 0x5414; + arm_linux_record_tdep.ioctl_TIOCMGET = 0x5415; + arm_linux_record_tdep.ioctl_TIOCMBIS = 0x5416; + arm_linux_record_tdep.ioctl_TIOCMBIC = 0x5417; + arm_linux_record_tdep.ioctl_TIOCMSET = 0x5418; + arm_linux_record_tdep.ioctl_TIOCGSOFTCAR = 0x5419; + arm_linux_record_tdep.ioctl_TIOCSSOFTCAR = 0x541A; + arm_linux_record_tdep.ioctl_FIONREAD = 0x541B; + arm_linux_record_tdep.ioctl_TIOCINQ = arm_linux_record_tdep.ioctl_FIONREAD; + arm_linux_record_tdep.ioctl_TIOCLINUX = 0x541C; + arm_linux_record_tdep.ioctl_TIOCCONS = 0x541D; + arm_linux_record_tdep.ioctl_TIOCGSERIAL = 0x541E; + arm_linux_record_tdep.ioctl_TIOCSSERIAL = 0x541F; + arm_linux_record_tdep.ioctl_TIOCPKT = 0x5420; + arm_linux_record_tdep.ioctl_FIONBIO = 0x5421; + arm_linux_record_tdep.ioctl_TIOCNOTTY = 0x5422; + arm_linux_record_tdep.ioctl_TIOCSETD = 0x5423; + arm_linux_record_tdep.ioctl_TIOCGETD = 0x5424; + arm_linux_record_tdep.ioctl_TCSBRKP = 0x5425; + arm_linux_record_tdep.ioctl_TIOCTTYGSTRUCT = 0x5426; + arm_linux_record_tdep.ioctl_TIOCSBRK = 0x5427; + arm_linux_record_tdep.ioctl_TIOCCBRK = 0x5428; + arm_linux_record_tdep.ioctl_TIOCGSID = 0x5429; + arm_linux_record_tdep.ioctl_TCGETS2 = 0x802c542a; + arm_linux_record_tdep.ioctl_TCSETS2 = 0x402c542b; + arm_linux_record_tdep.ioctl_TCSETSW2 = 0x402c542c; + arm_linux_record_tdep.ioctl_TCSETSF2 = 0x402c542d; + arm_linux_record_tdep.ioctl_TIOCGPTN = 0x80045430; + arm_linux_record_tdep.ioctl_TIOCSPTLCK = 0x40045431; + arm_linux_record_tdep.ioctl_FIONCLEX = 0x5450; + arm_linux_record_tdep.ioctl_FIOCLEX = 0x5451; + arm_linux_record_tdep.ioctl_FIOASYNC = 0x5452; + arm_linux_record_tdep.ioctl_TIOCSERCONFIG = 0x5453; + arm_linux_record_tdep.ioctl_TIOCSERGWILD = 0x5454; + arm_linux_record_tdep.ioctl_TIOCSERSWILD = 0x5455; + arm_linux_record_tdep.ioctl_TIOCGLCKTRMIOS = 0x5456; + arm_linux_record_tdep.ioctl_TIOCSLCKTRMIOS = 0x5457; + arm_linux_record_tdep.ioctl_TIOCSERGSTRUCT = 0x5458; + arm_linux_record_tdep.ioctl_TIOCSERGETLSR = 0x5459; + arm_linux_record_tdep.ioctl_TIOCSERGETMULTI = 0x545A; + arm_linux_record_tdep.ioctl_TIOCSERSETMULTI = 0x545B; + arm_linux_record_tdep.ioctl_TIOCMIWAIT = 0x545C; + arm_linux_record_tdep.ioctl_TIOCGICOUNT = 0x545D; + arm_linux_record_tdep.ioctl_TIOCGHAYESESP = 0x545E; + arm_linux_record_tdep.ioctl_TIOCSHAYESESP = 0x545F; + arm_linux_record_tdep.ioctl_FIOQSIZE = 0x5460; + + /* These values are the second argument of system call "sys_fcntl" + and "sys_fcntl64". They are obtained from Linux Kernel source. */ + arm_linux_record_tdep.fcntl_F_GETLK = 5; + arm_linux_record_tdep.fcntl_F_GETLK64 = 12; + arm_linux_record_tdep.fcntl_F_SETLK64 = 13; + arm_linux_record_tdep.fcntl_F_SETLKW64 = 14; + + arm_linux_record_tdep.arg1 = ARM_R1_REGNUM; + arm_linux_record_tdep.arg2 = ARM_R2_REGNUM; + arm_linux_record_tdep.arg3 = ARM_R3_REGNUM; + arm_linux_record_tdep.arg4 = ARM_R4_REGNUM; } /* Provide a prototype to silence -Wmissing-prototypes. */ Index: gdb/arm-tdep.h =================================================================== RCS file: /cvs/src/src/gdb/arm-tdep.h,v retrieving revision 1.59 diff -u -p -r1.59 arm-tdep.h --- gdb/arm-tdep.h 19 Apr 2013 15:09:45 -0000 1.59 +++ gdb/arm-tdep.h 23 Oct 2013 23:10:11 -0000 @@ -60,7 +60,24 @@ enum gdb_regnum { ARM_NUM_ARG_REGS = 4, ARM_LAST_ARG_REGNUM = ARM_A4_REGNUM, ARM_NUM_FP_ARG_REGS = 4, - ARM_LAST_FP_ARG_REGNUM = ARM_F3_REGNUM + ARM_LAST_FP_ARG_REGNUM = ARM_F3_REGNUM, + + /* GPRs register numbers in sequence. */ + ARM_R0_REGNUM = ARM_A1_REGNUM, + ARM_R1_REGNUM = ARM_R0_REGNUM + 1, + ARM_R2_REGNUM = ARM_R1_REGNUM + 1, + ARM_R3_REGNUM = ARM_R2_REGNUM + 1, + ARM_R4_REGNUM = ARM_R3_REGNUM + 1, + ARM_R5_REGNUM = ARM_R4_REGNUM + 1, + ARM_R6_REGNUM = ARM_R5_REGNUM + 1, + ARM_R7_REGNUM = ARM_R6_REGNUM + 1, + ARM_R8_REGNUM = ARM_R7_REGNUM + 1, + ARM_R9_REGNUM = ARM_R8_REGNUM + 1, + ARM_R10_REGNUM = ARM_R9_REGNUM + 1, + ARM_R11_REGNUM = ARM_R10_REGNUM + 1, + ARM_R12_REGNUM = ARM_R11_REGNUM + 1, + ARM_R13_REGNUM = ARM_R12_REGNUM + 1, + ARM_R14_REGNUM = ARM_R13_REGNUM + 1 }; /* Size of integer registers. */ @@ -205,8 +222,8 @@ struct gdbarch_tdep instruction. */ CORE_ADDR (*syscall_next_pc) (struct frame_info *frame); - /* Parse swi insn args, sycall record. */ - int (*arm_swi_record) (struct regcache *regcache); + /* syscall record. */ + int (*arm_syscall_record) (struct regcache *regcache, unsigned long svc_number); };