Yeah, I don't have any major concerns. I was hung up on the tagset.cmd_size being smaller when SGL's are used, but it took me a minute to determine that's okay as we were overallocating it before. The criteria used to select sgl usage makes sense, so good to go on my end. Reviewed-by: Keith Busch <keith.busch at intel.com>