On Wed, Jan 03, 2018 at 10:06:56PM +0000, Bart Van Assche wrote: > 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. Sounds fine to me. Can they rebase like that without conflicts? Jason -- 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
