Re: [PATCH 00/44] Convert FibreChannel bsg code to use bsg-lib

2016-10-12 Thread Johannes Thumshirn
On Tue, Oct 11, 2016 at 09:49:38AM -0700, Christoph Hellwig wrote: > Hi Johannes, > > this looks great to me. But is there a chance to consolidate it into > a more manageable set of patches? E.g. all the patches to call > export fc_bsg_jobdone, use it directly and remove the function pointer >

Re: [PATCH 00/44] Convert FibreChannel bsg code to use bsg-lib

2016-10-11 Thread Hannes Reinecke
On 10/11/2016 01:28 PM, Johannes Thumshirn wrote: This series converts the current bsg usage in the FibreChannel drivers over to use bsg-lib. SAS will follow but 44 patches are big enough for one shot, so I didn't want to include SAS in there as well. It looks a bit huge but most of the patches

Re: [PATCH 00/44] Convert FibreChannel bsg code to use bsg-lib

2016-10-11 Thread Christoph Hellwig
Hi Johannes, this looks great to me. But is there a chance to consolidate it into a more manageable set of patches? E.g. all the patches to call export fc_bsg_jobdone, use it directly and remove the function pointer could go together, possibly even including the new calling convention. Similar

[PATCH 00/44] Convert FibreChannel bsg code to use bsg-lib

2016-10-11 Thread Johannes Thumshirn
This series converts the current bsg usage in the FibreChannel drivers over to use bsg-lib. SAS will follow but 44 patches are big enough for one shot, so I didn't want to include SAS in there as well. It looks a bit huge but most of the patches are only code movement and slow API conversions so