On 11/2/22 5:32 AM, Pavel Begunkov wrote: > On 10/31/22 13:41, Dylan Yudaken wrote: >> Split out the specific sendzc parts of struct io_sr_msg as other opcodes >> are going to be specialized. > > I'd suggest to put the fields into a union and not splitting the structs > for now, it can be done later. The reason is that the file keeps changing > relatively often, and this change will add conflicts complicating > backporting and cross-tree development (i.e. series that rely on both > net and io_uring trees). Not super important, but I greatly prefer having them split. That way the ownership is much clearer than a union, which always gets a bit iffy. -- Jens Axboe