From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755358AbbJVAfX (ORCPT ); Wed, 21 Oct 2015 20:35:23 -0400 Received: from mail-ob0-f169.google.com ([209.85.214.169]:35008 "EHLO mail-ob0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753777AbbJVAfT (ORCPT ); Wed, 21 Oct 2015 20:35:19 -0400 Subject: Re: [PATCH 2/2] Fixed Trivial Warnings in file: Deleted Spaces prior to tabs, and added lines. modified: kernel/auditfilter.c To: Paul Moore References: <1444528675-13184-1-git-send-email-scott@matheina.com> <10476084.rIoW7K08e6@sifl> <5623DBF5.8020500@matheina.com> <20081879.d8apxLOWgP@sifl> Cc: eparis@redhat.com, linux-audit@redhat.com, linux-kernel@vger.kernel.org, trivial@kernel.org From: Scott Matheina Message-ID: <56282F48.4090800@matheina.com> Date: Wed, 21 Oct 2015 19:35:20 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20081879.d8apxLOWgP@sifl> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/21/2015 01:23 PM, Paul Moore wrote: > On Sunday, October 18, 2015 12:50:45 PM Scott Matheina wrote: >> On 10/14/2015 04:54 PM, Paul Moore wrote: >>> On Saturday, October 10, 2015 08:57:55 PM Scott Matheina wrote: >>>> Signed-off-by: Scott Matheina >>>> --- >>>> >>>> kernel/auditfilter.c | 17 ++++++++++------- >>>> 1 file changed, 10 insertions(+), 7 deletions(-) >>> Sorry for the delay in reviewing this, comments inline ... >>> >>>> diff --git a/kernel/auditfilter.c b/kernel/auditfilter.c >>>> index 7714d93..774f9ad 100644 >>>> --- a/kernel/auditfilter.c >>>> +++ b/kernel/auditfilter.c >>>> @@ -39,13 +39,13 @@ >>>> >>>> * Locking model: >>>> * >>>> >>>> * audit_filter_mutex: >>>> - * Synchronizes writes and blocking reads of audit's filterlist >>>> - * data. Rcu is used to traverse the filterlist and access >>>> - * contents of structs audit_entry, audit_watch and opaque >>>> - * LSM rules during filtering. If modified, these structures >>>> - * must be copied and replace their counterparts in the filterlist. >>>> - * An audit_parent struct is not accessed during filtering, so may >>>> - * be written directly provided audit_filter_mutex is held. >>>> + * Synchronizes writes and blocking reads of audit's filterlist >>>> + * data. Rcu is used to traverse the filterlist and access >>>> + * contents of structs audit_entry, audit_watch and opaque >>>> + * LSM rules during filtering. If modified, these structures >>>> + * must be copied and replace their counterparts in the filterlist. >>>> + * An audit_parent struct is not accessed during filtering, so may >>>> + * be written directly provided audit_filter_mutex is held. >>>> >>>> */ >>> Okay, that's fine. > ... > >> As you might have guessed, this is one of my first patches. I wasn't sure if >> a patch like this would even get reviewed, and responded to. I'm subscribed >> to the linux-kernel mail group, and seeing what is acceptable. >> >> Thanks for the review. I don't plan on making a habit of submitting such >> incredibly trivial patches, but you have to start somewhere, and I thought >> it'd be hard to screw up by fixing a couple of trivial style errors. > We all start somewhere, and with that in mind, if you want to resubmit this > patch with only the fix above (fixing the whitespace in the comment block), > I'll apply it. While the patch is trivial, it is does fix a minor nit with > near-zero risk. > > I would encourage you to try something a bit more substantial next time, as > they say, bug fixes are the quickest way to a maintainer's heart. > Thanks for the feedback. I'll resubmit. Now I get to figure out how to resubmit a patch with changes, so a good learning experience for me. Pure Hobbyist at this time, but I love to learn. Scott