On 9/26/22 8:43 AM, Christoph Hellwig wrote:
> On Mon, Sep 26, 2022 at 08:41:38AM -0600, Jens Axboe wrote:
>> Sure, I don't really care. What name do you want for it?
>
> Maybe slow and fast? Or simple and meta?
So you want 'result' in a named struct too then? Because right now
it looks like this:
struct nvme_uring_cmd_pdu {
union {
struct bio *bio;
struct request *req;
};
u32 meta_len;
u32 nvme_status;
union {
struct {
void *meta; /* kernel-resident buffer */
void __user *meta_buffer;
};
u64 result;
};
};
Or just the union named so it's clear it's a union? That'd make it
pdu->u.meta
and so forth. I think that might be cleaner.
--
Jens Axboe
