On 8/2/2017 1:16 AM, Johannes Thumshirn wrote:
+ for (;;) { + deferfcp = list_first_entry_or_null(&queue->pending_cmd_list, + struct nvmet_fc_defer_fcp_req, req_list); + if (!deferfcp) + break;while ((deferfcp = list_first_entry_or_null(&queue->pending_cmd_list, struct nvmet_fc_defer_fcp_req, req_list)) != NULL) { ?
To me - this is harder to read. But not a big deal. -- james