On 04/21/2015 11:57 PM, Christoph Hellwig wrote:
Btw, once we bump the version anyway we should sort out the problem of handling data to userspace vs CDB to userspace handling. Right now the transport_type field sais it's a virtual device, which means the core will do all kinds of ALUA and PR allocations and checks, which are wrong for something that's entirely pass through (see the handling for pscsi). Unfortuntately the module allows to select either mode at runtime, making this selection a bit of a mess.
Change se_subsystem_api.transport_type from a field into a *function that takes a *se_device parameter?
(maybe rework our TRANSPORT_PLUGIN_ defines at the same time? I'm not seeing how we need 3 values, just a PASSTHROUGH flag seems like enough)
? -- 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