Messages in this thread | | | Date | Wed, 21 Mar 2007 09:50:33 -0400 | From | Theodore Tso <> | Subject | Re: [PATCH 00/22 take 3] UBI: Unsorted Block Images |
| |
On Wed, Mar 21, 2007 at 10:44:35AM +0200, Artem Bityutskiy wrote: > > As I mentioned to you in IRC, in the future if there is pending > > changes in response to reviewer comments, it might be a good idea to > > mention that, so that reviewers know not make those comments again, or > > worry that the comments had been ignored. > > Teo, I wrote you 2 times that your point was understood and this would > be fixed. You should not think your comments are ignored because they > are not.
Artem, no need to be defensive. You did tell me that you were going to address them; but then you resubmitted patches where they weren't address. Normally, patch authors take all of the comments, clear them all, and then in the next repost of the patch, either explain why it wasn't feasible to handle some of the comments, *OR* why some of the comments were so hard to handle that they wouldn't be handled until a future version of the patch. Furthermore, in a patch of the size that you are submitting, a listing of what you *did* fix would also be good.
And at this point, I don't doubt that you will at some point going to heed my comments --- but note that doing so will involve a massive refactorization of the code, which will tend to invalidate the reviews done of this current (take 3) version of the patches; so I am bit curious what was your motiviation in reposting this round of the patches.
Believe it or not, the people who are responding on this thread are trying to help. Otherwise they would just be ignoring you and UBI. Keeping this thought in mind and trying to help them, where in some cases perhaps they are lacking in the same knowledge and experience and those who have been working on UBI and have spent many months thinking about the problem, may help keep things more constructive.
Regards,
- Ted - 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/
| |