lkml.org 
[lkml]   [2017]   [Jan]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] IB/cma: Fix reversed test
Date

> On Jan 28, 2017, at 2:47 PM, Majd Dibbiny <majd@mellanox.com> wrote:
>
Please ignore the previous email.
It was part of an internal discussion..
> We have message sniffer that checks for unwanted prints after each test..
>
> Sent from my iPhone
>
>> On Jan 28, 2017, at 8:59 AM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
>>
>> On Fri, Jan 27, 2017 at 07:05:52PM -0500, Doug Ledford wrote:
>>>> Do you think this patch needs "Fixes:" and "Cc: stable" tags?
>>>
>>> It does not.
>>
>> We always should have fixes tags.
>>
>> When I'm reviewing, I try to look up the patch which introduced the bug
>> so I can figure out what the intent was. Having a Fixes tag speeds up
>> my work.
>>
>> Looking at how the bug was introduced sometimes helps to prevent bugs
>> from recurring in the future. For example, I've seen several bugs
>> introduced because the right people weren't on the CC to review it. For
>> this particular bug it feels like probably this bug could have been
>> detected with more testing. I doubt it would have made it into a
>> released kernel.
>>
>> Also it let's you CC the original authors and hopefully they can Ack it.
>>
>> regards,
>> dan carpenter
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html

\
 
 \ /
  Last update: 2017-01-28 14:03    [W:0.142 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site