Re: [PATCH 4/8] sg: expand request states

[Bart Van Assche <bvanassche@xxxxxxx>]

On Fri, 2018-10-19 at 02:24 -0400, Douglas Gilbert wrote: 
+AD4 +-/+ACo Following defines are states of sg+AF8-request::rq+AF8-state +ACo-/
+AD4 +-+ACM-define SG+AF8-RQ+AF8-INACTIVE 0        /+ACo request not in use (e.g. on fl) +ACo-/
+AD4 +-+ACM-define SG+AF8-RQ+AF8-INFLIGHT 1        /+ACo SCSI request issued, no response yet +ACo-/
+AD4 +-+ACM-define SG+AF8-RQ+AF8-AWAIT+AF8-READ 2      /+ACo response received, awaiting read +ACo-/
+AD4 +-+ACM-define SG+AF8-RQ+AF8-DONE+AF8-READ 3       /+ACo read is ongoing or done +ACo-/
+AD4 +-+ACM-define SG+AF8-RQ+AF8-BUSY 4            /+ACo example: reserve request changing size +ACo-/

Please introduce an enumeration type instead of +ACM-defining these constants to
allow the compiler to verify assignments to sg+AF8-request::rq+AF8-state.

Thanks,

Bart.