On Thu, May 28, 2020 at 04:00:50PM +0800, Ming Lei wrote: > Move code for getting driver tag and budget into one helper, so > blk_mq_dispatch_rq_list gets a bit simplified, and easier to read. > > Meantime move updating of 'no_tag' and 'no_budget_available' into > the branch for handling partial dispatch because that is exactly > consumer of the two local variables. > > Also rename the parameter of 'got_budget' as 'ask_budget'. Personally, I find it harder to do multiple inversions of boolean. Well, it's propably just me. > > No functional change. > > Cc: Sagi Grimberg <sagi@xxxxxxxxxxx> > Cc: Baolin Wang <baolin.wang7@xxxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxxxxxxxxx> > Reviewed-by: Sagi Grimberg <sagi@xxxxxxxxxxx> > Reviewed-by: Christoph Hellwig <hch@xxxxxxxxxxxxx> > Reviewed-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx> > Signed-off-by: Ming Lei <ming.lei@xxxxxxxxxx> Reviewed-by: Daniel Wagner <dwagner@xxxxxxx>
