lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAGZ=bqLhcjzPDdb8mkGf5MbwqjrA_3ETLDe9Xiv-Wq2GnTgaUw@mail.gmail.com>
Date:	Wed, 24 Aug 2011 15:32:26 -0400
From:	Kyle Moffett <kyle@...fetthome.net>
To:	Pekka Enberg <penberg@...nel.org>
Cc:	Jens Axboe <axboe@...nel.dk>,
	Philipp Reisner <philipp.reisner@...bit.com>,
	linux-kernel@...r.kernel.org, drbd-dev@...ts.linbit.com
Subject: Re: [GIT PULL] drbd-8.4 for mainline

On Wed, Aug 24, 2011 at 14:16, Pekka Enberg <penberg@...nel.org> wrote:
> On Wed, Aug 24, 2011 at 8:54 PM, Jens Axboe <axboe@...nel.dk> wrote:
>> So, please do this right. The email you sent is perfect for a 0/N header
>> email, then each patch must follow as a reply to that. There's git
>> send-email to help you with this. If you haven't used it before, I'd
>> advise you to do a dry-run or two to your own email address to ensure
>> that it produces the right result. When it does, then send it off to
>> lkml and myself. Then I too will see reviews and have a better chance of
>> judging whether this is mergeable now or not.
>
> Another thing: there's roughly 500 commit in the pull request. Please
> don't send all of them at the same time if you want people to take a
> look at them.

Clearly you've been developing DRBD 8.4 for quite a while; it will save you
a lot of time overall in the future to submit patches as soon as you have
each one ready for review.

You should be merging functional and code improvements continuously
into every release window, instead of batching them up and submitting
500 at a time.

Cheers,
Kyle Moffett
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ