On 7/17/19 11:19 AM, Zhangguanghui wrote: > Hi > Does Hannes' patch has been lightly tested on my scenario, always use 2 seconds delay before retrying. > The patch looks good to me, well running. > But I'd wonder whether 'if (!pg->interval) ' is necessary condition for codes style? > Thanks very much! > Yes, it is, as the 'interval' setting might've been set already, in which case we shouldn't be modifying it. Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking hare@xxxxxxx +49 911 74053 688 SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg HRB 247165 (AG München), GF: Felix Imendörffer