To: corbet@lwn.net (Jonathan Corbet)
Cc: Neil Brown <neilb@suse.de>, Sam Ravnborg <sam@ravnborg.org>,
       linux-kernel@vger.kernel.org
Subject: Re: RFC: reviewer's statement of oversight
References: <31862.1191948560@lwn.net>
From: Roland Dreier <rdreier@cisco.com>
Date: Tue, 09 Oct 2007 10:25:57 -0700
In-Reply-To: <31862.1191948560@lwn.net> (Jonathan Corbet's message of "Tue, 09 Oct 2007 10:49:20 -0600")
Message-ID: <adaodf8i4ii.fsf@cisco.com>
User-Agent: Gnus/5.1008 (Gnus v5.10.8) XEmacs/21.4.20 (linux)
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 850
Lines: 19

 > > > + (b) Any problems, concerns, or questions relating to the patch have been
 > > > +     communicated back to the submitter.  I am satisfied with how the
 > > > +     submitter has responded to my comments.
 > > 
 > > This seems more detailed that necessary.  The process (communicated
 > > back / responded) is not really relevant.
 > 
 > Instead, it seems to me that the process is crucially important.

I agree with you Jon.  In fact my first reaction to your initial post
was that this section (b) was the most succinct distillation of the
most important part of reviewing that I've seen.

 - R.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/