Re: [PATCH 03/12] scsi_debug: replace jiffy timers with hr timers

[Bart Van Assche <bart.vanassche@xxxxxxxxxxx>]

On 04/25/2016 09:16 AM, Douglas Gilbert wrote:
> -				if (sdebug_ndelay > 0) {
> +				if ((sdebug_delay > 0) ||
> +				    (sdebug_ndelay > 0)) {
> [ ... ]
> -				if (sdebug_ndelay > 0) {
> +				if ((sdebug_delay > 0) ||
> +				    (sdebug_ndelay > 0)) {
> [ ... ]
> +	if ((delta_jiff > 0) || (sdebug_ndelay > 0)) {

Please do not introduce superfluous parentheses since this makes the 
code unnecessarily hard to read. Otherwise this patch looks fine to me.

Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html