Q: alternative hosting for strace-devel mailing list

2015-02-25 Thread Dmitry V. Levin
Hi, On Tue, Feb 24, 2015 at 07:51:52AM +0300, Dmitry V. Levin wrote: I've configured strace.git mirrors where I push updates when git.code.sf.net/p/strace/code.git is not available: https://github.com/ldv-alt/strace.git git://git.altlinux.org/people/ldv/public/strace.git This and other

Re: [PATCH v6] Add decoding for evdev ioctls.

2015-02-21 Thread Dmitry V. Levin
On Fri, Feb 20, 2015 at 05:14:10PM +0100, Etienne Gemsa wrote: This patch introduces decoding for evdev ioctls. For EVIOCSFF the five first members are decoded, the union is not. The code was compiled and tested on x86_64 with Linux 3.13. I've applied trivial fixes, merged it, and added 3 more

Re: [PATCH v5] Add decoding for evdev ioctls.

2015-02-20 Thread Dmitry V. Levin
On Thu, Feb 19, 2015 at 05:11:20PM +0100, Etienne Gemsa wrote: This patch introduces decoding for evdev ioctls. For EVIOCSFF the five first members are decoded, the union is not. The code was compiled and tested on x86_64 with Linux 3.13. The patch is getting better. I hope v6 will be good

Re: [PATCH] ia64: fix up builds after 16bit uuid support

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 02:15:33AM -0500, Mike Frysinger wrote: --- a/linux/ia64/syscallent.h +++ b/linux/ia64/syscallent.h @@ -26,6 +26,9 @@ * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ +/* Undo any defines that an earlier dummy.h inclusion setup. */

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 01:10:39AM -0500, Mike Frysinger wrote: a few signed warnings have crept in: socketutils.c: In function 'receive_responses': socketutils.c:145:110: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] netlink_inet_diag.c: In

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 01:10:39AM -0500, Mike Frysinger wrote: vFAIL: build; x86_64/32-bit/LSB linux-3.18.4 kernel-headers-3.19.0 glibc-2.21 gcc-4.9.2 The build was broken; hopefully fixed by commit 19fcb27764f1b6040d7c50739a26f401aa576f38. -- ldv pgpn45h8B0dyi.pgp Description: PGP

Re: [PATCH] hppa: fix fanotify_mark decoding

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 02:16:13PM -0500, Mike Frysinger wrote: The parisc kernel has some legacy baggage here and decodes the 64bit field in the reverse order. Handle it in strace too. * fanotify.c [HPPA] (sys_fanotify_mark): Reverse the mask bits. --- fanotify.c | 4 1 file

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 01:10:39AM -0500, Mike Frysinger wrote: vFAIL: test; i686/32-bit/LSB linux-3.18.4 kernel-headers-3.3.0 glibc-2.17 gcc-4.7.2 I'd like to know how to reproduce this. -- ldv pgpdMcNKgs_2V.pgp Description: PGP signature

Re: [PATCH] hppa: fix fanotify_mark decoding

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 05:21:10PM -0500, Mike Frysinger wrote: On 27 Feb 2015 00:36, Dmitry V. Levin wrote: BTW, syscall names defined in linux/hppa/syscallent.h are out of sync with arch/parisc/include/uapi/asm/unistd.h; this is why stat32-v.test fails there. And it's not a recent

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 01:10:39AM -0500, Mike Frysinger wrote: vFAIL: test; ppc64/64-bit/MSB linux-3.12.20-gentoo kernel-headers-3.9.0 glibc-2.17 gcc-4.7.3 This is a test issue; hopefully fixed by commits 977550de794baac058d4368682167fcbebbdf653 and 856b7596b83d70d6e7e27eef56fa7b92a0dac4af.

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 11:31:59PM -0500, Mike Frysinger wrote: On 27 Feb 2015 05:30, Dmitry V. Levin wrote: On Thu, Feb 26, 2015 at 05:56:43PM -0500, Mike Frysinger wrote: detach-stopped.test.tmp contains just this: Process 1267 attached restart_syscall(... resuming interrupted call

Re: Difficulty in Understanding the Strace code base

2015-02-26 Thread Dmitry V. Levin
Hi, On Thu, Feb 26, 2015 at 11:26:34PM +0100, Alangi Derick wrote: Hello, I am a new comer to strace and i have been looking at the source codes(code base) for the entire utility. But since the day i started, it has not been easy for me to understand. I have installed it in my system

Re: Preparing for the next release: call for testing

2015-02-26 Thread Dmitry V. Levin
On Thu, Feb 26, 2015 at 05:56:43PM -0500, Mike Frysinger wrote: On 27 Feb 2015 01:23, Dmitry V. Levin wrote: On Thu, Feb 26, 2015 at 01:10:39AM -0500, Mike Frysinger wrote: vFAIL: test; i686/32-bit/LSB linux-3.18.4 kernel-headers-3.3.0 glibc-2.17 gcc-4.7.2 I'd like to know how

Re: GSoC 2015

2015-03-24 Thread Dmitry V. Levin
the current implementation (which I labelled inherently unreliable) as a GSoC project? P.S. Top posting was not a good idea. On Sun, Mar 22, 2015 at 5:11 AM, Dmitry V. Levin wrote: On Sun, Mar 22, 2015 at 12:34:55AM +0530, Prateek Gupta wrote: As far as I percieve, the multiarchitecture

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-03-24 Thread Dmitry V. Levin
On Tue, Mar 24, 2015 at 04:22:43PM +0100, Denys Vlasenko wrote: On 03/24/2015 01:49 AM, Dmitry V. Levin wrote: On Mon, Mar 23, 2015 at 05:50:39AM +0300, Dmitry V. Levin wrote: On Sat, Mar 21, 2015 at 05:06:32PM +, Denys Vlasenko wrote: [...] commit

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-03-24 Thread Dmitry V. Levin
On Tue, Mar 24, 2015 at 11:30:39PM +0300, Dmitry V. Levin wrote: Any ideas why on ALPHA and MIPS scno == 516, and on S390/S390X scno == -516 instead of 0 when attaching to a sleeping process? I think I can answer this myself: because these architectures (also m68k and or1k) use the same

Re: GSOC2015 - Suggestions/Criticism - Some Test Scripts for Basic System Calls

2015-03-17 Thread Dmitry V. Levin
On Sat, Mar 14, 2015 at 10:25:48PM +0530, Rohan Bavishi wrote: Hi everyone, As I had mentioned in the thread before, I had planned to write some small test-suite code in order to get familiar with the code base. I have come up with some small preliminary test-benches covering system-calls

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-03-23 Thread Dmitry V. Levin
On Mon, Mar 23, 2015 at 05:50:39AM +0300, Dmitry V. Levin wrote: On Sat, Mar 21, 2015 at 05:06:32PM +, Denys Vlasenko wrote: [...] commit 8497b6222ed8ef606996d0ceb2bae260d82f95e2 Author: Denys Vlasenko dvlas...@redhat.com Date: Sat Mar 21 17:51:52 2015 +0100 Show the syscall

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-03-23 Thread Dmitry V. Levin
On Tue, Mar 24, 2015 at 03:49:10AM +0300, Dmitry V. Levin wrote: On Mon, Mar 23, 2015 at 05:50:39AM +0300, Dmitry V. Levin wrote: On Sat, Mar 21, 2015 at 05:06:32PM +, Denys Vlasenko wrote: [...] commit 8497b6222ed8ef606996d0ceb2bae260d82f95e2 Author: Denys Vlasenko dvlas

Re: [PATCH] Add support for Altera's Nios-II softcore architecture

2015-04-20 Thread Dmitry V. Levin
On Sun, Apr 19, 2015 at 03:36:31PM -0300, Ezequiel Garcia wrote: On 04/18/2015 06:56 PM, Dmitry V. Levin wrote: On Sat, Apr 18, 2015 at 05:46:19PM -0300, Ezequiel Garcia wrote: On 04/18/2015 05:33 PM, Ezequiel Garcia wrote: This commit adds strace support for Altera's Nios-II

Re: [Intel-gfx] [RFC 0/2] strace/drm: Add i915 ioctls to strace

2015-05-12 Thread Dmitry V. Levin
On Tue, May 12, 2015 at 07:37:59PM +0200, Gabriel Laskar wrote: On Tue, 12 May 2015 14:35:28 +0200, Patrik Jakobsson wrote: On Mon, May 11, 2015 at 08:08:19PM +0200, Gabriel Laskar wrote: On Mon, 11 May 2015 15:54:24 +0200, Patrik Jakobsson wrote: On Mon, May 11, 2015 at 12:50:36PM

Re: strace 4.10 doesn't build on MIPS arch

2015-04-15 Thread Dmitry V. Levin
On Wed, Apr 15, 2015 at 11:26:08AM +0200, Arturo Borrero Gonzalez wrote: On 14 April 2015 at 15:13, Dmitry V. Levin l...@altlinux.org wrote: On Tue, Apr 14, 2015 at 09:32:39AM +0200, Arturo Borrero Gonzalez wrote: Hi there! I've been taking a look at the debian log for strace on the MIPS

Re: Fwd: GSOC 2015 : Structured Output

2015-04-15 Thread Dmitry V. Levin
Hi, On Wed, Apr 15, 2015 at 12:46:32AM +0530, Amish Mittal wrote: i grep the code of strace 4.10 in and out for this project. This is what all i got for this project: we don't need to change the tprintf and tprints calls, we only need to slightly change their definition. My idea for this

Re: [PATCH] v4l2.c: add decoding for VIDIOC_CREATE_BUFS's arg.

2015-04-16 Thread Dmitry V. Levin
On Thu, Apr 16, 2015 at 03:33:34PM +0200, Philippe De Muyter wrote: * v4l2.c (v4l2_ioctl): Add decoding for VIDIOC_CREATE_BUFS's arg. --- v4l2.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/v4l2.c b/v4l2.c index ae1ebb8..c121c61 100644 --- a/v4l2.c +++

Re: strace 4.10 doesn't build on MIPS arch

2015-04-17 Thread Dmitry V. Levin
On Fri, Apr 17, 2015 at 11:06:45AM +0200, Arturo Borrero Gonzalez wrote: On 15 April 2015 at 15:52, Dmitry V. Levin l...@altlinux.org wrote: On Wed, Apr 15, 2015 at 11:26:08AM +0200, Arturo Borrero Gonzalez wrote: On 14 April 2015 at 15:13, Dmitry V. Levin l...@altlinux.org wrote: On Tue

Re: strace 4.10 doesn't build on MIPS arch

2015-04-17 Thread Dmitry V. Levin
On Fri, Apr 17, 2015 at 12:42:55PM +0200, Arturo Borrero Gonzalez wrote: On 17 April 2015 at 11:23, Dmitry V. Levin l...@altlinux.org wrote: ../syscall.c:1798:3: error: too few arguments to function 'decode_mips_subcall' decode_mips_subcall(); That's correct, it has

Re: [PATCH] v4l2.c: add decoding for VIDIOC_CREATE_BUFS's arg.

2015-04-17 Thread Dmitry V. Levin
On Fri, Apr 17, 2015 at 12:47:10PM +0200, Philippe De Muyter wrote: On Thu, Apr 16, 2015 at 09:56:30PM +0300, Dmitry V. Levin wrote: On Thu, Apr 16, 2015 at 03:33:34PM +0200, Philippe De Muyter wrote: + else { + static char fmt[] = {index=%u, count=%u

Re: [PATCH] Add support for Altera's Nios-II softcore architecture

2015-04-18 Thread Dmitry V. Levin
On Sat, Apr 18, 2015 at 05:46:19PM -0300, Ezequiel Garcia wrote: On 04/18/2015 05:33 PM, Ezequiel Garcia wrote: This commit adds strace support for Altera's Nios-II. The architecture is supported by Linux since v3.19, and it implements the generic syscall ABI. * Makefile.am,

Re: strace 4.10 doesn't build on MIPS arch

2015-04-17 Thread Dmitry V. Levin
On Fri, Apr 17, 2015 at 02:39:45PM +0200, Arturo Borrero Gonzalez wrote: Find attached the log for this patch. Still errors :-( The patch seems to work, so I've merged it to HEAD, thus fixing the last of remaining FTBFS issues mentioned in Debian bug #781040. The log shows that it failed on a

Re: strace 4.10 doesn't build on MIPS arch

2015-04-14 Thread Dmitry V. Levin
Hi, On Tue, Apr 14, 2015 at 09:32:39AM +0200, Arturo Borrero Gonzalez wrote: Hi there! I've been taking a look at the debian log for strace on the MIPS architecture [0]. There is a test that doesn't pass and therefore the build is taken as invalid (and the debian package isn't created).

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-04-06 Thread Dmitry V. Levin
On Tue, Mar 24, 2015 at 05:13:13AM +0300, Dmitry V. Levin wrote: It also doesn't work on alpha and s390/s390x: $ sleep 2 sleep 1; ./strace -erestart_syscall -p $! [1] 12345 Process 12345 attached restart_syscall(... resuming interrupted unknown ...) = 0 +++ exited with 0 +++ [1]+ Done

Q: bugfix release

2015-04-05 Thread Dmitry V. Levin
Hi, I've been asked at [1] to update strace package for post-4.10 aarch64 fixes. As current HEAD contains several critical fixes for aarch64 and there are no known regressions, I think it's a good idea to cut a bugfix release. Could you check HEAD for regressions, please? [1]

Re: [PATCH] Introduce xmalloc, memory allocator with die_out_of_memory()

2015-04-06 Thread Dmitry V. Levin
On Tue, Apr 07, 2015 at 12:40:35AM +0900, Masatake YAMATO wrote: --- a/defs.h +++ b/defs.h @@ -434,6 +434,15 @@ void perror_msg_and_die(const char *fmt, ...) ATTRIBUTE_FORMAT((printf, 1, 2)) ATTRIBUTE_NORETURN; void die_out_of_memory(void) ATTRIBUTE_NORETURN; +/* + * Memory

Re: [SCM] strace branch, master, updated. v4.10-25-g8497b62

2015-04-07 Thread Dmitry V. Levin
On Tue, Apr 07, 2015 at 12:48:45PM +0200, Denys Vlasenko wrote: On 04/07/2015 02:08 AM, Dmitry V. Levin wrote: On Tue, Mar 24, 2015 at 05:13:13AM +0300, Dmitry V. Levin wrote: It also doesn't work on alpha and s390/s390x: $ sleep 2 sleep 1; ./strace -erestart_syscall -p $! [1] 12345

Re: [PATCH v2] v4l2: VIDIOC_CROPCAP: add missing braces around arg printout.

2015-05-19 Thread Dmitry V. Levin
On Tue, May 19, 2015 at 11:54:21PM +0200, Philippe De Muyter wrote: * v4l2.c (v4l2_ioctl): case VIDIOC_CROPCAP: add missing braces around arg printout. Thanks, merged. -- ldv pgp0kH5ehP1ac.pgp Description: PGP signature

Re: [PATCH] maint/ioctls_gen.sh: avoid effects of locale when sorting files

2015-05-19 Thread Dmitry V. Levin
On Wed, Apr 22, 2015 at 10:52:09AM +0200, Philippe De Muyter wrote: Signed-off-by: Philippe De Muyter p...@macqel.be --- maint/ioctls_gen.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/maint/ioctls_gen.sh b/maint/ioctls_gen.sh index d8da875..56c03d8 100755 ---

Re: [PATCH] Add support for Altera's Nios-II softcore architecture

2015-05-19 Thread Dmitry V. Levin
On Sat, Apr 25, 2015 at 05:29:36PM -0300, Ezequiel Garcia wrote: [...] In any case, all these issues are unrelated to the strace Nios-II port. Quite the opposite, I'd say the strace port will be useful to spot these kind of issues. Thanks, I've merged the patch. -- ldv pgpCljW52Qg7o.pgp

Re: [PATCH 3/4] drm: Add decoding of i915 ioctls

2015-06-09 Thread Dmitry V. Levin
On Tue, Jun 09, 2015 at 01:26:43PM +0200, Patrik Jakobsson wrote: [...] +static int i915_getparam(struct tcb *tcp, const unsigned int code, long arg) +{ + struct drm_i915_getparam param; + int value; + + if (entering(tcp) || umove(tcp, arg, param)) + return 0; +

Re: [PATCH 4/4] drm: Add decoding of DRM and KMS ioctls

2015-06-09 Thread Dmitry V. Levin
On Tue, Jun 09, 2015 at 01:26:44PM +0200, Patrik Jakobsson wrote: [...] +static int drm_version(struct tcb *tcp, const unsigned int code, long arg) +{ + struct drm_version ver; + char *name, *date, *desc; + int ret; + + if (entering(tcp) || umove(tcp, arg, ver)) +

Re: [PATCH 4/4] drm: Add decoding of DRM and KMS ioctls

2015-06-09 Thread Dmitry V. Levin
On Tue, Jun 09, 2015 at 04:38:40PM +0200, Gabriel Laskar wrote: On Tue, 9 Jun 2015 16:29:31 +0200 Patrik Jakobsson patrik.jakobs...@linux.intel.com wrote: On Tue, Jun 09, 2015 at 03:51:08PM +0200, Gabriel Laskar wrote: On Tue, 9 Jun 2015 13:26:44 +0200 Patrik Jakobsson

Re: [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-09 Thread Dmitry V. Levin
On Tue, Jun 09, 2015 at 01:26:42PM +0200, Patrik Jakobsson wrote: [...] +static int drm_get_driver_name(struct tcb *tcp, char *name, size_t bufsize) +{ + char path[PATH_MAX]; + char link[PATH_MAX]; + int ret; + + ret = getfdpath(tcp, tcp-u_arg[0], path, PATH_MAX - 1); +

Re: [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-09 Thread Dmitry V. Levin
On Tue, Jun 09, 2015 at 01:26:42PM +0200, Patrik Jakobsson wrote: [...] --- a/Makefile.am +++ b/Makefile.am @@ -121,6 +121,7 @@ strace_SOURCES = \ utime.c \ utimes.c\ v4l2.c \ + drm.c \ vsprintf.c \ wait.c

Re: [Intel-gfx] [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-14 Thread Dmitry V. Levin
On Sun, Jun 14, 2015 at 01:12:45PM +0200, Patrik Jakobsson wrote: [...] How about adding a void *private field to struct tcb. That way any syscall can store additional data across the life of the tcb. We can add a field to struct tcb, but its semantics wrt memory management should be strictly

Re: [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-12 Thread Dmitry V. Levin
On Thu, Jun 11, 2015 at 04:11:49PM +0200, Patrik Jakobsson wrote: On Thu, Jun 11, 2015 at 02:26:59AM +0300, Dmitry V. Levin wrote: On Wed, Jun 10, 2015 at 01:52:33PM +0200, Patrik Jakobsson wrote: On Wed, Jun 10, 2015 at 01:14:20AM +0300, Dmitry V. Levin wrote: On Tue, Jun 09, 2015 at 01

Re: [Intel-gfx] [PATCH 3/4] drm: Add decoding of i915 ioctls

2015-06-12 Thread Dmitry V. Levin
On Thu, Jun 11, 2015 at 03:34:14PM +0200, Patrik Jakobsson wrote: On Thu, Jun 11, 2015 at 02:27:12AM +0300, Dmitry V. Levin wrote: On Wed, Jun 10, 2015 at 02:45:24PM +0200, Patrik Jakobsson wrote: On Wed, Jun 10, 2015 at 01:35:35AM +0300, Dmitry V. Levin wrote: On Tue, Jun 09, 2015 at 01

Re: [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-10 Thread Dmitry V. Levin
On Wed, Jun 10, 2015 at 01:52:33PM +0200, Patrik Jakobsson wrote: On Wed, Jun 10, 2015 at 01:14:20AM +0300, Dmitry V. Levin wrote: On Tue, Jun 09, 2015 at 01:26:42PM +0200, Patrik Jakobsson wrote: [...] +#define DRM_MAX_NAME_LEN 128 + +inline int drm_is_priv(const unsigned int num

Re: [PATCH 3/4] drm: Add decoding of i915 ioctls

2015-06-10 Thread Dmitry V. Levin
On Wed, Jun 10, 2015 at 02:45:24PM +0200, Patrik Jakobsson wrote: On Wed, Jun 10, 2015 at 01:35:35AM +0300, Dmitry V. Levin wrote: On Tue, Jun 09, 2015 at 01:26:43PM +0200, Patrik Jakobsson wrote: [...] +static int i915_setparam(struct tcb *tcp, const unsigned int code, long arg

Re: [PATCH v3 2/5] drm: Add private data field to trace control block

2015-07-02 Thread Dmitry V. Levin
On Wed, Jul 01, 2015 at 02:52:45PM +0200, Patrik Jakobsson wrote: [...] --- a/defs.h +++ b/defs.h @@ -266,6 +266,13 @@ struct tcb { int u_error;/* Error code */ long scno; /* System call number */ long u_arg[MAX_ARGS]; /* System call arguments */

Re: [PATCH] v4l2: add decoding for VIDIOC_S_CROP's arg.

2015-05-21 Thread Dmitry V. Levin
On Wed, May 20, 2015 at 05:50:21PM +0200, Philippe De Muyter wrote: * v4l2.c (v4l2_ioctl): Add decoding for VIDIOC_S_CROP's arg. Merged, thanks. -- ldv pgpraiugPAm3z.pgp Description: PGP signature -- One dashboard

Re: [PATCH] v4l2: Improve decoding of VIDIOC_QBUF's arg.

2015-05-22 Thread Dmitry V. Levin
On Fri, May 22, 2015 at 09:22:22AM +0200, Philippe De Muyter wrote: --- a/v4l2.c +++ b/v4l2.c @@ -646,6 +646,10 @@ v4l2_ioctl(struct tcb *tcp, const unsigned int code, long arg) case VIDIOC_DQBUF: { struct v4l2_buffer b; + if (exiting(tcp) syserror(tcp))

Re: [PATCH] v4l2: VIDIOC_CROPCAP: add missing braces around arg printout.

2015-05-19 Thread Dmitry V. Levin
On Sun, Apr 26, 2015 at 03:52:48PM +0200, Philippe De Muyter wrote: --- a/v4l2.c +++ b/v4l2.c @@ -558,12 +558,12 @@ v4l2_ioctl(struct tcb *tcp, const unsigned int code, long arg) if (entering(tcp) || umove(tcp, arg, c) 0) return 0; -

Re: [PATCH 1/2] Decode setsockopt() multicast arguments

2015-05-26 Thread Dmitry V. Levin
On Sat, Feb 07, 2015 at 08:17:46PM +0300, Dmitry V. Levin wrote: On Sat, Feb 07, 2015 at 06:05:56PM +0100, Ben Noordhuis wrote: On Sat, Feb 7, 2015 at 12:55 AM, Dmitry V. Levin wrote: On Thu, Feb 05, 2015 at 07:28:45PM +0100, Ben Noordhuis wrote: +#ifdef IP_ADD_MEMBERSHIP +static void

Re: [PATCH] signal: SIGSYS: decode si_syscall si_arch fields

2015-08-18 Thread Dmitry V. Levin
On Tue, Aug 18, 2015 at 03:24:59AM -0400, Mike Frysinger wrote: When receiving SIGSYS, the si_syscall si_arch fields are set to known values, so make sure we decode their values into the symbol settings. This makes stracing seccomp failures much easier. Applied, thanks. -- ldv

Re: compile error for MIPS

2015-08-18 Thread Dmitry V. Levin
Hi, On Tue, Aug 18, 2015 at 10:15:14AM +0200, Thomas Schmiedl wrote: Hello, I tried to cross-compile strace 4.10 for MIPS in this buid-environment - freetz.org. configure stopped with this error: checking for supported architecture... mips checking for MIPS ABI... unknown configure:

Re: compile error for MIPS

2015-08-19 Thread Dmitry V. Levin
On Tue, Aug 18, 2015 at 01:55:09PM +0200, Thomas Schmiedl wrote: Hello Dmitry, could you explain, how to do this (I'm not a developer). I only imported a preconfigured VM in VirtualBox (http://sourceforge.net/projects/freetz-linux/) and build using this tutorial

Re: [PATCH/RFC] decode extend getsockopt/setsockopt options

2015-08-19 Thread Dmitry V. Levin
On Tue, Aug 18, 2015 at 05:03:20PM -0400, Mike Frysinger wrote: [...] * util.c (printxval): Rename to ... (printxvals): ... this. Rewrite to be varargs based. * xlat/getsockipoptions.in: New xlat list. * xlat/getsockipv6options.in, xlat/setsockipoptions.in, xlat/setsockipv6options.in:

Re: [PATCH/RFC] decode extend getsockopt/setsockopt options

2015-08-19 Thread Dmitry V. Levin
On Wed, Aug 19, 2015 at 01:26:56PM -0400, Mike Frysinger wrote: On 19 Aug 2015 18:56, Dmitry V. Levin wrote: On Tue, Aug 18, 2015 at 05:03:20PM -0400, Mike Frysinger wrote: [...] * util.c (printxval): Rename to ... (printxvals): ... this. Rewrite to be varargs based. * xlat

Re: decoding of input/output args ?

2015-08-19 Thread Dmitry V. Levin
On Wed, Aug 19, 2015 at 04:59:38PM -0400, Mike Frysinger wrote: On 19 Aug 2015 22:36, Dmitry V. Levin wrote: On Wed, Aug 19, 2015 at 03:17:39PM -0400, Mike Frysinger wrote: some syscalls have arguments that are read/write. for example, getsockopt passes in a pointer to a length

Re: [PATCH v3] decode extend getsockopt/setsockopt options

2015-08-20 Thread Dmitry V. Levin
On Wed, Aug 19, 2015 at 01:29:27PM -0400, Mike Frysinger wrote: Currently the code assumes the set of valid options between getsockopt and setsockopt are exactly the same and thus maintains one list. The kernel unfortunately does not do this -- it allows for different opts between the get and

Re: mips build broken?

2015-07-29 Thread Dmitry V. Levin
On Wed, Jul 29, 2015 at 11:20:02AM -0700, enh wrote: it looks like linux/mips/genstub.sh wasn't updated for the new SEN stuff? Yes, we must have missed it. Fixed now, thanks. do we even need the -stub.h files now? does the SEN stuff make that obsolete? This new SEN stuff was introduced to

Re: [Intel-gfx] [PATCH v3 1/5] drm: Add config for detecting libdrm

2015-08-01 Thread Dmitry V. Levin
On Fri, Jul 31, 2015 at 11:09:11AM +0200, Patrik Jakobsson wrote: On Thu, Jul 30, 2015 at 10:04:49AM -0400, Mike Frysinger wrote: On 30 Jul 2015 15:30, Patrik Jakobsson wrote: On Thu, Jul 23, 2015 at 05:48:21AM -0400, Mike Frysinger wrote: On 01 Jul 2015 14:52, Patrik Jakobsson wrote:

Re: [PATCH] device mapper support for strace

2015-08-01 Thread Dmitry V. Levin
On Fri, Jul 31, 2015 at 12:04:48PM -0400, Mikulas Patocka wrote: On Fri, 31 Jul 2015, Dmitry V. Levin wrote: On Fri, Jul 31, 2015 at 10:49:44AM -0400, Mikulas Patocka wrote: Hi Here I'm sending a patch that makes it possible to decode device mapper ioctls in strace. How

Re: [PATCH] device mapper support for strace

2015-07-31 Thread Dmitry V. Levin
Hi, On Fri, Jul 31, 2015 at 10:49:44AM -0400, Mikulas Patocka wrote: Hi Here I'm sending a patch that makes it possible to decode device mapper ioctls in strace. How to apply the patch: Download strace 4.10 from

Re: [PATCH] fix builds where HAVE_MQUEUE_H isn't defined

2015-07-28 Thread Dmitry V. Levin
On Mon, Jul 27, 2015 at 05:49:48PM -0700, enh wrote: Android doesn't have mqueue.h. diff --git a/ipc.c b/ipc.c index 33f4872..52a1d7a 100644 --- a/ipc.c +++ b/ipc.c @@ -342,7 +342,7 @@ SYS_FUNC(mq_open) /* mode */ tprintf(, %#lo, , tcp-u_arg[2]); # ifndef

Re: [PATCH v3 1/5] drm: Add config for detecting libdrm

2015-07-23 Thread Dmitry V. Levin
On Thu, Jul 23, 2015 at 05:48:21AM -0400, Mike Frysinger wrote: On 01 Jul 2015 14:52, Patrik Jakobsson wrote: Use pkg-config to try to find libdrm. If that fails use the standard include directory for kernel drm headers in /usr/include/drm. * configure.ac: Use pkg-config to find libdrm

Re: Proliferation of hundreds of tiny files

2015-10-23 Thread Dmitry V. Levin
On Fri, Oct 23, 2015 at 09:53:02AM +0200, Denys Vlasenko wrote: > On 10/23/2015 08:32 AM, Dmitry V. Levin wrote: > > Hi, > > > > On Thu, Oct 22, 2015 at 05:27:24PM +0200, Denys Vlasenko wrote: > >> Hello Dmitry, > >> > >> Two years ago, at

Re: ambiguous m32_printer_defs.h rules

2015-11-08 Thread Dmitry V. Levin
On Sun, Nov 08, 2015 at 03:38:20AM +0300, Line Printer wrote: > On Sun, Nov 8, 2015 at 2:39 AM, Dmitry V. Levin <l...@altlinux.org> wrote: > > On Sat, Oct 31, 2015 at 05:20:16PM +0300, Dmitry V. Levin wrote: > >> On Tue, Oct 27, 2015 at 04:15:57PM -0700, enh w

Re: ambiguous m32_printer_defs.h rules

2015-11-07 Thread Dmitry V. Levin
On Sat, Oct 31, 2015 at 05:20:16PM +0300, Dmitry V. Levin wrote: > On Tue, Oct 27, 2015 at 04:15:57PM -0700, enh wrote: > > ./bootstrap && ./configure && make is failing for me on x86-64: > > > > In file included from m32_printer_defs.h:51:0, &g

GSoC 2015 Mentor Summit this weekend

2015-11-06 Thread Dmitry V. Levin
Hi, I'm going to take part in the GSoC 2015 Mentor Summit this weekend. There will be a lot of opportunities to meet and discuss any topics, related to strace or not. Please let me know if you are interested. -- ldv pgpbThVSzcU2T.pgp Description: PGP signature

Re: [PATCH] support C libs w/out System V shared memory/ipc

2015-10-31 Thread Dmitry V. Levin
Hi, On Sat, Oct 31, 2015 at 12:47:53AM -0400, Mike Frysinger wrote: > diff --git a/ipc_msg.c b/ipc_msg.c > index 55747cb..2fc7470 100644 > --- a/ipc_msg.c > +++ b/ipc_msg.c > @@ -31,9 +31,7 @@ > */ > > #include "defs.h" > - > -#include > -#include > +#include "ipc_defs.h" > > #include

Re: ambiguous m32_printer_defs.h rules

2015-10-31 Thread Dmitry V. Levin
On Tue, Oct 27, 2015 at 04:15:57PM -0700, enh wrote: > ./bootstrap && ./configure && make is failing for me on x86-64: > > In file included from m32_printer_defs.h:51:0, > from syscall.c:155: > m32_printer_decls.h:2:1: error: expected expression before ‘extern’ > extern bool

Re: uio.test failure

2015-10-08 Thread Dmitry V. Levin
On Wed, Oct 07, 2015 at 06:10:26PM -0400, Lennart Sorensen wrote: > I have been seeing uio.test failing consistently in one build scenario > and finally tracked down the problem. > > My build has an extra file handle open when it builds strace, and uio.test > assumes only stdin, stdout and stderr

Re: [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-10 Thread Dmitry V. Levin
On Fri, Jul 10, 2015 at 02:36:38PM +0200, Patrik Jakobsson wrote: On Wed, Jul 08, 2015 at 03:11:36AM +0300, Dmitry V. Levin wrote: On Mon, Jul 06, 2015 at 04:40:24PM +0200, Gabriel Laskar wrote: [...] Anyway, SYS_FUNC(ioctl) is a bit complicated, and the handling of the fallbacks

Re: [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-07 Thread Dmitry V. Levin
On Mon, Jul 06, 2015 at 04:40:24PM +0200, Gabriel Laskar wrote: On Mon, 6 Jul 2015 12:35:52 +0200, Patrik Jakobsson wrote: On Fri, Jul 03, 2015 at 03:36:09AM +0300, Dmitry V. Levin wrote: On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote: [...] --- a/drm.c +++ b

Re: Adding BLKPG_RESIZE_PARTITION

2015-07-09 Thread Dmitry V. Levin
Hi, On Wed, Jul 08, 2015 at 11:36:49AM +0200, Christian Neukirchen wrote: Hi, since Linux 3.6 there is an BLKPG (sub-)ioctl to tell the kernel that partition size has changed, e.g. used by partx -u. With strace 4.10 this is dumped as ioctl(3, BLKPG, {0x3 /* BLKPG_??? */, flags=0,

Re: [PATCH v3] decode extend getsockopt/setsockopt options

2015-08-26 Thread Dmitry V. Levin
On Thu, Aug 20, 2015 at 09:22:08AM -0400, Mike Frysinger wrote: On 20 Aug 2015 14:42, Dmitry V. Levin wrote: On Wed, Aug 19, 2015 at 01:29:27PM -0400, Mike Frysinger wrote: * util.c (printxval): Rename to ... (printxvals): ... this. Rewrite to be varargs based. OK, let's have two

Re: Multiple personalities support

2015-08-29 Thread Dmitry V. Levin
Hi, On Sat, Aug 22, 2015 at 03:36:42AM +0300, Elvira Khabirova wrote: Hello, As a part of my project for GSoC 2015 a subsystem for proper multiple personalities support have been developed. Currently some parsers do not handle differences in definitions of data types between

Re: [Intel-gfx] [PATCH v4 2/5] drm: Add private data field to trace control block

2015-09-07 Thread Dmitry V. Levin
On Mon, Aug 31, 2015 at 02:37:07PM +0200, Patrik Jakobsson wrote: [...] > Here's my take on it (I assume it needs some discussion): > > int > set_tcb_priv_data(struct tcb *tcp, void *priv_data) > { > /* A free callback is required before setting private data and private >* data must

Re: [PATCH v4 4/5] drm: Add decoding of i915 ioctls

2015-09-07 Thread Dmitry V. Levin
On Mon, Aug 24, 2015 at 02:42:49PM +0200, Patrik Jakobsson wrote: > +static int i915_getparam(struct tcb *tcp, const unsigned int code, long arg) > +{ > + struct drm_i915_getparam param; > + int value; > + > + if (umove(tcp, arg, )) > + return RVAL_DECODED; > + > + if

Re: [PATCH v4 3/5] drm: Add dispatcher and driver identification for DRM

2015-09-07 Thread Dmitry V. Levin
On Mon, Aug 24, 2015 at 02:42:48PM +0200, Patrik Jakobsson wrote: > * Makefile.am: Add compilation of drm.c. > * defs.h: Add extern declaration of drm_ioctl when drm headers are found. > * drm.c: New file. > * ioctl.c (ioctl_decode): Dispatch drm ioctls when drm headers are found. * defs.h

Re: [PATCH v4 4/5] drm: Add decoding of i915 ioctls

2015-09-07 Thread Dmitry V. Levin
On Tue, Sep 08, 2015 at 04:18:11AM +0300, Dmitry V. Levin wrote: [...] > So the whole function should look smth like this: > > static int i915_getparam(struct tcb *tcp, const unsigned int code, long arg) > { > struct drm_i915_getparam param; > >

Re: [PATCH v4 5/5] drm: Add decoding of DRM and KMS ioctls

2015-09-08 Thread Dmitry V. Levin
On Mon, Aug 24, 2015 at 02:42:50PM +0200, Patrik Jakobsson wrote: > First batch of drm / kms ioctls. Several comments in addition to issues similar to 4/5 patch. > +static int drm_mode_rm_fb(struct tcb *tcp, const unsigned int code, long arg) > +{ > + unsigned int handle; > + > + > + if

Re: [PATCH] Add PowerPC specific ptrace constants

2015-09-09 Thread Dmitry V. Levin
Hi, On Wed, Sep 09, 2015 at 04:28:54PM +1000, Anton Blanchard wrote: > Hi Dmitry, > > > Should other PPC_PTRACE_* constants be added along with these? > > The PPC_PTRACE_*_3264 calls were used when bootstrapping 64bit years > ago, I doubt they are used anywhere these days. > > We could add

Re: [PATCH v2] ioctl: fix ioctl command number decoding in case of conflicts

2015-09-23 Thread Dmitry V. Levin
On Wed, Sep 23, 2015 at 10:11:55AM +0200, Gabriel Laskar wrote: > When a command number was decoded through ioctl_decode_command_number(), > there was no check for conflicts with other potential ioctls numbers. Merged, thanks. -- ldv pgpiEZIHmyV0L.pgp Description: PGP signature

Re: [PATCH] ioctl: allow to stop decoding number

2015-12-04 Thread Dmitry V. Levin
On Fri, Dec 04, 2015 at 01:07:33AM +0100, Gabriel Laskar wrote: > For some ioctls, like from drm, the identification of the correct ioctl > is done with custom code. Specifying IOCTL_NUMBER_STOP_LOOKUP on > return of ioctl_decode_command_number() disable further calls to > ioctl_lookup() Applied,

Re: Preparing for the next release: call for testing

2015-12-10 Thread Dmitry V. Levin
Hi, On Thu, Dec 10, 2015 at 10:54:55AM -0800, Elliott Hughes wrote: > other than one small patch to build (below and attached, since gmail > mangles these things), lgtm on Android. > > If we build strace against the untouched Linux kernel uapi headers (as used > by bionic), the key fields are

Preparing for the next release: call for testing

2015-12-09 Thread Dmitry V. Levin
Hi, There is quite enough good stuff accumulated in strace.git since v4.10 so I think it's time to cut a new release. If you have pending patches to include before the release, please submit them now. There have been quite a few changes since v4.10 that could have caused regressions, so please

Re: Preparing for the next release: call for testing

2015-12-16 Thread Dmitry V. Levin
On Thu, Dec 10, 2015 at 09:37:35PM -0500, Mike Frysinger wrote: > i haven't had a chance yet to triage these. > > testing commit bab4ef4272cd2596c7390b34ea8acc086ee8fdb2 (v4.10-566-gbab4ef4) > > native (build+tests): > vFAIL: build; x86_64 ??? linux-4.1.6 kernel-headers-3.4.0

Re: Preparing for the next release: call for testing

2015-12-15 Thread Dmitry V. Levin
On Mon, Dec 14, 2015 at 04:45:33PM -0500, Mike Frysinger wrote: > On 15 Dec 2015 00:03, Dmitry V. Levin wrote: > > On Mon, Dec 14, 2015 at 03:45:40PM -0500, Mike Frysinger wrote: > > > On 11 Dec 2015 06:30, Dmitry V. Levin wrote: > > > > On Thu, Dec 10, 2015 at 09

Re: Preparing for the next release: call for testing

2015-12-14 Thread Dmitry V. Levin
On Tue, Dec 15, 2015 at 12:59:48AM +0300, Dmitry V. Levin wrote: > On Mon, Dec 14, 2015 at 04:46:49PM -0500, Mike Frysinger wrote: [...] > > we probably should add a sanity check to the gawk script so > > it asserts rather than OOMs. > > Sure. I've pushed a fix fo

Re: SYS_FUNC macro fix for musl

2015-12-14 Thread Dmitry V. Levin
Hi, On Mon, Dec 14, 2015 at 09:05:50PM +0100, Szabolcs Nagy wrote: > currently the syscall_name argument of the SYS_FUNC and > SEN macros is subject to macro expansion (because it is > passed to other function-like macros as parameter before > it is prefixed). > > if lfs64 support is requested

Re: SYS_FUNC macro fix for musl

2015-12-15 Thread Dmitry V. Levin
On Tue, Dec 15, 2015 at 10:56:49PM +0100, Szabolcs Nagy wrote: > * Dmitry V. Levin <l...@altlinux.org> [2015-12-15 14:59:32 +0300]: > > On Tue, Dec 15, 2015 at 12:02:31PM +0100, Szabolcs Nagy wrote: > > > * Dmitry V. Levin <l...@altlinux.org> [2015-12-15 00:35:56

Re: Preparing for the next release: call for testing

2015-12-15 Thread Dmitry V. Levin
On Mon, Dec 14, 2015 at 04:45:33PM -0500, Mike Frysinger wrote: > On 15 Dec 2015 00:03, Dmitry V. Levin wrote: > > On Mon, Dec 14, 2015 at 03:45:40PM -0500, Mike Frysinger wrote: [...] > > > looks like strace doesn't like it when you call clock_gettime: > &g

Re: SYS_FUNC macro fix for musl

2015-12-15 Thread Dmitry V. Levin
On Wed, Dec 16, 2015 at 02:45:45AM +0100, Szabolcs Nagy wrote: > * Dmitry V. Levin <l...@altlinux.org> [2015-12-16 03:33:03 +0300]: > > On Tue, Dec 15, 2015 at 10:56:49PM +0100, Szabolcs Nagy wrote: > > > attached the test logs too > > > > Looks like all tests

Re: Preparing for the next release: call for testing

2015-12-10 Thread Dmitry V. Levin
On Thu, Dec 10, 2015 at 09:37:35PM -0500, Mike Frysinger wrote: > i haven't had a chance yet to triage these. > > testing commit bab4ef4272cd2596c7390b34ea8acc086ee8fdb2 (v4.10-566-gbab4ef4) > > native (build+tests): > vFAIL: build; x86_64 ??? linux-4.1.6 kernel-headers-3.4.0

Re: Preparing for the next release: call for testing

2015-12-14 Thread Dmitry V. Levin
On Mon, Dec 14, 2015 at 04:46:49PM -0500, Mike Frysinger wrote: > On 15 Dec 2015 00:24, Dmitry V. Levin wrote: > > On Mon, Dec 14, 2015 at 03:45:40PM -0500, Mike Frysinger wrote: > > [...] > > > the older version of gawk used seems to be unhappy with the script. >

Re: Preparing for the next release: call for testing

2015-12-14 Thread Dmitry V. Levin
On Mon, Dec 14, 2015 at 03:45:40PM -0500, Mike Frysinger wrote: [...] > the older version of gawk used seems to be unhappy with the script. > when i run the command manually: > $ gawk -v VAR_NAME=mpers_target_var -v ARCH_FLAG=m32 -f ./mpers.awk > mpers-m32/kernel_dirent.d2 > #include > typedef >

Re: Preparing for the next release: call for testing

2015-12-15 Thread Dmitry V. Levin
On Tue, Dec 15, 2015 at 07:06:33AM +0300, Dmitry V. Levin wrote: > On Tue, Dec 15, 2015 at 12:59:48AM +0300, Dmitry V. Levin wrote: > > On Mon, Dec 14, 2015 at 04:46:49PM -0500, Mike Frysinger wrote: > [...] > > > we probably should add a sanity check to the gawk script so &g

Re: [PATCH v2] Add some tests for -f option

2015-12-30 Thread Dmitry V. Levin
On Thu, Dec 17, 2015 at 10:45:00AM +0800, Fei Jie wrote: > tests: add tests from option-f_001.test to option-f_003.test These tests would be a good addition 5 years ago when we had none, or even 2 years ago when we had just 8 automated tests. Now we need more rigorous tests, like fork-f.test and

Re: make check failing on times with -m32

2016-01-07 Thread Dmitry V. Levin
Hi, On Thu, Jan 07, 2016 at 01:49:08AM +, Steve McIntyre wrote: > Hey folks, > > I'm seeing this both in the 4.11 tarball (which I was just hoping to > push into Debian) and wuth current git HEAD. Everything builds fine > for me on amd64 and tests run for 64-bit fine, but in tests-32 I get a

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