> + trace.flags = 0;
> + trace.magic = cpu_to_be32(ZFCP_BLK_DRV_DATA_MAGIC);
> + if (fsf_req->adapter->adapter_features & FSF_FEATURE_MEASUREMENT_DATA) {
> + trace.flags |= ZFCP_BLK_LAT_VALID;
> + lat_inf = &fsf_req->qtcb->prefix.prot_status_qual.latency_info;
> + trace.channel_lat = cpu_to_be64(lat_inf->channel_lat * ticks);
> + trace.fabric_lat = cpu_to_be64(lat_inf->fabric_lat * ticks);
> + }
> + if (fsf_req->status & ZFCP_STATUS_FSFREQ_ERROR)
> + trace.flags |= ZFCP_BLK_REQ_ERROR;
> + trace.inb_usage = cpu_to_be16(fsf_req->qdio_inb_usage);
> + trace.outb_usage = cpu_to_be16(fsf_req->qdio_outb_usage);
Could we please not add any cpu_to_be* calls to s390 only code? s390 is a big
endian architecture and therefore these calls are confusing and pointless.
> +#else
> +#define zfcp_fsf_trace_latency(fsf_req) do { } while (0)
> +#endif
Please convert that to a static inline function for type checking.
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrace" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
