+ if (!test_bit(QUEUE_FLAG_POLL, &q->queue_flags)) + bio->bi_opf &= ~REQ_HIPRI; +
Maybe we can simply check (q->queue_flags & (1 << QUEUE_FLAG_POLL)) and avoid the extra atomic operation in the host path? Would it make sense?
+ if (!test_bit(QUEUE_FLAG_POLL, &q->queue_flags)) + bio->bi_opf &= ~REQ_HIPRI; +
Maybe we can simply check (q->queue_flags & (1 << QUEUE_FLAG_POLL)) and avoid the extra atomic operation in the host path? Would it make sense?