Hello Jinke. On Tue, Feb 28, 2023 at 04:59:35PM +0800, Jinke Han <hanjinke.666@xxxxxxxxxxxxx> wrote: > and gives a huge priority to sync write ios. Sync queue only make sense > for write ios as we treat all read io as sync io. [...] > +/* For write ios, dispatch 4 sync ios and 1 normal io in one loop */ > +#define THROTL_SYNC_FACTOR 4 This looks like an unexplained number wired in. E.g. I'd expect that 1st iteration would have this factor==1, i.e. RR between sync and async queue. What is the reason for this value? E.g. I assume it'd depend on number of synchronous writers and flusher threads and their activity, i.e. could you describe why this value caters the best for wide range of various sync and async situations? Or add a comment what is the particular value based on (so that it can be reassessed should those dependencies change in the future). Thanks, Michal
Attachment:
signature.asc
Description: PGP signature
