On Wed, Feb 28, 2024 at 3:49 AM Yu Kuai <yukuai1@xxxxxxxxxxxxxxx> wrote: > > From: Yu Kuai <yukuai3@xxxxxxxxxx> > > Changes in v3: > - add patch 2, and fix that setup_conf() is missing in patch3; > - add some review tag from Xiao Ni(other than patch 2,3); It appears that v3 causes mdadm test "01replace" to run forever. I haven't figured out why, but v2 doesn't seem to have this issue. Thanks, Song > Changes in v2: > - add new conter in conf for patch 2; > - fix the case choose next idle while there is no other idle disk in > patch 3; > - add some review tag from Xiao Ni for patch 1, 4-8 > > The original idea is that Paul want to optimize raid1 read > performance([1]), however, we think that the original code for > read_balance() is quite complex, and we don't want to add more > complexity. Hence we decide to refactor read_balance() first, to make > code cleaner and easier for follow up.
