Re: Release plans?

2012-03-12 Thread Dmitry V. Levin
On Mon, Mar 12, 2012 at 01:26:35PM +0100, Denys Vlasenko wrote: > On 03/10/2012 11:17 PM, Dmitry V. Levin wrote: > > On Thu, Feb 09, 2012 at 07:38:27PM +0100, Denys Vlasenko wrote: > >> On 02/09/2012 05:29 PM, Dmitry V. Levin wrote: > >>> But I'd like to

Re: Release plans?

2012-03-12 Thread Dmitry V. Levin
On Mon, Mar 12, 2012 at 08:00:46PM +0100, Denys Vlasenko wrote: > On 03/12/2012 02:47 PM, Dmitry V. Levin wrote: > >>>>> But I'd like to hear from Denys about PTRACE_SEIZE perspective first. > >>>>> We surely won't release PTRACE_SEIZE_DEVEL cod

Re: Release plans?

2012-03-12 Thread Dmitry V. Levin
On Mon, Mar 12, 2012 at 06:37:01PM +, Grant Edwards wrote: > On 2012-03-12, Denys Vlasenko wrote: > > On 03/10/2012 11:17 PM, Dmitry V. Levin wrote: > >> On Thu, Feb 09, 2012 at 07:38:27PM +0100, Denys Vlasenko wrote: > >>> On 02/09/2012 05:29 PM, Dmitry V. Levin

Re: tagged version vs released tarball

2012-03-12 Thread Dmitry V. Levin
On Mon, Mar 12, 2012 at 08:50:44PM +0100, Abdoulaye Walsimou GAYE wrote: > Sorry for this thread hijack, Could you at least change the subject next time, please? Thanks. > but I noticed that tagged version and corresponding > tarball does not contain the same files. This prevents for example to

upcoming release: what needs to be done

2012-03-12 Thread Dmitry V. Levin
Hi, There is quite enough good stuff accumulated in strace.git since v4.6 to have a new release soon. If there are no objections, here is a list of pre-release work to be done for this release: - merge pending patches if any (please speak up); - test on supported non-x86 architectures; - update

Re: [SCM] strace branch, master, updated. v4.6-258-ge8172b7

2012-03-12 Thread Dmitry V. Levin
On Fri, Mar 09, 2012 at 12:03:01PM +, Denys Vlasenko wrote: > This is an automated email from the git hooks/post-receive script. It was > generated because a ref change was pushed to the repository containing > the project "strace". [...] > Allow -p PID to take comma or whitespace-separated

Re: upcoming release: what needs to be done

2012-03-14 Thread Dmitry V. Levin
On Wed, Mar 14, 2012 at 08:39:35PM -0400, Mike Frysinger wrote: > On Monday 12 March 2012 17:50:48 Dmitry V. Levin wrote: > > - test on supported non-x86 architectures; > > build fails with <=glibc-2.12 because SWAP_FLAG_DISCARD wasn't added until > then. files.c unc

Re: RFC: better signal names?

2012-03-15 Thread Dmitry V. Levin
On Thu, Mar 15, 2012 at 04:13:18PM +0100, Denys Vlasenko wrote: > Currently we emit something like this: > > --- {si_signo=SIGCHLD, si_code=CLD_EXITED, si_pid=19483, si_status=0, > si_utime=0, si_stime=0} (Child exited) --- > --- {si_signo=SIGWINCH, si_code=SI_USER, si_pid=15407, si_uid=0} (Windo

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Wed, Mar 14, 2012 at 08:39:35PM -0400, Mike Frysinger wrote: > alpha looks broken with signals ... Not only with signals, IPC decoding also seems to be broken on alpha: SYS_ipc_subcall is naturally not defined, but indirect_ipccall() returns 1 there. Could you confirm that, e.g. by checking wh

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Fri, Mar 16, 2012 at 01:10:37AM +0400, Dmitry V. Levin wrote: > On Wed, Mar 14, 2012 at 08:39:35PM -0400, Mike Frysinger wrote: > > alpha looks broken with signals ... > > Not only with signals, IPC decoding also seems to be broken on alpha: > SYS_ipc_subcall is naturall

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Thu, Mar 15, 2012 at 01:56:12PM -0400, Mike Frysinger wrote: > superh compiles and passes make check. it warns about the new system.c code > (strict aliasing), but this is a 4.4 compiler, so i'm going to update it and > see if that makes a difference. btw, I've got these warnings on ARM EABI

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Thu, Mar 15, 2012 at 06:36:16PM -0400, Mike Frysinger wrote: > On Thursday 15 March 2012 17:10:37 Dmitry V. Levin wrote: > > On Wed, Mar 14, 2012 at 08:39:35PM -0400, Mike Frysinger wrote: > > > alpha looks broken with signals ... > > > > Not only with signals,

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Thu, Mar 15, 2012 at 12:53:23AM -0400, Mike Frysinger wrote: > arm hits this warning: > signal.c: In function ‘sys_rt_sigaction’: > signal.c:1345:4: warning: left shift count >= width of type I saw this warning a year ago as well. The code is unreachable on ARM, but gcc seems to be too dumb to

Re: upcoming release: what needs to be done

2012-03-15 Thread Dmitry V. Levin
On Thu, Mar 15, 2012 at 01:17:56AM -0400, Mike Frysinger wrote: > hppa hit this warning: > ipc.c: In function ‘sys_shmat’: > ipc.c:407:16: warning: unused variable ‘raddr’ > but looking at the code, we could add another ifdef around the variable > definition, but i'm not sure that real

Q: can rlim_t be a long long type on linux

2012-03-16 Thread Dmitry V. Levin
Hi, While writing a prlimit64 syscall parser I found a seemingly dead HAVE_LONG_LONG_RLIM_T code in resource.c that implements [gs]etrlimit parsers for systems where sizeof(rlim_t) == sizeof(long long). Looking at "struct rlimit" and "struct compat_rlimit" definitions in include/linux/resource.h

Re: Q: can rlim_t be a long long type on linux

2012-03-16 Thread Dmitry V. Levin
On Fri, Mar 16, 2012 at 05:45:51PM -0400, Mike Frysinger wrote: > On Friday 16 March 2012 16:18:33 Andreas Schwab wrote: > > "Dmitry V. Levin" writes: > > > Is it correct that rlim_t cannot be a long long type on linux, > > > or am I missing something?

Re: [SCM] strace branch, master, updated. v4.6-333-g31972d5

2012-03-17 Thread Dmitry V. Levin
On Sat, Mar 17, 2012 at 12:28:50PM +, Denys Vlasenko wrote: > This is an automated email from the git hooks/post-receive script. It was > generated because a ref change was pushed to the repository containing > the project "strace". [...] > commit 31972d52b1059d8faca1c5f417c2db1a90b868ae > Auth

Re: Q: can rlim_t be a long long type on linux

2012-03-17 Thread Dmitry V. Levin
On Fri, Mar 16, 2012 at 05:45:51PM -0400, Mike Frysinger wrote: > On Friday 16 March 2012 16:18:33 Andreas Schwab wrote: > > "Dmitry V. Levin" writes: > > > Is it correct that rlim_t cannot be a long long type on linux, > > > or am I missing something?

Re: bug in commit 20f6b54385d2462d858419f7c67509cb3d22d155

2012-03-26 Thread Dmitry V. Levin
On Mon, Mar 26, 2012 at 01:56:22PM +0200, Andreas Schwab wrote: > Denys Vlasenko writes: > > > Hi Dmitry, > > > > for file in "$logfile".*; do > > [ -f "$file" ] || continue > > - suffix=${file:1+$pfxlen} > > - [ "$suffix" -gt 0 ] 2> /dev/null || { > > - echo "S

Re: [PATCH] powerpc: Add syscall entries for socket system calls

2012-03-26 Thread Dmitry V. Levin
On Tue, Mar 27, 2012 at 10:34:54AM +1100, Anton Blanchard wrote: > > * linux/powerpc/syscallent.h: Add socket system calls Just for the record, what have happened with the socket subcall on ppc? > diff --git a/linux/powerpc/syscallent.h b/linux/powerpc/syscallent.h > index 8dd56bd..a382bdc

Re: Concurrency output issue

2012-03-26 Thread Dmitry V. Levin
Hi, On Mon, Mar 26, 2012 at 05:24:07PM -0400, Bao Hong Tan wrote: > Hi all, > > I looked at strace's output and saw the following: > > 01:20:00.727079 recv(-1345034744, 01:20:00.727768 write(24, "338", > 301:20:00.728785 SYS_224(0x3adfa8, 0x40025730, 0xa, 0x45006f40) = 0xccd000 > 01:20:00.72982

Re: [PATCH v2] decode mtd ioctls

2012-04-03 Thread Dmitry V. Levin
On Tue, Apr 03, 2012 at 02:24:33AM -0400, Mike Frysinger wrote: > I got tired of figuring out mtd structures (which show up a lot > in the embedded space), so add decoders for those ioctls. > > * defs.h (mtd_ioctl): New prototype. > (print_loff_t): Likewise. > * io.c (print_loff_t): Delete static

Re: upcoming release: what needs to be done

2012-04-03 Thread Dmitry V. Levin
On Mon, Apr 02, 2012 at 11:26:06PM -0400, Mike Frysinger wrote: > On Monday 12 March 2012 17:50:48 Dmitry V. Levin wrote: > > There is quite enough good stuff accumulated in strace.git since v4.6 to > > have a new release soon. If there are no objections, here is a list of > >

Re: [PATCH v2] decode mtd ioctls

2012-04-04 Thread Dmitry V. Levin
On Tue, Apr 03, 2012 at 10:22:48PM -0400, Mike Frysinger wrote: > On Tuesday 03 April 2012 19:11:32 Dmitry V. Levin wrote: > > On Tue, Apr 03, 2012 at 02:24:33AM -0400, Mike Frysinger wrote: > > > I got tired of figuring out mtd structures (which show up a lot > > > in

Re: [PATCH v2] decode mtd ioctls

2012-04-04 Thread Dmitry V. Levin
On Wed, Apr 04, 2012 at 08:31:34PM -0400, Mike Frysinger wrote: > On Wednesday 04 April 2012 14:14:49 Dmitry V. Levin wrote: > > On Tue, Apr 03, 2012 at 10:22:48PM -0400, Mike Frysinger wrote: > > > On Tuesday 03 April 2012 19:11:32 Dmitry V. Levin wrote: > > > > On T

Re: [PATCH] fix indefinite hang on no-mmu systems

2012-04-06 Thread Dmitry V. Levin
On Thu, Apr 05, 2012 at 01:52:25AM -0400, Mike Frysinger wrote: > The ptrace setoptions code will fork a child which goes to sleep and > expects the parent to continue on to do tests. Unfortunately, this > does not work on no-mmu systems as fork() is actually vfork() and any > vforked children wil

Re: [PATCH v4] decode mtd ioctls

2012-04-06 Thread Dmitry V. Levin
On Wed, Apr 04, 2012 at 10:22:01PM -0400, Mike Frysinger wrote: > I got tired of figuring out mtd structures (which show up a lot > in the embedded space), so add decoders for those ioctls. > > * defs.h (mtd_ioctl): New prototype. > (print_loff_t): Likewise. > * io.c (print_loff_t): Delete static

Re: PATCH: Add x32 support to strace

2012-04-16 Thread Dmitry V. Levin
On Sun, Apr 15, 2012 at 07:09:02PM -0700, H.J. Lu wrote: > --- a/configure.ac > +++ b/configure.ac > @@ -8,6 +8,11 @@ AM_INIT_AUTOMAKE([foreign check-news dist-xz no-dist-gzip > silent-rules]) > AM_MAINTAINER_MODE > AC_CANONICAL_HOST > > +AC_PROG_CC > +AC_GNU_SOURCE > + > +AC_USE_SYSTEM_EXTENS

Re: [PATCH 1/2] Cast clock_t type to unsigned long

2012-04-16 Thread Dmitry V. Levin
Hi, On Mon, Apr 16, 2012 at 07:39:58AM -0700, H.J. Lu wrote: > On Sun, Apr 15, 2012 at 11:17:13AM -0700, H.J. Lu wrote: > > > > Linux kernel v3.4 adds x32 support whose clock_t is long long. This > > patch casts clock_t type to unsigned long for "%lu". > > > > H.J. > > --- > > Here is a patch

Re: [PATCH 1/2] Cast clock_t type to unsigned long

2012-04-16 Thread Dmitry V. Levin
On Mon, Apr 16, 2012 at 08:31:46AM -0700, H.J. Lu wrote: > On Mon, Apr 16, 2012 at 8:14 AM, Dmitry V. Levin wrote: > > On Mon, Apr 16, 2012 at 07:39:58AM -0700, H.J. Lu wrote: > >> On Sun, Apr 15, 2012 at 11:17:13AM -0700, H.J. Lu wrote: > >> > > >> >

Re: PATCH: Add ia32 support to x32 strace

2012-04-17 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 11:05:04AM -0700, H.J. Lu wrote: > Hi, > > This patch adds ia32 support to x32 strace. Tested on Linux/x32. [...] > @@ -868,15 +874,14 @@ get_scno(struct tcb *tcp) > } > # endif > # ifdef X32 > - if (currpers == 0 || currpers == 1) { > - fprintf(std

Re: PATCH: Add ia32 support to x32 strace

2012-04-17 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 05:03:20PM -0700, H.J. Lu wrote: > On Tue, Apr 17, 2012 at 4:58 PM, Dmitry V. Levin wrote: > > On Tue, Apr 17, 2012 at 11:05:04AM -0700, H.J. Lu wrote: > >> Hi, > >> > >> This patch adds ia32 support to x32 strace.  Tested on Linux/x3

Re: [PATCH] Decode /dev/loop ioctls

2012-04-17 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 04:36:12PM -0400, Mike Frysinger wrote: > Needed to debug some losetup failures, and it's easier when you can see > what the kernel is getting vs what you think you're sending, so add some > decoders for those ioctls. Thanks. [...] > + if (!abbrev(tcp)) { > +

Re: PATCH: Add ia32 support to x32 strace

2012-04-18 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 05:39:28PM -0700, H.J. Lu wrote: > On Tue, Apr 17, 2012 at 5:13 PM, Dmitry V. Levin wrote: > > On Tue, Apr 17, 2012 at 05:03:20PM -0700, H.J. Lu wrote: > >> On Tue, Apr 17, 2012 at 4:58 PM, Dmitry V. Levin wrote: > >> > On Tue, Apr 17, 201

Re: [PATCH v2] Decode /dev/loop ioctls

2012-04-18 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 10:19:31PM -0400, Mike Frysinger wrote: > Needed to debug some losetup failures, and it's easier when you can see > what the kernel is getting vs what you think you're sending, so add some > decoders for those ioctls. Applied, thanks. [...] > + if (!abbrev(tcp)

Re: strace sys_pread / sys_pwrite : double definition

2012-04-26 Thread Dmitry V. Levin
Hi, On Thu, Apr 26, 2012 at 01:52:28PM +0100, dark_footix wrote: > Hello Strace, > > Do you have a bugzilla ? It's perfectly fine to report bugs in strace to this mailing list. > There is an issue of compilation on the release 4.6,  with   > > ../strace-4.6/io.c:356:1: error: redefinition of '

Re: upcoming release: what needs to be done

2012-04-27 Thread Dmitry V. Levin
On Wed, Apr 04, 2012 at 03:38:06AM +0400, Dmitry V. Levin wrote: > On Mon, Apr 02, 2012 at 11:26:06PM -0400, Mike Frysinger wrote: > > On Monday 12 March 2012 17:50:48 Dmitry V. Levin wrote: > > > There is quite enough good stuff accumulated in strace.git since v4.6 to > &

Re: [PATCH] cast current_wordsize to an int

2012-04-27 Thread Dmitry V. Levin
On Fri, Apr 27, 2012 at 06:58:20PM -0400, Mike Frysinger wrote: > On 64bit systems with a single personality, we see: > count.c: In function 'call_summary': > count.c:223:5: warning: format '%u' expects type 'unsigned int', > but argument 3 has type 'long unsigned int' > > Since on multi-per

Re: [PATCH] NEWS: clarify & fix typo

2012-04-27 Thread Dmitry V. Levin
On Fri, Apr 27, 2012 at 07:13:55PM -0400, Mike Frysinger wrote: > NEWS |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Thanks. -- ldv pgpNpUCt3TGub.pgp Description: PGP signature -- Live Security Virtu

Re: [SCM] strace branch, master, updated. v4.6-390-g3efa7c7

2012-05-01 Thread Dmitry V. Levin
On Sat, Apr 28, 2012 at 03:01:15PM +, Denys Vlasenko wrote: [...] > - Log - > commit 3efa7c7f1be0f54e0656de5cb4a5f4c39db10150 > Author: Denys Vlasenko > Date: Sat Apr 28 16:59:47 2012 +0200 > > Enable printing of uts.domain

Re: upcoming release: what needs to be done

2012-05-01 Thread Dmitry V. Levin
On Sat, Apr 28, 2012 at 05:18:47PM -0400, Mike Frysinger wrote: > On Friday 27 April 2012 19:21:26 Mike Frysinger wrote: > > my sh/sparc boxes are offline atm ... need to get someone to kick them. > > i recovered the sh box. `make check` failed because the timeout in strace-f > was too low. it'

Re: [SCM] strace branch, master, updated. v4.6-390-g3efa7c7

2012-05-01 Thread Dmitry V. Levin
On Tue, May 01, 2012 at 09:55:56PM +0400, Dmitry V. Levin wrote: > On Sat, Apr 28, 2012 at 03:01:15PM +, Denys Vlasenko wrote: > [...] > > - Log - > > commit 3efa7c7f1be0f54e0656de5cb4a5f4c39db10150 > >

Re: [PATCH v2] Decode /dev/loop ioctls

2012-05-02 Thread Dmitry V. Levin
On Tue, Apr 17, 2012 at 10:19:31PM -0400, Mike Frysinger wrote: > Needed to debug some losetup failures, and it's easier when you can see > what the kernel is getting vs what you think you're sending, so add some > decoders for those ioctls. > > * defs.h (loop_ioctl): New prototype. > (string_quot

Re: [PATCH v2] Decode /dev/loop ioctls

2012-05-02 Thread Dmitry V. Levin
On Wed, May 02, 2012 at 02:09:55PM +0400, Dmitry V. Levin wrote: > On Tue, Apr 17, 2012 at 10:19:31PM -0400, Mike Frysinger wrote: > > Needed to debug some losetup failures, and it's easier when you can see > > what the kernel is getting vs what you think you're sending,

Re: [PATCH v2] Decode /dev/loop ioctls

2012-05-02 Thread Dmitry V. Levin
On Wed, May 02, 2012 at 11:54:19AM -0400, Mike Frysinger wrote: > On Wednesday 02 May 2012 06:38:44 Dmitry V. Levin wrote: > > On Wed, May 02, 2012 at 02:09:55PM +0400, Dmitry V. Levin wrote: > > > On Tue, Apr 17, 2012 at 10:19:31PM -0400, Mike Frysinger wrote: > > > >

strace 4.7 released

2012-05-02 Thread Dmitry V. Levin
On Fri, Apr 27, 2012 at 09:59:23PM +0400, Dmitry V. Levin wrote: [...] > I suggest to test HEAD once more on available architectures, to ensure that > we haven't accidentally broken something last month. If there are no > regressions, we could release 4.7 in the middle of the next

on updating NEWS file

2012-05-03 Thread Dmitry V. Levin
On Fri, Apr 27, 2012 at 09:59:23PM +0400, Dmitry V. Levin wrote: > On Wed, Apr 04, 2012 at 03:38:06AM +0400, Dmitry V. Levin wrote: > > On Mon, Apr 02, 2012 at 11:26:06PM -0400, Mike Frysinger wrote: [...] > > > anything still pending ? > > > > The NEWS file; unf

Re: strace -f (v4.7) randomly failing on grsecurity and Ubuntu kernels

2012-05-09 Thread Dmitry V. Levin
Hi, On Tue, May 08, 2012 at 07:43:28PM -0400, Brad Spengler wrote: > Hi guys, > > I'm writing to report this before the recent release sees more > widespread use. I've already had one report from a user of strace v4.7 > failing on a grsecurity kernel when run with the -f argument. Strace > (

Re: strace shows unknown syscall newfstatat.

2012-05-10 Thread Dmitry V. Levin
On Thu, May 10, 2012 at 06:01:01PM +0600, Марк Коренберг wrote: > There is no such syscall. May be you mean fstatat64()? strace$ git grep -E 'newfstatat|fstatat64' linux/i386/ linux/x86_64/ linux/i386/syscallent.h:{ 4,TD|TF, sys_newfstatat, "fstatat64" }, /* 300 */ linux/

Re: [PATCH] Modify the OS string parsing to handle two number strings

2012-05-14 Thread Dmitry V. Levin
On Mon, May 14, 2012 at 10:34:46PM +1000, Bryce Gibson wrote: > Modify OS string parsing to handle a two character OS string. > If a shorter than expected OS string is found, pad a zero on the end and then > try the comparison. > --- > strace.c | 5 + > 1 file changed, 5 insertions(+) > > diff

Re: strace scheduling fairness

2012-05-16 Thread Dmitry V. Levin
On Tue, May 15, 2012 at 05:09:21PM +0200, Denys Vlasenko wrote: > #include > #include > #include > #define NTHD 10 > static int thd_no; > static void *sub_thd(void *c) { > printf("thread created: %d\n", ++thd_no); > for (;;) getuid(); > return NULL; > } > int main(int

Re: strace scheduling fairness

2012-05-17 Thread Dmitry V. Levin
On Wed, May 16, 2012 at 12:47:14PM +0200, Denys Vlasenko wrote: > On 05/16/2012 11:48 AM, Dmitry V. Levin wrote: [...] > >> One solution is to run waitpid() in a loop until it returns 0 "no more > >> tracees > >> to wait", then handle them all. It was NA

Re: missing clock_gettime constants

2012-05-19 Thread Dmitry V. Levin
On Sun, May 20, 2012 at 12:15:24AM +0600, Марк Коренберг wrote: > clock_gettime(0x4 /* CLOCK_??? */, {190897, 528960021}) = 0 > > in my case, it was CLOCK_MONOTONIC_RAW. > > It will be nice if someone adds all known clock_gettime constants to strace. Some constants (including CLOCK_MONOTONIC_RAW

Re: [PATCH] x32: update syscall table

2012-06-05 Thread Dmitry V. Levin
On Mon, Jun 04, 2012 at 02:01:59PM -0400, Mike Frysinger wrote: > This syncs with the syscall table as it is in linux 3.4. > > * linux/x32/syscallent.h (59): Fix comment typo. > (78): Add missing getdents entry. > (174): Delete create_module entry (not in the kernel). > (181, 182, 183, 184, 185):

Q: drop PTRACE_SEIZE_DEVEL?

2012-07-09 Thread Dmitry V. Levin
On Mon, Mar 12, 2012 at 01:26:35PM +0100, Denys Vlasenko wrote: > On 03/10/2012 11:17 PM, Dmitry V. Levin wrote: > > On Thu, Feb 09, 2012 at 07:38:27PM +0100, Denys Vlasenko wrote: > >> On 02/09/2012 05:29 PM, Dmitry V. Levin wrote: > >>> But I'd like to

Re: How to detach if a process is setuid or setgid?

2012-07-19 Thread Dmitry V. Levin
On Thu, Jul 19, 2012 at 03:31:51PM -0700, Aleatha Parker-Wood wrote: > I'm working with a modified version of strace to collect some information on > the long term behaviors of process trees and file system accesses. I've got > it logging data on a couple of different systems, some of which I do n

Re: [PATCH] x32: update {g,s}etsockopt syscall numbers

2012-08-22 Thread Dmitry V. Levin
On Wed, Aug 22, 2012 at 11:56:15AM -0400, Mike Frysinger wrote: > Starting with linux 3.6 (and backported to earlier kernels), these two > syscalls have changed numbers (moving from native to compat entry points). > Update the strace syscall list accordingly. What will happen with applications usi

Re: [PATCH] x32: update {g,s}etsockopt syscall numbers

2012-08-24 Thread Dmitry V. Levin
On Wed, Aug 22, 2012 at 02:51:33PM -0400, Mike Frysinger wrote: > On Wednesday 22 August 2012 12:56:36 Dmitry V. Levin wrote: > > On Wed, Aug 22, 2012 at 11:56:15AM -0400, Mike Frysinger wrote: > > > Starting with linux 3.6 (and backported to earlier kernels), these two > >

Re: Why "Close pipe and wait for the pipe process termination"?

2012-09-13 Thread Dmitry V. Levin
to wait for its completion, and that behavior wasn't documented. As result, strace users sometimes experienced unexpected data losses. Is there any regressions with the change? > commit cf53436f737c0273028503186fc0f837c7191085 > Author: Dmitry V. Levin > Date: Thu Jul 12 20:54:46

[RFC PATCH] Ignore fflush(3) return value

2012-09-17 Thread Dmitry V. Levin
strace used to honor fflush(3) return value in trace_syscall_entering which resulted to tracees not being PTRACE_SYSCALL'ed which in turn caused nasty hangups like this one: $ strace -o'|:' pwd |:: Broken pipe There is little strace can do in case of fflush(3) returning EOF, and hangup is certain

Re: [PATCH] x86_64/stat64: define struct stat64 as struct stat

2012-10-10 Thread Dmitry V. Levin
On Wed, Oct 10, 2012 at 12:59:43PM +0100, Markos Chandras wrote: > From: Markos Chandras > > x86_64 does not define a struct stat64 in asm/stat.h. It uses > struct stat instead. > > This fixes a compilation problem when LFS is used > > file.c:1073:16: error: storage size of ‘statbuf’ isn’t know

Re: [PATCH] x86_64/stat64: define struct stat64 as struct stat

2012-10-10 Thread Dmitry V. Levin
On Wed, Oct 10, 2012 at 01:48:02PM +0100, Markos Chandras wrote: > On Wed, Oct 10, 2012 at 1:35 PM, Dmitry V. Levin wrote: > > On Wed, Oct 10, 2012 at 12:59:43PM +0100, Markos Chandras wrote: > >> From: Markos Chandras > >> > >> x86_64 does not define a

Re: [PATCH] x86_64/stat64: define struct stat64 as struct stat

2012-10-10 Thread Dmitry V. Levin
On Wed, Oct 10, 2012 at 02:03:57PM +0100, Markos Chandras wrote: > On Wed, Oct 10, 2012 at 1:55 PM, Dmitry V. Levin wrote: > > On Wed, Oct 10, 2012 at 01:48:02PM +0100, Markos Chandras wrote: > >> On Wed, Oct 10, 2012 at 1:35 PM, Dmitry V. Levin wrote: > >> > On W

Re: [PATCH] x86_64/stat64: define struct stat64 as struct stat

2012-10-10 Thread Dmitry V. Levin
On Wed, Oct 10, 2012 at 02:19:30PM +0100, Markos Chandras wrote: > On Wed, Oct 10, 2012 at 2:09 PM, Dmitry V. Levin wrote: > > On Wed, Oct 10, 2012 at 02:03:57PM +0100, Markos Chandras wrote: > >> On Wed, Oct 10, 2012 at 1:55 PM, Dmitry V. Levin wrote: > >> > On W

Re: [RFC] Add support for tracing memory related syscalls

2012-10-18 Thread Dmitry V. Levin
Hi, On Thu, Oct 18, 2012 at 05:25:27PM +0900, Namhyung Kim wrote: > Hi, > > When I was debugging a memory related problem, I really want to see > only those syscalls but it wans't suppport by strace. Hence I decided > to create a patch. I guess others may want this too. > > This patch was made

Re: [PATCH] Add -e trace=memory option

2012-10-21 Thread Dmitry V. Levin
On Fri, Oct 19, 2012 at 10:01:09AM +0900, Namhyung Kim wrote: > Add a new 'memory' category for tracing memory related syscalls. Most of syscalls are memory related because they take an address argument, so I would rather call this subset as memory mapping related. > Affected syscalls are: brk, m

Re: [PATCH] Add -e trace=memory option

2012-10-23 Thread Dmitry V. Levin
On Tue, Oct 23, 2012 at 06:01:16PM +0900, Namhyung Kim wrote: > 2012-10-22 오전 4:25, Dmitry V. Levin 쓴 글: > > On Fri, Oct 19, 2012 at 10:01:09AM +0900, Namhyung Kim wrote: > >> Add a new 'memory' category for tracing memory related syscalls. > > > > Most of

Re: [PATCH 1/2] Update errno and ioctl definitions for newer Linux

2012-10-23 Thread Dmitry V. Levin
On Tue, Oct 23, 2012 at 05:36:58PM +0100, Steve McIntyre wrote: > Update errno and ioctl definitions for newer Linux > > Signed-off-by: Steve McIntyre > --- > linux/errnoent.h|4 +- > linux/ioctlent.h.in | 121 > +++ > 2 files changed, 12

Re: [PATCH] Add -e trace=memory option

2012-10-24 Thread Dmitry V. Levin
On Wed, Oct 24, 2012 at 10:01:36AM +0900, Namhyung Kim wrote: > 2012-10-22 오전 4:25, Dmitry V. Levin 쓴 글: > > vmsplice is not a memory mapping related. I see it was not actually > > changed by the patch. > > But vmsplice takes memory addresses through iov->iov_base. Isn

Re: [PATCH 2/2] Add AArch64 support to strace

2012-10-24 Thread Dmitry V. Levin
On Tue, Oct 23, 2012 at 05:38:22PM +0100, Steve McIntyre wrote: [...] > * syscall.c: Support AArch64. Uses new PTRACE_GETREGSET ptrace request > instead of PTRACE_GETREGS, so also include sys/uio.h and elf.h for needed > definitions. Could you please enlighten us what was the rationale behind this

Re: [PATCH v2] Add -e trace=memory option

2012-10-27 Thread Dmitry V. Levin
On Wed, Oct 24, 2012 at 11:41:57AM +0900, Namhyung Kim wrote: > Add a new 'memory' category for tracing memory mapping related syscalls. I've applied this change with minor corrections. Thanks! -- ldv pgpxNrcjMTmmT.pgp Description: PGP signature --

Re: [PATCH 1/2] Update errno and ioctl definitions for newer Linux

2012-10-27 Thread Dmitry V. Levin
On Wed, Oct 24, 2012 at 02:49:20PM +0100, Steve McIntyre wrote: > On Wed, Oct 24, 2012 at 03:58:46AM +0400, Dmitry V. Levin wrote: > >On Tue, Oct 23, 2012 at 05:36:58PM +0100, Steve McIntyre wrote: > >> Update errno and ioctl definitions for newer Linux > >> > &g

Re: [PATCH 2/2] Add AArch64 support to strace

2012-10-27 Thread Dmitry V. Levin
On Wed, Oct 24, 2012 at 05:58:16PM +0100, Steve McIntyre wrote: > On Tue, Oct 23, 2012 at 02:11:59PM -0400, Mike Frysinger wrote: > >On Tuesday 23 October 2012 12:38:22 Steve McIntyre wrote: > >> Add AArch64 support to strace [...] > New patch included here with these changes included. Applied wit

Re: [PATCH 1/2] Update errno and ioctl definitions for newer Linux

2012-10-27 Thread Dmitry V. Levin
On Tue, Oct 23, 2012 at 10:22:56PM -0400, Mike Frysinger wrote: > On Tuesday 23 October 2012 19:58:46 Dmitry V. Levin wrote: > > On Tue, Oct 23, 2012 at 05:36:58PM +0100, Steve McIntyre wrote: > > > --- a/linux/ioctlent.h.in > > > +++ b/linux/ioctl

Re: Q: drop PTRACE_SEIZE_DEVEL?

2012-10-31 Thread Dmitry V. Levin
On Tue, Jul 10, 2012 at 04:36:18PM +0200, Denys Vlasenko wrote: > On 07/10/2012 01:01 AM, Dmitry V. Levin wrote: [...] > > I suppose we can safely drop PTRACE_SEIZE_DEVEL from USE_SEIZE code now. > > I'm going to do that. Also will drop PTRACE_EVENT_STOP1 > and enable

Re: Update AArch64 strace to also support tracing 32-bit ARM binaries

2012-11-12 Thread Dmitry V. Levin
On Sat, Nov 10, 2012 at 11:24:48AM +, Steve McIntyre wrote: > On Sat, Nov 10, 2012 at 12:20:34AM -0500, Mike Frysinger wrote: > >On Thursday 08 November 2012 12:36:05 Steve McIntyre wrote: [...] > Here's the updated patch. Thanks for the quick review. Applied (with trailing whitespaces removed

Re: Bug in setbpt/clearbpt for SPARC/SPARC64?

2012-11-29 Thread Dmitry V. Levin
Hi, On Wed, Nov 28, 2012 at 01:14:33PM +, James Hogan wrote: > While implementing regset support in strace for the metag architecture, > I think I've noticed a problem in util.c. > > For SPARC/SPARC64, arg_setup and arg_finish_change are defined to use > PTRACE_GETREGS and PTRACE_SETREGS. How

Re: [PATCH 1/1] Add state argument to change_syscall and fix SPARC

2012-11-29 Thread Dmitry V. Levin
On Thu, Nov 29, 2012 at 05:37:37PM +, James Hogan wrote: > Add a state argument to change_syscall() so that SPARC can modify that > instead of read-modify-writing the whole register set. This function is > always called within an arg_setup/arg_finish_change sequence which on > certain architect

Re: [PATCH] check if __GLIBC__ is defined before using it

2012-12-03 Thread Dmitry V. Levin
On Mon, Dec 03, 2012 at 12:29:20AM +0100, John Spencer wrote: > --- a/signal.c > +++ b/signal.c > @@ -419,7 +419,7 @@ print_sigset(struct tcb *tcp, long addr, int rt) > # define SI_FROMUSER(sip)((sip)->si_code <= 0) > #endif > > -#if __GLIBC_MINOR__ < 1 > +#if defined(__GLIBC__) && __GLIBC_

Re: [PATCH] sys_semtimedop(): fix timeval argument index in wrapped call

2012-12-07 Thread Dmitry V. Levin
On Fri, Dec 07, 2012 at 09:30:51PM +0100, Stanislav Brabec wrote: > Looking at the implementation of wrapped semtimedop() call inside glibc > and kernel, I started to believe, that timeval should be located in > tcp->u_arg[4] and not tcp->u_arg[5]. Yes, in indirect_ipccall non-s390 case its canoni

Re: Strace not showing all system calls

2013-01-12 Thread Dmitry V. Levin
On Wed, Jan 09, 2013 at 11:31:57AM +0100, Frits Hoogland wrote: > Hello list, I hope you can help me with an observation I've made with strace > on Oracle Linux 6.3 (latest) X64. > When I strace a process, I only see successful io_getevents() calls, not > non-successful io_getevents() calls with

Re: improper output when executing program via strace on armhf device

2013-01-20 Thread Dmitry V. Levin
On Sun, Jan 20, 2013 at 02:53:57PM +0100, Rune Kjær Svendsen wrote: > Hi list > > I'm using strace running in Ubuntu on a Samsung ARM chromebook. It's using > the 3.4.0 kernel from here: > http://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel.git (the > one the comes with Chrome OS). >

Re: improper output when executing program via strace on armhf device

2013-01-20 Thread Dmitry V. Levin
On Sun, Jan 20, 2013 at 11:00:32PM +0100, Rune Kjær Svendsen wrote: > Using strace 4.5.20-2.3ubuntu2 from Ubuntu 12.10. I'm sorry, but strace 4.5.20 was released in April of 2010, please try something more up to date, e.g. the latest release 4.7 or strace.git. -- ldv pgpEb6oqxPoQ0.pgp Descrip

Re: [PATCH 1/2] mmap: decode MAP_UNINITIALIZE

2013-01-21 Thread Dmitry V. Levin
On Mon, Jan 21, 2013 at 02:36:55PM +0100, Bernhard Reutner-Fischer wrote: > Signed-off-by: Bernhard Reutner-Fischer > --- > mem.c |3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/mem.c b/mem.c > index 0279030..53fcf85 100644 > --- a/mem.c > +++ b/mem.c > @@ -159,6 +159,9 @@ static

Re: [PATCH 1/2] mmap: decode MAP_UNINITIALIZE

2013-01-22 Thread Dmitry V. Levin
On Tue, Jan 22, 2013 at 10:16:42AM +0100, Bernhard Reutner-Fischer wrote: > On 21 January 2013 15:23, Dmitry V. Levin wrote: > > On Mon, Jan 21, 2013 at 02:36:55PM +0100, Bernhard Reutner-Fischer wrote: > >> Signed-off-by: Bernhard Reutner-Fischer > >> --- > >

Re: [PATCH 1/2] mmap: decode MAP_UNINITIALIZE

2013-01-22 Thread Dmitry V. Levin
On Tue, Jan 22, 2013 at 06:09:50PM +0100, Bernhard Reutner-Fischer wrote: > On 22 January 2013 10:35, Dmitry V. Levin wrote: > > On Tue, Jan 22, 2013 at 10:16:42AM +0100, Bernhard Reutner-Fischer wrote: > >> On 21 January 2013 15:23, Dmitry V. Levin wrote: > >> >

Re: [PATCH 1/2] mmap: decode MAP_UNINITIALIZE

2013-01-22 Thread Dmitry V. Levin
On Tue, Jan 22, 2013 at 01:10:13PM -0500, Mike Frysinger wrote: > On Tuesday 22 January 2013 04:16:42 Bernhard Reutner-Fischer wrote: > > On 21 January 2013 15:23, Dmitry V. Levin wrote: > > > On Mon, Jan 21, 2013 at 02:36:55PM +0100, Bernhard Reutner-Fischer wrote:

Re: improper output when executing program via strace on armhf device

2013-01-22 Thread Dmitry V. Levin
On Tue, Jan 22, 2013 at 07:45:59PM +0100, Rune Kjær Svendsen wrote: > Just built 4.7 and it works like a charm. I've reported the following bug > in Ubuntu to get them to pull in stace 4.7 for Raring. So if anyone else is > experiencing this bug in Ubuntu, then please mark the below bug as > affect

Re: [PATCH] Add tilegx support to strace

2013-02-05 Thread Dmitry V. Levin
On Tue, Feb 05, 2013 at 11:16:31AM -0500, Chris Metcalf wrote: > Ping? > > Would it make sense for me to be able to directly commit tile-specific > changes to strace? The previous tile support for strace was committed by an > strace maintainer on my behalf (commit c8c6698ef7cde in Dec 2009). S

Re: [PATCHv2 1/2] mmap: decode MAP_UNINITIALIZED

2013-02-05 Thread Dmitry V. Levin
On Tue, Feb 05, 2013 at 07:31:55PM +0100, Bernhard Reutner-Fischer wrote: > mem.c |3 +++ On Tue, Feb 05, 2013 at 07:31:56PM +0100, Bernhard Reutner-Fischer wrote: > system.c |2 ++ Thanks, I've added changelog entries and applied them. -- ldv pgpB64z6BJzsr.pgp Description: PGP signat

Re: [PATCH] Improve perf_event_open argument decoding

2013-02-05 Thread Dmitry V. Levin
On Mon, Feb 04, 2013 at 12:04:57AM +0100, Ben Noordhuis wrote: > * configure.ac: Add check. > * desc.c: Add sys_perf_event_open I've added missing bits to the changelog entry, fixed a typo and applied this. Thanks, -- ldv pgpSsgHmwJdQW.pgp Description: PGP signature

Re: [PATCH] tile: Fix merge skew with new get_regs architecture

2013-02-05 Thread Dmitry V. Levin
On Tue, Feb 05, 2013 at 01:02:42PM -0500, Chris Metcalf wrote: > Also fix a compiler warning about pt_reg_t in a printf expression. This time it was me who made this patch harder to merge, so I've adjusted it a bit, added a changelog entry and applied. Thanks, -- ldv pgpIWK5RQQ2sr.pgp Descri

Re: [PATCH] Improve perf_event_open argument decoding

2013-02-05 Thread Dmitry V. Levin
On Wed, Feb 06, 2013 at 12:57:34AM +0100, Ben Noordhuis wrote: > On Wed, Feb 6, 2013 at 12:31 AM, Dmitry V. Levin wrote: > > On Mon, Feb 04, 2013 at 12:04:57AM +0100, Ben Noordhuis wrote: > >> * configure.ac: Add check. > >> * desc.c: Add sys_perf_event_open > &g

Re: Q: drop PTRACE_SEIZE_DEVEL?

2013-02-07 Thread Dmitry V. Levin
On Wed, Feb 06, 2013 at 01:57:46PM +0100, Denys Vlasenko wrote: > On 10/31/2012 04:37 PM, Dmitry V. Levin wrote: > > It seems to be working so far, but with annoying addition to the strace > > output: > > > > $ strace true > > --- stopped by SIGSTOP --- > >

Re: [PATCH] if we are on a glibc system, assume it's at least glibc 2.1

2013-02-08 Thread Dmitry V. Levin
On Fri, Feb 08, 2013 at 03:22:09PM +0100, Denys Vlasenko wrote: > Hi Dmitry, > > glibc 2.1.1 was released in 1999. > > Is this patch ok with you? Yes. -- ldv pgptQKFXjn4MZ.pgp Description: PGP signature -- Free Next

Re: [PATCH] fix decoding of sysctl() when oldval fields are NULL

2013-02-08 Thread Dmitry V. Levin
Applied, thanks. -- ldv pgppapJLMDS8T.pgp Description: PGP signature -- Free Next-Gen Firewall Hardware Offer Buy your Sophos next-gen firewall before the end March 2013 and get the hardware for free! Learn more. http

Re: strace hangs on sigaltstack system call with invalid arguments (hacky patch attached)

2013-02-08 Thread Dmitry V. Levin
Hi, On Fri, Feb 08, 2013 at 06:07:59PM -0700, kawil...@ucalgary.ca wrote: > I've been using strace for a while now, and just found what I would > consider to be a bug in strace. Essentially, if a traced program executes > a particular system call (SYS_sigaltstack) with `invalid' arguments (which >

Re: Suggestion for patch

2013-02-08 Thread Dmitry V. Levin
Hi, On Thu, Feb 07, 2013 at 03:15:17PM +, Frediano Ziglio wrote: > I was trying to add support to Xen ioctl to strace. To distinguish > from other ioctl I used the entire code (not only type and number but > also size and direction) so I changed sys_ioctl in io.c to call a > function that ch

Re: [PATCH] new x86 personality detection

2013-02-11 Thread Dmitry V. Levin
On Mon, Feb 11, 2013 at 12:45:48PM +0100, Denys Vlasenko wrote: > Hi, > > Recent addition of AArch64 used a novel way of detecting > 32/64-bitness of the process by looking at the returned size > of ptrace(PTRACE_GETREGSET, NT_PRSTATUS). > > I played with it on x86 and it appears to be working th

Re: [PATCH] new x86 personality detection

2013-02-11 Thread Dmitry V. Levin
On Mon, Feb 11, 2013 at 03:12:23PM +0100, Denys Vlasenko wrote: > On 02/11/2013 01:46 PM, Denys Vlasenko wrote: > > On 02/11/2013 12:45 PM, Denys Vlasenko wrote: > >> This patch implements a (hopefully) correct way to check for > >> syscall bitness on x86. > >> > >> I tested it to work when stracin

<    3   4   5   6   7   8   9   10   11   12   >