From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41149) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xnvuz-0007n0-Rj for qemu-devel@nongnu.org; Mon, 10 Nov 2014 15:50:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xnvuv-0007SS-Hq for qemu-devel@nongnu.org; Mon, 10 Nov 2014 15:50:33 -0500 Received: from mail-la0-f54.google.com ([209.85.215.54]:36048) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xnvuv-0007SO-9E for qemu-devel@nongnu.org; Mon, 10 Nov 2014 15:50:29 -0500 Received: by mail-la0-f54.google.com with SMTP id s18so8527119lam.27 for ; Mon, 10 Nov 2014 12:50:28 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <1415629278-24675-1-git-send-email-eduardo.otubo@profitbricks.com> References: <1415629278-24675-1-git-send-email-eduardo.otubo@profitbricks.com> From: Peter Maydell Date: Mon, 10 Nov 2014 20:50:07 +0000 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PULL 0/2] seccomp branch queue List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Otubo Cc: Paul Moore , QEMU Developers , Philipp Gesang On 10 November 2014 14:21, Eduardo Otubo wrote: > Hi, > > This is the seccomp branch queue with fixes regarding a build crach on armv7l > and adding new syscalls to the whitelist. Details below. > > The following changes since commit 6e76d125f244e10676b917208f2a074729820246: > > Update version for v2.2.0-rc0 release (2014-11-05 15:21:04 +0000) > > are available in the git repository at: > > git://github.com/otubo/qemu.git tags/pull-seccomp-20141110 > > for you to fetch changes up to f5c76d3eb66a63604d9d8b47ce94268f9babea10: > > seccomp: change configure to avoid arm 32 to break (2014-11-07 16:42:18 +0100) > > ---------------------------------------------------------------- > seccomp: change configure to avoid arm 32 to break > seccomp: whitelist syscalls fallocate(), fadvise64(), inotify_init1() and inotify_add_watch() > > ---------------------------------------------------------------- > Eduardo Otubo (1): > seccomp: change configure to avoid arm 32 to break > > Philipp Gesang (1): > seccomp: whitelist syscalls fallocate(), fadvise64(), inotify_init1() and inotify_add_watch() Hi. I'm afraid I'm going to have to ask you to respin this, because the patches are missing various reviewed-by/signed-off etc tags. Firstly, and least critically, the configure patch should have the reviewed-by and acked-by tags that people have posted on the mailing list for it; it's useful to keep a permanent record of these in the commit history (for instance it can give a good idea of who to cc if a change turns out to have a bug in future). More significantly, the patch from Philipp is missing a Signed-off-by: line from you. Every commit in a pullreq should have a signed-off-by: from the submaintainer(s) of the tree that's being pulled, because it represents your assertion that these changes are good to go into QEMU without further checking. Sorry to have to be picky about this. The good news is, your signed tag seems OK. You should make sure you've pushed your public key out to the public keyservers if you haven't already (gpg --send-keys 12F8BD2F) since it's not currently visible there [there is a lag of a day or two though so if you've sent it out recently it's probably just in that delay.] thanks -- PMM