On Wed, 2015-05-13 at 07:32 +0200, Christoph Hellwig wrote: > On Tue, May 12, 2015 at 10:08:51PM -0700, Nicholas A. Bellinger wrote: > > The series is bisectable. With patch #1 in place ->device_list[] is > > still kzalloc()'ed, but new RCU pointer assignments are made into > > lun_entry_hlist[]. > > > > Squashing the RCU reader paths (#2-6) for merge is OK, but it's still > > nice to break up reader / updater changes into separate patches. > > Having the full data structure switch over in one patch really makes > reviewing and understning the change a lot easier. But changes to > say move to a mutex should indeed stay separate. So patches 1-6 > really should be mostly one. Well, was thinking 1-6 is too big for one patch, but I guess it's not so bad: drivers/target/target_core_device.c | 268 +++++++++++++++++++++++++++--------------------- drivers/target/target_core_fabric_configfs.c | 35 ++++--- drivers/target/target_core_internal.h | 1 + drivers/target/target_core_pr.c | 1 + drivers/target/target_core_pscsi.c | 17 ++- drivers/target/target_core_spc.c | 27 +++-- drivers/target/target_core_stat.c | 180 ++++++++++++++++---------------- drivers/target/target_core_tpg.c | 59 +++-------- drivers/target/target_core_ua.c | 51 ++++++--- include/target/target_core_base.h | 8 +- 10 files changed, 350 insertions(+), 297 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
