Re: [PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-03 Thread Tycho Andersen
On Mon, Dec 03, 2018 at 07:17:26PM -0700, Tycho Andersen wrote: > On Tue, Dec 04, 2018 at 10:07:38AM +0800, kbuild test robot wrote: > > Hi Tycho, > > > > I love your patch! Yet something to improve: > > > > [auto build test ERROR on linus/master] > > [also build test ERROR on v4.20-rc5 next-2018

Re: [PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-03 Thread Tycho Andersen
On Tue, Dec 04, 2018 at 10:07:38AM +0800, kbuild test robot wrote: > Hi Tycho, > > I love your patch! Yet something to improve: > > [auto build test ERROR on linus/master] > [also build test ERROR on v4.20-rc5 next-20181203] > [if your patch is applied to the wrong git tree, please drop us a note

Re: [PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-03 Thread kbuild test robot
Hi Tycho, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.20-rc5 next-20181203] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/

Re: [PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-03 Thread Paul Moore
On Mon, Dec 3, 2018 at 12:01 AM Serge E. Hallyn wrote: > On Sun, Dec 02, 2018 at 08:28:25PM -0700, Tycho Andersen wrote: > > The const qualifier causes problems for any code that wants to write to the > > third argument of the seccomp syscall, as we will do in a future patch in > > this series. >

Re: [PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-02 Thread Serge E. Hallyn
On Sun, Dec 02, 2018 at 08:28:25PM -0700, Tycho Andersen wrote: > The const qualifier causes problems for any code that wants to write to the > third argument of the seccomp syscall, as we will do in a future patch in > this series. > > The third argument to the seccomp syscall is documented as vo

[PATCH v9 2/4] seccomp: switch system call argument type to void *

2018-12-02 Thread Tycho Andersen
The const qualifier causes problems for any code that wants to write to the third argument of the seccomp syscall, as we will do in a future patch in this series. The third argument to the seccomp syscall is documented as void *, so rather than just dropping the const, let's switch everything to u