On Fri, 2017-04-21 at 12:55 +0900, damien.lemoal@xxxxxxx wrote: > [ ... ] > +static int validate_hardware_zone_model(struct dm_table *table, > + struct queue_limits *limits) > +{ > + [ ... ] > + unsigned int i = 0; > + [ ... ] > + while (i < num_targets) { > + [ ... ] > + i++; > + } Hello Damien, A minor comment: maybe it's more appropriate to implement this loop as a for-loop? Thanks, Bart. -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel