Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-07 Thread Stefan Hajnoczi
On Fri, Dec 6, 2013 at 4:40 PM, Will Drewry wrote: > On Fri, Dec 6, 2013 at 3:13 AM, Stefan Hajnoczi wrote: >> On Thu, Dec 05, 2013 at 10:12:00AM -0600, Will Drewry wrote: >>> On Thu, Dec 5, 2013 at 7:15 AM, Stefan Hajnoczi wrote: >>> > On Wed, Dec 04, 2013 at 11:21:12AM -0200, Eduardo Otubo wro

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-06 Thread Will Drewry
On Fri, Dec 6, 2013 at 3:13 AM, Stefan Hajnoczi wrote: > On Thu, Dec 05, 2013 at 10:12:00AM -0600, Will Drewry wrote: >> On Thu, Dec 5, 2013 at 7:15 AM, Stefan Hajnoczi wrote: >> > On Wed, Dec 04, 2013 at 11:21:12AM -0200, Eduardo Otubo wrote: >> >> On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote:

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-06 Thread Stefan Hajnoczi
On Thu, Dec 05, 2013 at 10:12:00AM -0600, Will Drewry wrote: > On Thu, Dec 5, 2013 at 7:15 AM, Stefan Hajnoczi wrote: > > On Wed, Dec 04, 2013 at 11:21:12AM -0200, Eduardo Otubo wrote: > >> On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote: > >> >On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wr

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-05 Thread Will Drewry
On Thu, Dec 5, 2013 at 7:15 AM, Stefan Hajnoczi wrote: > On Wed, Dec 04, 2013 at 11:21:12AM -0200, Eduardo Otubo wrote: >> On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote: >> >On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote: >> >>>Developers will only be happy with seccomp if it's easy an

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-05 Thread Stefan Hajnoczi
On Wed, Dec 04, 2013 at 11:21:12AM -0200, Eduardo Otubo wrote: > On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote: > >On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote: > >>>Developers will only be happy with seccomp if it's easy and rewarding to > >>>support/debug. > >> > >>Agreed. > >> > >>

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-04 Thread Corey Bryant
On 12/04/2013 08:21 AM, Eduardo Otubo wrote: On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote: On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote: Developers will only be happy with seccomp if it's easy and rewarding to support/debug. Agreed. As a developer, how do you feel about th

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-04 Thread Eduardo Otubo
On 12/04/2013 07:39 AM, Stefan Hajnoczi wrote: On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote: Developers will only be happy with seccomp if it's easy and rewarding to support/debug. Agreed. As a developer, how do you feel about the audit/syslog based approach I mentioned earlie

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-04 Thread Eduardo Otubo
The existing approach clearly doesn't support the full range of options that users specify on the command-line. Bugs. It will get fixed in time with more testing/debugging. Eduardo is working on improving the testing and RH's QA folks are working hard to shake out the bugs too. I just poste

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-12-04 Thread Stefan Hajnoczi
On Fri, Nov 22, 2013 at 11:00:24AM -0500, Paul Moore wrote: > > Developers will only be happy with seccomp if it's easy and rewarding to > > support/debug. > > Agreed. > > As a developer, how do you feel about the audit/syslog based approach I > mentioned earlier? I used the commands you posted

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Paul Moore
On Friday, November 22, 2013 04:48:41 PM Stefan Hajnoczi wrote: > On Fri, Nov 22, 2013 at 09:44:42AM -0500, Paul Moore wrote: > > On Friday, November 22, 2013 11:39:31 AM Stefan Hajnoczi wrote: > > > On Thu, Nov 21, 2013 at 10:48:58AM -0500, Paul Moore wrote: > > > > I'm always open to suggestions

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Stefan Hajnoczi
On Fri, Nov 22, 2013 at 09:44:42AM -0500, Paul Moore wrote: > On Friday, November 22, 2013 11:39:31 AM Stefan Hajnoczi wrote: > > On Thu, Nov 21, 2013 at 10:48:58AM -0500, Paul Moore wrote: > > > I'm always open to suggestions on how to improve the development/debugging > > > process, so if you hav

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Paul Moore
On Friday, November 22, 2013 11:39:31 AM Stefan Hajnoczi wrote: > On Thu, Nov 21, 2013 at 10:48:58AM -0500, Paul Moore wrote: > > I'm always open to suggestions on how to improve the development/debugging > > process, so if you have any ideas please let me know. > > The failure mode is terrible:

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Paul Moore
On Friday, November 22, 2013 11:34:41 AM Stefan Hajnoczi wrote: > IMO this seccomp approach is doomed since QEMU does not practice > privilege separation. QEMU is monolithic so it's really hard to create > a meaningful sets of system calls. I'm a big fan of decomposing QEMU, but based on previous

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Stefan Hajnoczi
On Thu, Nov 21, 2013 at 10:48:58AM -0500, Paul Moore wrote: > On Thursday, November 21, 2013 04:14:11 PM Paolo Bonzini wrote: > > Il 30/10/2013 11:04, Stefan Hajnoczi ha scritto: > > > On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: > > >> On 10/22/2013 11:00 AM, Anthony Liguori wrot

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-22 Thread Stefan Hajnoczi
On Wed, Oct 30, 2013 at 11:04:39AM +0100, Stefan Hajnoczi wrote: > On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: > > On 10/22/2013 11:00 AM, Anthony Liguori wrote: > > >On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo > > > wrote: > > >>Inverting the way sandbox handles arguments, m

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-21 Thread Paolo Bonzini
Il 30/10/2013 11:04, Stefan Hajnoczi ha scritto: > On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: >> >> >> On 10/22/2013 11:00 AM, Anthony Liguori wrote: >>> On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo >>> wrote: Inverting the way sandbox handles arguments, making possible

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-21 Thread Eduardo Otubo
On 11/21/2013 01:48 PM, Paul Moore wrote: On Thursday, November 21, 2013 04:14:11 PM Paolo Bonzini wrote: Il 30/10/2013 11:04, Stefan Hajnoczi ha scritto: On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: On 10/22/2013 11:00 AM, Anthony Liguori wrote: On Tue, Oct 22, 2013 at 12

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-11-21 Thread Paul Moore
On Thursday, November 21, 2013 04:14:11 PM Paolo Bonzini wrote: > Il 30/10/2013 11:04, Stefan Hajnoczi ha scritto: > > On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: > >> On 10/22/2013 11:00 AM, Anthony Liguori wrote: > >>> On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo > >>> > >>

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-10-30 Thread Stefan Hajnoczi
On Wed, Oct 23, 2013 at 12:42:34PM -0200, Eduardo Otubo wrote: > > > On 10/22/2013 11:00 AM, Anthony Liguori wrote: > >On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo > > wrote: > >>Inverting the way sandbox handles arguments, making possible to have no > >>argument and still have '-sandbox on' e

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-10-23 Thread Eduardo Otubo
On 10/22/2013 11:00 AM, Anthony Liguori wrote: On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo wrote: Inverting the way sandbox handles arguments, making possible to have no argument and still have '-sandbox on' enabled. Signed-off-by: Eduardo Otubo --- The option '-sandbox on' is now used

Re: [Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-10-22 Thread Anthony Liguori
On Tue, Oct 22, 2013 at 12:21 PM, Eduardo Otubo wrote: > Inverting the way sandbox handles arguments, making possible to have no > argument and still have '-sandbox on' enabled. > > Signed-off-by: Eduardo Otubo > --- > > The option '-sandbox on' is now used by default by virt-test[0] -- it has be

[Qemu-devel] [PATCH for-1.7] seccomp: setting "-sandbox on" by default

2013-10-22 Thread Eduardo Otubo
Inverting the way sandbox handles arguments, making possible to have no argument and still have '-sandbox on' enabled. Signed-off-by: Eduardo Otubo --- The option '-sandbox on' is now used by default by virt-test[0] -- it has been merged into the 'next' branch and will be available in the next r