lkml.org 
[lkml]   [2019]   [Sep]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2] selftests: watchdog: Validate optional file argument
From
Date
On 9/17/19 8:54 AM, Eugeniu Rosca wrote:
> Shuah,
>
> On Mon, Sep 16, 2019 at 07:19:35PM -0600, shuah wrote:
>> On 9/16/19 12:49 PM, George G. Davis wrote:
>>> As reported by Eugeniu Rosca, a side of affect of commit c3f2490d6e92
>>> ("selftests: watchdog: Add optional file argument") is that arbitrary files
>>> may be opened for watchdog testing, e.g.
>>>
>>
>> You don't need to say this here since you are already have a
>> Reported-by tag.
>
> This looks like asking people to stick to your personal taste which
> BTW doesn't really match the patterns established in Linux community.
>
> With a bit of scripting, I am able to find around 4600 vanilla commits
> which happen to mention the name of the reporter in addition to
> Reported-by: https://paste.ubuntu.com/p/wNXfdGCJbX/ .
>
> I really don't care if my name is mentioned once or twice, but I do
> believe that requesting a new patch revision just based on this criteria
> is nonsense. Can you please revise your review criteria?

I already said what I want. I want two patches and the first one with
Fixes tag. The reason for that is that the first patch fixes a problem
in patch that is already in my tree which is fixes a problem.

I am going to mark the patch for stables and the first patch in this
series.

I would like the commit log written clearly. Having a clear commit log
is a critical review comment. It is important for any change to have
clear commit logs for clarity and maintainability.

So please send me two patches one with Fixes tag and second that has
the -i support.

thanks,
-- Shuah


\
 
 \ /
  Last update: 2019-09-17 17:26    [W:0.044 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site