Messages in this thread | | | Date | Tue, 5 Jan 2016 11:33:00 +0100 | From | David Sterba <> | Subject | Re: [PATCH] BTRFS: Adds an option to select RAID Stripe size |
| |
On Wed, Dec 30, 2015 at 04:21:47PM -0500, Sanidhya Solanki wrote: > On Wed, 30 Dec 2015 17:17:22 +0100 > David Sterba <dsterba@suse.cz> wrote: > > > Let me note that a good reputation is also built from patch reviews > > (hint hint). > > Unfortunately, not too many patches coming in for BTRFS presently. > Mailing list activity is down to 25-35 mails per day. Mostly feature > and bug requests. > > I will try to pitch in with patch reviews where possible.
It was not meant specifically to you, but I won't discourage you from doing reviews of course. The period where a review is expected can vary and is bound to the development cycle of kernel. At the latest, they should come before the integration branch is put togheter (before the merge window), and for the rc's it's before the next schedule (less than a week).
The reviewed-by tag has a real meaning and weight in the community
http://lxr.free-electrons.com/source/Documentation/SubmittingPatches#L552
and besides that, subscribes the person to the blame game and can cause bad feelings if the code turns out to be buggy later on.
| |