Re: [PATCH v2] Fix crash in ipc_sem test

2015-03-22 Thread Andreas Schwab
Mike Frysinger vapier-abrp7r+bbdudnm+yrof...@public.gmane.org writes: On 22 Mar 2015 10:08, Andreas Schwab wrote: Mike Frysinger vapier-abrp7r+bbdudnm+yrofe0a-xmd5yjdbdmrexy1tmh2...@public.gmane.org writes: with the older ipc_sem (before 4ed340bae64c84897fa7e42f6142592ef899f0cd),

Re: [PATCH v2] Fix crash in ipc_sem test

2015-03-22 Thread Mike Frysinger
On 17 Mar 2015 09:20, Andreas Schwab wrote: Mike Frysinger vapier-abrp7r+bbdudnm+yrof...@public.gmane.org writes: the kernel is still broken, In which way? with the older ipc_sem (before 4ed340bae64c84897fa7e42f6142592ef899f0cd), building that ipc_sem and running strace on it creates an

Re: [PATCH v2] Fix crash in ipc_sem test

2015-03-17 Thread Mike Frysinger
fwiw, 68804b326709fadc7bb03f510a11771f07216a59 passes on ppc32 sparc32 userland where they were crashing/hanging before the kernel is still broken, but at least the strace tests don't hit that :) -mike signature.asc Description: Digital signature

Re: [PATCH v2] Fix crash in ipc_sem test

2015-03-17 Thread Andreas Schwab
Mike Frysinger vapier-abrp7r+bbdudnm+yrof...@public.gmane.org writes: the kernel is still broken, In which way? Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 And now for something completely different.

[PATCH v2] Fix crash in ipc_sem test

2015-03-16 Thread Andreas Schwab
Properly use union semun as argument of semctl. * tests/ipc_sem.c (main): Properly use union semun as argument of semctl. Don't handle EFAULT specially. * tests/ipc_sem.test: Revert last change. --- tests/ipc_sem.c| 24 tests/ipc_sem.test | 19 --- 2

Re: [PATCH v2] Fix crash in ipc_sem test

2015-03-16 Thread Dmitry V. Levin
On Mon, Mar 16, 2015 at 12:06:12PM +0100, Andreas Schwab wrote: Properly use union semun as argument of semctl. * tests/ipc_sem.c (main): Properly use union semun as argument of semctl. Don't handle EFAULT specially. * tests/ipc_sem.test: Revert last change. OK, please commit. -- ldv