All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eduardo Otubo <eduardo.otubo@profitbricks.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Paul Moore <pmoore@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Juan Quintela <quintela@redhat.com>
Subject: Re: [Qemu-devel] [PULL for v2.3 01/01] seccomp: update libseccomp version and remove arch restriction
Date: Thu, 26 Mar 2015 16:06:45 +0100	[thread overview]
Message-ID: <20150326150645.GC24566@vader> (raw)
In-Reply-To: <CAFEAcA_-LSZeuwNd-_Pw87TB+C5XDUvjRhZ6ews8qCTQy_4nxA@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1492 bytes --]

On Thu, Mar 26, 2015 at 02=44=14PM +0000, Peter Maydell wrote:
> On 26 March 2015 at 14:37, Eduardo Otubo <eduardo.otubo@profitbricks.com> wrote:
> > I completely understand your concern. Perhaps a ping on libseccomp
> > Fedora package maintainer would be a better way to tackle this issue
> > instead of reverting this commit. Libseccomp 2.2.0 is released since Feb
> > 12th and I actually gave it a little time frame for other distros to
> > update their packages so we don't run into issues like this.
> 
> Well, we shouldn't really be mandating latest-and-greatest versions
> of our upstream dependencies unless the maintainer of those dependencies
> feels the earlier versions are so badly broken that it would be better
> to refuse to use them at all.
> 
> > It's important to remember that this patch is also the proper fix for
> > this bug: https://bugs.launchpad.net/qemu/+bug/1363641
> 
> If that only applies to certain architectures we can make the
> dependency version vary depending on which arch we're building
> for, I suppose.
> 

This sounds more like a reasonable approach that could solve the above
mentioned problem and also making virt-test to be able to keep using
this feature as well -- which is also very important in order to get
more important system calls for the whitelist. I'll roll out a new patch
for that.

Thanks for the idea, Paul.
Sorry for the trouble on your side, Juan.

Regards,

-- 
Eduardo Otubo
ProfitBricks GmbH

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

  parent reply	other threads:[~2015-03-26 15:06 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-25 10:26 [Qemu-devel] [PULL for v2.3 00/01] seccomp branch queue Eduardo Otubo
2015-03-25 10:26 ` [Qemu-devel] [PULL for v2.3 01/01] seccomp: update libseccomp version and remove arch restriction Eduardo Otubo
2015-03-26 13:38   ` Juan Quintela
2015-03-26 14:09     ` Juan Quintela
2015-03-26 14:17       ` Peter Maydell
2015-03-26 14:41       ` Eduardo Otubo
2015-03-26 14:37     ` Eduardo Otubo
2015-03-26 14:44       ` Peter Maydell
2015-03-26 14:50         ` Juan Quintela
2015-03-26 15:06         ` Eduardo Otubo [this message]
2015-03-26 15:30           ` Juan Quintela
2015-03-26 14:58       ` Peter Maydell
2015-03-26 12:18 ` [Qemu-devel] [PULL for v2.3 00/01] seccomp branch queue Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150326150645.GC24566@vader \
    --to=eduardo.otubo@profitbricks.com \
    --cc=peter.maydell@linaro.org \
    --cc=pmoore@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.