On Wed, 30 May 2018, Christoph Hellwig wrote: > > - req->completion_data = cqr; > > + *((struct dasd_ccw_req **) blk_mq_rq_to_pdu(req)) = cqr; > > + > > Please don't play such tricks. In general your driver structure > should have struct request embedded. If for some reason > struct dasd_ccw_req has a different life time please create a new > structure instead of these hacks. Would you agree that this is an improvement over using completion_data (at least now we don't mess with the block layers internal data)? I already looked into what needs to be done to put struct dasd_ccw_req behind the request and let it be managed by the block layer. My concerns are lifetime and memory constraints (some of our HW interfaces still use 31 bit pointers) but I think both of these are manageable with the blk-mq interfaces. But this would not be a small change and nothing for rc7. Would you be OK with doing the small fix now and I'll work on the mentioned improvement? Regards, Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
