+struct nvme_tcp_send_ctx {
+ struct bio *curr_bio;
+ struct iov_iter iter;
+ size_t offset;
+ size_t data_sent;
+ enum nvme_tcp_send_state state;
+};
+
+struct nvme_tcp_recv_ctx {
+ struct iov_iter iter;
+ struct bio *curr_bio;
+};
I don't understand these structures. There should only be
a bio to be send or receive, not both. Why do we need two
curr_bio pointers?
We don't really need both...
To me it seems like both structures should just go away and
move into nvme_tcp_request ala:
struct bio *curr_bio;
/* send state */
struct iov_iter send_iter;
size_t send_offset;
enum nvme_tcp_send_state send_state;
size_t data_sent;
/* receive state */
struct iov_iter recv_iter;
Sure, will move this.
