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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
