On Wed, 2018-01-03 at 14:51 -0700, Jason Gunthorpe wrote:
> On Wed, Jan 03, 2018 at 01:39:10PM -0800, Bart Van Assche wrote:
> > Hello Jason,
> >
> > This patch series adds RDMA/CM support to the SRP initiator and target drivers
> > and also includes a series of bug fixes for the SRPT driver. Please consider
> > at least the SRPT bug fixes for kernel v4.16 since one of these fixes is a
> > security fix.
>
> Bart, this is really big for the rc6 timeframe.. This might make it if
> there is a rc8 this cycle.
>
> I'll grab the two patches marked for stable to for-rc, they look
> trivial.
>
> I'm with the DaveM school of thought here that patch series should be
> 10-15 at most, and have a clear purpose, even if that purpose is just
> 'assorted one patch fixes'.. If you resend it can you split the series?
Hello Jason,
How about splitting this series as follows:
* Patches 1..4 with the SRP initiator changes.
* Patches 5..6 with the SRPT bugfixes that have the "Cc: stable" tag.
* Patches 7,8,11..19 with trival SRPT changes.
* Patches 9,10,20..28 with the deeper SRPT changes.
Thanks,
Bart.��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
