Use stl_le_p() and ldl_le_p() to read and write data from
buffers, rather than using pointer casts and cpu_to_le32()
for writes and le32_to_cpup() for reads. This:
* avoids lots of casts
* works even if the buffer isn't as aligned as the host would like
* avoids using the *_to_cpup() functions which we want to get rid of
Note that there may still be some places where a pointer from the
guest is cast to a pointer to a host structure; these would also
have to be changed for the device to work on a host CPU which
enforces alignment restrictions.
Signed-off-by: Peter Maydell
---
hw/usb/dev-network.c | 63 +---
1 file changed, 30 insertions(+), 33 deletions(-)
diff --git a/hw/usb/dev-network.c b/hw/usb/dev-network.c
index 74306b5..5c18198 100644
--- a/hw/usb/dev-network.c
+++ b/hw/usb/dev-network.c
@@ -670,48 +670,49 @@ static int ndis_query(USBNetState *s, uint32_t oid,
/* general oids (table 4-1) */
/* mandatory */
case OID_GEN_SUPPORTED_LIST:
-for (i = 0; i < ARRAY_SIZE(oid_supported_list); i++)
-((le32 *) outbuf)[i] = cpu_to_le32(oid_supported_list[i]);
+for (i = 0; i < ARRAY_SIZE(oid_supported_list); i++) {
+stl_le_p(outbuf + (i * sizeof(le32)), oid_supported_list[i]);
+}
return sizeof(oid_supported_list);
/* mandatory */
case OID_GEN_HARDWARE_STATUS:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
/* mandatory */
case OID_GEN_MEDIA_SUPPORTED:
-*((le32 *) outbuf) = cpu_to_le32(s->medium);
+stl_le_p(outbuf, s->medium);
return sizeof(le32);
/* mandatory */
case OID_GEN_MEDIA_IN_USE:
-*((le32 *) outbuf) = cpu_to_le32(s->medium);
+stl_le_p(outbuf, s->medium);
return sizeof(le32);
/* mandatory */
case OID_GEN_MAXIMUM_FRAME_SIZE:
-*((le32 *) outbuf) = cpu_to_le32(ETH_FRAME_LEN);
+stl_le_p(outbuf, ETH_FRAME_LEN);
return sizeof(le32);
/* mandatory */
case OID_GEN_LINK_SPEED:
-*((le32 *) outbuf) = cpu_to_le32(s->speed);
+stl_le_p(outbuf, s->speed);
return sizeof(le32);
/* mandatory */
case OID_GEN_TRANSMIT_BLOCK_SIZE:
-*((le32 *) outbuf) = cpu_to_le32(ETH_FRAME_LEN);
+stl_le_p(outbuf, ETH_FRAME_LEN);
return sizeof(le32);
/* mandatory */
case OID_GEN_RECEIVE_BLOCK_SIZE:
-*((le32 *) outbuf) = cpu_to_le32(ETH_FRAME_LEN);
+stl_le_p(outbuf, ETH_FRAME_LEN);
return sizeof(le32);
/* mandatory */
case OID_GEN_VENDOR_ID:
-*((le32 *) outbuf) = cpu_to_le32(s->vendorid);
+stl_le_p(outbuf, s->vendorid);
return sizeof(le32);
/* mandatory */
@@ -720,58 +721,57 @@ static int ndis_query(USBNetState *s, uint32_t oid,
return strlen((char *)outbuf) + 1;
case OID_GEN_VENDOR_DRIVER_VERSION:
-*((le32 *) outbuf) = cpu_to_le32(1);
+stl_le_p(outbuf, 1);
return sizeof(le32);
/* mandatory */
case OID_GEN_CURRENT_PACKET_FILTER:
-*((le32 *) outbuf) = cpu_to_le32(s->filter);
+stl_le_p(outbuf, s->filter);
return sizeof(le32);
/* mandatory */
case OID_GEN_MAXIMUM_TOTAL_SIZE:
-*((le32 *) outbuf) = cpu_to_le32(RNDIS_MAX_TOTAL_SIZE);
+stl_le_p(outbuf, RNDIS_MAX_TOTAL_SIZE);
return sizeof(le32);
/* mandatory */
case OID_GEN_MEDIA_CONNECT_STATUS:
-*((le32 *) outbuf) = cpu_to_le32(s->media_state);
+stl_le_p(outbuf, s->media_state);
return sizeof(le32);
case OID_GEN_PHYSICAL_MEDIUM:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
case OID_GEN_MAC_OPTIONS:
-*((le32 *) outbuf) = cpu_to_le32(
-NDIS_MAC_OPTION_RECEIVE_SERIALIZED |
-NDIS_MAC_OPTION_FULL_DUPLEX);
+stl_le_p(outbuf, NDIS_MAC_OPTION_RECEIVE_SERIALIZED |
+ NDIS_MAC_OPTION_FULL_DUPLEX);
return sizeof(le32);
/* statistics OIDs (table 4-2) */
/* mandatory */
case OID_GEN_XMIT_OK:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
/* mandatory */
case OID_GEN_RCV_OK:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
/* mandatory */
case OID_GEN_XMIT_ERROR:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
/* mandatory */
case OID_GEN_RCV_ERROR:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return sizeof(le32);
/* mandatory */
case OID_GEN_RCV_NO_BUFFER:
-*((le32 *) outbuf) = cpu_to_le32(0);
+stl_le_p(outbuf, 0);
return