On Fri, Mar 3, 2017 at 1:49 AM, Shaohua Li <shli@xxxxxxxxxx> wrote: > On Thu, Mar 02, 2017 at 10:11:54AM +0800, Ming Lei wrote: >> Hi Shaohua, >> >> On Wed, Mar 1, 2017 at 7:31 AM, Shaohua Li <shli@xxxxxxxxxx> wrote: >> > On Tue, Feb 28, 2017 at 11:41:35PM +0800, Ming Lei wrote: >> >> This patch gets each page's reference of each bio for resync, >> >> then r1buf_pool_free() gets simplified a lot. >> >> >> >> The same policy has been taken in raid10's buf pool allocation/free >> >> too. >> > >> > We are going to delete the code, this simplify isn't really required. >> >> Could you explain it a bit? Or I can put this patch into this patchset if you >> can provide one? > > I mean you will replace the code soon, with the resync_pages stuff. So I > thought this isn't really required. OK, one benefit of this patch is to make the following one easier to review, but not a big deal, I may merge this into the following patch. Thanks, Ming Lei
