On Wed, Feb 28, 2018 at 11:28:16AM -0800, Bart Van Assche wrote:
> static bool blk_poll_stats_enable(struct request_queue *q)
> {
> - if (test_bit(QUEUE_FLAG_POLL_STATS, &q->queue_flags) ||
> - test_and_set_bit(QUEUE_FLAG_POLL_STATS, &q->queue_flags))
> + if (blk_queue_flag_test_and_set(QUEUE_FLAG_POLL_STATS, q))
Is this one really needed or just for symmetry? Even if something
would change the queue_flags after the first test_bit() call, the
test_and_set_bit() would still do the right thing, wouldn't it?
--
Johannes Thumshirn Storage
jthumshirn@xxxxxxx +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
