From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1165835AbdD1Q0n (ORCPT ); Fri, 28 Apr 2017 12:26:43 -0400 Received: from mail-ua0-f180.google.com ([209.85.217.180]:35253 "EHLO mail-ua0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162184AbdD1Q0c (ORCPT ); Fri, 28 Apr 2017 12:26:32 -0400 MIME-Version: 1.0 X-Originating-IP: [108.49.102.27] In-Reply-To: References: <15bb2070c10.2852.85c95baa4474aabc7814e68940a78392@paul-moore.com> From: Paul Moore Date: Fri, 28 Apr 2017 12:26:31 -0400 Message-ID: Subject: Re: Boot regression caused by kauditd To: Cong Wang Cc: LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 28, 2017 at 12:11 PM, Cong Wang wrote: > On Fri, Apr 28, 2017 at 8:30 AM, Paul Moore wrote: >> On Thu, Apr 27, 2017 at 8:47 PM, Paul Moore wrote: >>> In that case please send a proper inline patch to the audit mailing list >>> and we'll review it. >>> >>> Thanks. >> >> Now that I'm back in front of a proper screen/keyboard I've been >> looking over your patch and while you are very right in that the >> current RCU usage is very wrong, there are quite a few things I would >> like to see changed in your patch ... I'm working on something right >> now, I'll post an RFC draft to the audit list and CC you once I get >> this sorted out, expect something in a few hours. >> >> Also, once you've had a look at this new patch, and assuming you are >> okay with it, I'd like to add your sign-off to it. This may not be >> your patch exactly, but a significant portion of it is borrowed from >> your patch yesterday. > > So your review process is: if people's V1 patch is not perfect, you > will rewrite it by yourself? As I mentioned earlier I didn't get a chance to properly review your patch yesterday for two important reasons: 1) it hit my inbox at the end of my day and I simply didn't have time and 2) you sent it as an attachment which makes it hard to review and provide feedback. I took a closer look at your patch this morning and noticed a number of things that needed additional work as well as some merge/porting things; considering I never saw a response from you on my last email asking for an inline patch submission and taking into account where we are in the merge window (I'd like to submit this fix during the v4.12 merge window) I went ahead and put together a patch based on your prototype. You'll see I just posted it and CC'd you (our emails probably crossed paths), asking if it was okay to add your sign-off and give you credit. I'm only trying to speed up the process. There is no malice here, I actually thought I was helping you out ... I suppose the old adage rings true: no good deed goes unpunished ;) -- paul moore www.paul-moore.com