On Fri, Apr 08, 2016 at 02=16=33PM +0100, James Hogan wrote: > The cacheflush system call (found on MIPS and ARM) has been included in > the libseccomp header since 2.2.0, so include it back to that version. > Previously it was only enabled since 2.2.3 since that is when it was > enabled properly for ARM. > > This will allow seccomp support to be enabled for MIPS back to > libseccomp 2.2.0. > > Signed-off-by: James Hogan <james.ho...@imgtec.com> > Reviewed-By: Andrew Jones <drjo...@redhat.com> > Cc: Eduardo Otubo <eduardo.ot...@profitbricks.com> > Cc: Aurelien Jarno <aurel...@aurel32.net> > --- > Changes in v2: > - Added Peter's comment > --- > qemu-seccomp.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/qemu-seccomp.c b/qemu-seccomp.c > index 2866e3c2a660..138ee022a8fe 100644 > --- a/qemu-seccomp.c > +++ b/qemu-seccomp.c > @@ -16,11 +16,13 @@ > #include <seccomp.h> > #include "sysemu/seccomp.h" > > +/* For some architectures (notably ARM) cacheflush is not supported until > + * libseccomp 2.2.3, but configure enforces that we are using a more recent > + * version on those hosts, so it is OK for this check to be less strict. > + */ > #if SCMP_VER_MAJOR >= 3 > #define HAVE_CACHEFLUSH > -#elif SCMP_VER_MAJOR == 2 && SCMP_VER_MINOR >= 3 > - #define HAVE_CACHEFLUSH > -#elif SCMP_VER_MAJOR == 2 && SCMP_VER_MINOR == 2 && SCMP_VER_MICRO >= 3 > +#elif SCMP_VER_MAJOR == 2 && SCMP_VER_MINOR >= 2 > #define HAVE_CACHEFLUSH > #endif > > -- > 2.4.10 >
Acked-by: Eduardo Otubo <eduardo.ot...@profitbricks.com> (if nothing else comes up for the seccomp queue this week, I'll prepare a pull request by Friday. Thanks for the contribution) -- Eduardo Otubo ProfitBricks GmbH
signature.asc
Description: Digital signature
