On 01/20/2012 08:12 AM, Christoph Hellwig wrote: > On Thu, Jan 19, 2012 at 01:39:17PM -0800, Andy Grover wrote: >> This saves all fabrics from calling core_tmr_alloc_req() and >> having to check the result. The downside is se_cmd gets bigger for all >> requests, but hopefully later patches will reduce it. > > Without patches to void the overhead it's not acceptable. Fortunately > it should be doable fairly simply by using an union for command vs > TMR fields. This was my thought too. We should be able to move cmd variables into a union w/tmr struct very soon, with a low risk of introducing bugs. -- Andy -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html