[SCSI] isci: fix breakage caused by >16byte CDB patch - linux-dev - Linux kernel development work

diff options

author	James Bottomley <JBottomley@Parallels.com>	2013-07-24 12:43:18 -0700
committer	James Bottomley <JBottomley@Parallels.com>	2013-07-24 14:18:12 -0700
commit	e1be09808e030d57bb743618eb41e2bc31dd464c (patch)
tree	d17034c79f22fbd304ab95146e8f9de22ad30f96 /include/uapi/linux/raid
parent	[SCSI] mvsas: Fix kernel panic on tile due to unaligned data access (diff)

[SCSI] isci: fix breakage caused by >16byte CDB patch

Oops, apparently no-one I cc'd at intel actually bothered to check this patch for the isci driver: commit e73823f7a2c921dcf068d34ea03bd682498d9e42 Author: James Bottomley <JBottomley@Parallels.com> Date: Tue May 7 15:38:18 2013 -0700 [SCSI] libsas: implement > 16 byte CDB support sci_swab32_cpy needs multiples of four, so for commands that aren't that, it's rounding the wrong way. fix by doing (len+3)/4 instead of len/4. Reported-by: Tony Luck <tony.luck@intel.com> Tested-by: Tony Luck <tony.luck@intel.com> Signed-off-by: James Bottomley <JBottomley@Parallels.com>

Diffstat (limited to 'include/uapi/linux/raid')

0 files changed, 0 insertions, 0 deletions

/* * Copyright (C) International Business Machines Corp., 2000-2002 * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by * the Free Software Foundation; either version 2 of the License, or * (at your option) any later version. * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See * the GNU General Public License for more details. * * You should have received a copy of the GNU General Public License * along with this program; if not, write to the Free Software * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ #ifndef H_JFS_XATTR #define H_JFS_XATTR #include <linux/xattr.h> /* * jfs_ea_list describe the on-disk format of the extended attributes. * I know the null-terminator is redundant since namelen is stored, but * I am maintaining compatibility with OS/2 where possible. */ struct jfs_ea { u8 flag; /* Unused? */ u8 namelen; /* Length of name */ __le16 valuelen; /* Length of value */ char name[0]; /* Attribute name (includes null-terminator) */ }; /* Value immediately follows name */ struct jfs_ea_list { __le32 size; /* overall size */ struct jfs_ea ea[0]; /* Variable length list */ }; /* Macros for defining maxiumum number of bytes supported for EAs */ #define MAXEASIZE 65535 #define MAXEALISTSIZE MAXEASIZE /* * some macros for dealing with variable length EA lists. */ #define EA_SIZE(ea) \ (sizeof (struct jfs_ea) + (ea)->namelen + 1 + \ le16_to_cpu((ea)->valuelen)) #define NEXT_EA(ea) ((struct jfs_ea *) (((char *) (ea)) + (EA_SIZE (ea)))) #define FIRST_EA(ealist) ((ealist)->ea) #define EALIST_SIZE(ealist) le32_to_cpu((ealist)->size) #define END_EALIST(ealist) \ ((struct jfs_ea *) (((char *) (ealist)) + EALIST_SIZE(ealist))) extern int __jfs_setxattr(tid_t, struct inode *, const char *, const void *, size_t, int); extern ssize_t __jfs_getxattr(struct inode *, const char *, void *, size_t); extern ssize_t jfs_listxattr(struct dentry *, char *, size_t); extern const struct xattr_handler *jfs_xattr_handlers[]; #ifdef CONFIG_JFS_SECURITY extern int jfs_init_security(tid_t, struct inode *, struct inode *, const struct qstr *); #else static inline int jfs_init_security(tid_t tid, struct inode *inode, struct inode *dir, const struct qstr *qstr) { return 0; } #endif #endif /* H_JFS_XATTR */


context:
space:
mode: