SCSI multibyte accessors
Subsystems (2)
PATCH 2/4 SCSI multibyte accessors
Introduce scsi_{get,put}_u{16,24,32}
The u16 and u32 variants are just aliases for be16 and be32 unaligned accessors. u24 is open-coded byte accessors as the meaning of a byteswapped u24 quantity isn't entirely unambiguous, but we know what we want to accomplish with scsi_get_u24.
Also remove scsi_to_u32 and change its only user to scsi_get_u32().
drivers/scsi/libsas/sas_expander.c +4 -4
@@ -541,10 +541,10 @@ int sas_smp_get_phy_events(struct sas_phy *phy)if (!res) goto out; - phy->invalid_dword_count =scsi_to_u32 (&resp[12 ]); - phy->running_disparity_error_count =scsi_to_u32 (&resp[16 ]); - phy->loss_of_dword_sync_count =scsi_to_u32 (&resp[20 ]); - phy->phy_reset_problem_count =scsi_to_u32 (&resp[24 ]); + phy->invalid_dword_count =scsi_get_u32 (&resp[12 ]); + phy->running_disparity_error_count =scsi_get_u32 (&resp[16 ]); + phy->loss_of_dword_sync_count =scsi_get_u32 (&resp[20 ]); + phy->phy_reset_problem_count =scsi_get_u32 (&resp[24 ]); out:kfree (resp);
include/scsi/scsi.h +19 -3
@@ -486,10 +486,26 @@ struct scsi_lun {/* Used to obtain the PCI location of a device */ #define SCSI_IOCTL_GET_PCI 0x5387 -/* Pull a u32 out of a SCSI message (using BE SCSI conventions) */ -static inline __u32 scsi_to_u32 (__u8 *ptr) + +/* SCSI is a big-endian protocol and does not align data naturally */ +#include <asm/unaligned.h> + +#define scsi_get_u16 (addr) get_unaligned_be16(addr) +#define scsi_get_u32 (addr) get_unaligned_be32(addr) +#define scsi_put_u16 (data, addr) put_unaligned_be16(data, addr) +#define scsi_put_u32 (data, addr) put_unaligned_be32(data, addr) + +/* READ6 / WRITE6 contain a 24-bit quantity */ +static inline unsigned int scsi_get_u24 (unsigned char *addr) +{ +return (addr[0 ] <<16 ) | (addr[1 ] <<8 ) | addr[2 ]; +} + +static inline void scsi_put_u24 (unsigned int data,unsigned char *addr) { -return (ptr[0 ]<<24 ) + (ptr[1 ]<<16 ) + (ptr[2 ]<<8 ) + ptr[3 ]; + addr[0 ] = data >>16 ; + addr[1 ] = data >>8 ; + addr[2 ] = data; }#endif /* _SCSI_SCSI_H */
Signature · 4 lines
Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."
James Bottomley
maintainerQuoted 8 lines · Matthew Wilcox
Introduce scsi_{get,put}_u{16,24,32}
The u16 and u32 variants are just aliases for be16 and be32 unaligned accessors. u24 is open-coded byte accessors as the meaning of a byteswapped u24 quantity isn't entirely unambiguous, but we know what we want to accomplish with
scsi_get_u24.Also remove
scsi_to_u32and change its only user toscsi_get_u32().
If we're just doing this in the abstract, then we need a u64 as well for things like port addresses.
Thanks,
James
Quoted 8 lines · James Bottomley
Quoted 7 lines · Matthew Wilcox
Introduce scsi_{get,put}_u{16,24,32}
The u16 and u32 variants are just aliases for be16 and be32 unaligned accessors. u24 is open-coded byte accessors as the meaning of a byteswapped u24 quantity isn't entirely unambiguous, but we know what we want to accomplish with
scsi_get_u24.
Quoted 4 lines
Quoted 1 line
Also remove
scsi_to_u32and change its only user toscsi_get_u32().If we're just doing this in the abstract, then we need a u64 as well for things like port addresses.
Ah, I hadn't noticed any u64s in the specs I've been looking at. Trivial to add, updated patch below. I'm not entirely doing this in the abstract; patch 3/4 uses scsi_put_u16().
----
Introduce scsi_{get,put}_u{16,24,32,64}
The u16, u32 and u64 variants are just aliases for the big-endian unaligned accessors. u24 is written as open-coded byte accessors since the meaning of a byteswapped u24 quantity isn't entirely unambiguous, but we know what we want to accomplish with scsi_get_u24.
Also remove scsi_to_u32 and change its only user to scsi_get_u32().
drivers/scsi/libsas/sas_expander.c +4 -4
@@ -541,10 +541,10 @@ int sas_smp_get_phy_events(struct sas_phy *phy)if (!res) goto out; - phy->invalid_dword_count =scsi_to_u32 (&resp[12 ]); - phy->running_disparity_error_count =scsi_to_u32 (&resp[16 ]); - phy->loss_of_dword_sync_count =scsi_to_u32 (&resp[20 ]); - phy->phy_reset_problem_count =scsi_to_u32 (&resp[24 ]); + phy->invalid_dword_count =scsi_get_u32 (&resp[12 ]); + phy->running_disparity_error_count =scsi_get_u32 (&resp[16 ]); + phy->loss_of_dword_sync_count =scsi_get_u32 (&resp[20 ]); + phy->phy_reset_problem_count =scsi_get_u32 (&resp[24 ]); out:kfree (resp);
include/scsi/scsi.h +21 -3
@@ -486,10 +486,28 @@ struct scsi_lun {/* Used to obtain the PCI location of a device */ #define SCSI_IOCTL_GET_PCI 0x5387 -/* Pull a u32 out of a SCSI message (using BE SCSI conventions) */ -static inline __u32 scsi_to_u32 (__u8 *ptr) + +/* SCSI is a big-endian protocol and does not align data naturally */ +#include <asm/unaligned.h> + +#define scsi_get_u16 (addr) get_unaligned_be16(addr) +#define scsi_get_u32 (addr) get_unaligned_be32(addr) +#define scsi_get_u64 (addr) get_unaligned_be64(addr) +#define scsi_put_u16 (data, addr) put_unaligned_be16(data, addr) +#define scsi_put_u32 (data, addr) put_unaligned_be32(data, addr) +#define scsi_put_u64 (data, addr) put_unaligned_be64(data, addr) + +/* READ6 / WRITE6 contain a 24-bit quantity */ +static inline unsigned int scsi_get_u24 (unsigned char *addr) +{ +return (addr[0 ] <<16 ) | (addr[1 ] <<8 ) | addr[2 ]; +} + +static inline void scsi_put_u24 (unsigned int data,unsigned char *addr) { -return (ptr[0 ]<<24 ) + (ptr[1 ]<<16 ) + (ptr[2 ]<<8 ) + ptr[3 ]; + addr[0 ] = data >>16 ; + addr[1 ] = data >>8 ; + addr[2 ] = data; }#endif /* _SCSI_SCSI_H */
Signature · 4 lines
Intel are signing my paycheques ... these opinions are still mine "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."