RE: [PATCH v2] fs/ntfs3: Use kernel ALIGN macros over driver specific

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> From: Kari Argillander <kari.argillander@xxxxxxxxx>
> Sent: Thursday, August 26, 2021 11:56 AM
> To: Konstantin Komarov <almaz.alexandrovich@xxxxxxxxxxxxxxxxxxxx>; ntfs3@xxxxxxxxxxxxxxx
> Cc: Kari Argillander <kari.argillander@xxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Subject: [PATCH v2] fs/ntfs3: Use kernel ALIGN macros over driver specific
> 
> The static checkers (Smatch) were complaining because QuadAlign() was
> buggy.  If you try to align something higher than UINT_MAX it got
> truncated to a u32.
> 
> Smatch warning was:
> 	fs/ntfs3/attrib.c:383 attr_set_size_res()
> 	warn: was expecting a 64 bit value instead of '~7'
> 
> So that this will not happen again we will change all these macros to
> kernel made ones. This can also help some other static analyzing tools
> to give us better warnings.
> 
> Patch was generated with Coccinelle script and after that some style
> issue was hand fixed.
> 
> Coccinelle script:
> 
> virtual patch
> 
> @alloc depends on patch@
> expression x;
> @@
> (
> -	#define QuadAlign(n)		(((n) + 7u) & (~7u))
> |
> -	QuadAlign(x)
> +	ALIGN(x, 8)
> |
> -	#define IsQuadAligned(n)	(!((size_t)(n)&7u))
> |
> -	IsQuadAligned(x)
> +	IS_ALIGNED(x, 8)
> |
> -	#define Quad2Align(n)		(((n) + 15u) & (~15u))
> |
> -	Quad2Align(x)
> +	ALIGN(x, 16)
> |
> -	#define IsQuad2Aligned(n)	(!((size_t)(n)&15u))
> |
> -	IsQuad2Aligned(x)
> +	IS_ALIGNED(x, 16)
> |
> -	#define Quad4Align(n)		(((n) + 31u) & (~31u))
> |
> -	Quad4Align(x)
> +	ALIGN(x, 32)
> |
> -	#define IsSizeTAligned(n)	(!((size_t)(n) & (sizeof(size_t) - 1)))
> |
> -	IsSizeTAligned(x)
> +	IS_ALIGNED(x, sizeof(size_t))
> |
> -	#define DwordAlign(n)		(((n) + 3u) & (~3u))
> |
> -	DwordAlign(x)
> +	ALIGN(x, 4)
> |
> -	#define IsDwordAligned(n)	(!((size_t)(n)&3u))
> |
> -	IsDwordAligned(x)
> +	IS_ALIGNED(x, 4)
> |
> -	#define WordAlign(n)		(((n) + 1u) & (~1u))
> |
> -	WordAlign(x)
> +	ALIGN(x, 2)
> |
> -	#define IsWordAligned(n)	(!((size_t)(n)&1u))
> |
> -	IsWordAligned(x)
> +	IS_ALIGNED(x, 2)
> |
> )
> 
> Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> Signed-off-by: Kari Argillander <kari.argillander@xxxxxxxxx>
> ---
> Disgussion that these should be changed can be found here
> lore.kernel.org/ntfs3/20210824104910.v3mvpw5zb2jqmvbo@kari-VirtualBox/
> 
> V2:
> 	Make better commit message. Request by Dan Carpenter.
> ---
>  fs/ntfs3/attrib.c  |  2 +-
>  fs/ntfs3/debug.h   | 11 -----------
>  fs/ntfs3/frecord.c | 14 +++++++-------
>  fs/ntfs3/fslog.c   | 32 ++++++++++++++++----------------
>  fs/ntfs3/fsntfs.c  |  4 ++--
>  fs/ntfs3/index.c   |  4 ++--
>  fs/ntfs3/inode.c   |  8 ++++----
>  fs/ntfs3/namei.c   |  2 +-
>  fs/ntfs3/ntfs.h    | 16 ++++++++--------
>  fs/ntfs3/ntfs_fs.h |  2 +-
>  fs/ntfs3/record.c  | 10 +++++-----
>  fs/ntfs3/super.c   | 10 +++++-----
>  fs/ntfs3/xattr.c   |  8 ++++----
>  13 files changed, 56 insertions(+), 67 deletions(-)
> 
> diff --git a/fs/ntfs3/attrib.c b/fs/ntfs3/attrib.c
> index 046dc57f75f2..c15467ec12ed 100644
> --- a/fs/ntfs3/attrib.c
> +++ b/fs/ntfs3/attrib.c
> @@ -380,7 +380,7 @@ static int attr_set_size_res(struct ntfs_inode *ni, struct ATTRIB *attr,
>  	u32 rsize = le32_to_cpu(attr->res.data_size);
>  	u32 tail = used - aoff - asize;
>  	char *next = Add2Ptr(attr, asize);
> -	s64 dsize = QuadAlign(new_size) - QuadAlign(rsize);
> +	s64 dsize = ALIGN(new_size, 8) - ALIGN(rsize, 8);
> 
>  	if (dsize < 0) {
>  		memmove(next + dsize, next, tail);
> diff --git a/fs/ntfs3/debug.h b/fs/ntfs3/debug.h
> index dfaa4c79dc6d..fe5ed660c3b0 100644
> --- a/fs/ntfs3/debug.h
> +++ b/fs/ntfs3/debug.h
> @@ -12,17 +12,6 @@
>  #define PtrOffset(B, O)		((size_t)((size_t)(O) - (size_t)(B)))
>  #endif
> 
> -#define QuadAlign(n)		(((n) + 7u) & (~7u))
> -#define IsQuadAligned(n)	(!((size_t)(n)&7u))
> -#define Quad2Align(n)		(((n) + 15u) & (~15u))
> -#define IsQuad2Aligned(n)	(!((size_t)(n)&15u))
> -#define Quad4Align(n)		(((n) + 31u) & (~31u))
> -#define IsSizeTAligned(n)	(!((size_t)(n) & (sizeof(size_t) - 1)))
> -#define DwordAlign(n)		(((n) + 3u) & (~3u))
> -#define IsDwordAligned(n)	(!((size_t)(n)&3u))
> -#define WordAlign(n)		(((n) + 1u) & (~1u))
> -#define IsWordAligned(n)	(!((size_t)(n)&1u))
> -
>  #ifdef CONFIG_PRINTK
>  __printf(2, 3)
>  void ntfs_printk(const struct super_block *sb, const char *fmt, ...);
> diff --git a/fs/ntfs3/frecord.c b/fs/ntfs3/frecord.c
> index c3121bf9c62f..28c471e5b098 100644
> --- a/fs/ntfs3/frecord.c
> +++ b/fs/ntfs3/frecord.c
> @@ -1249,7 +1249,7 @@ static int ni_expand_mft_list(struct ntfs_inode *ni)
>  	if (err < 0)
>  		goto out;
> 
> -	run_size = QuadAlign(err);
> +	run_size = ALIGN(err, 8);
>  	err = 0;
> 
>  	if (plen < svcn) {
> @@ -1269,7 +1269,7 @@ static int ni_expand_mft_list(struct ntfs_inode *ni)
>  	if (err < 0)
>  		goto out;
> 
> -	run_size = QuadAlign(err);
> +	run_size = ALIGN(err, 8);
>  	err = 0;
> 
>  	if (plen < evcn + 1 - svcn) {
> @@ -1392,7 +1392,7 @@ int ni_insert_nonresident(struct ntfs_inode *ni, enum ATTR_TYPE type,
>  	struct ATTRIB *attr;
>  	bool is_ext =
>  		(flags & (ATTR_FLAG_SPARSED | ATTR_FLAG_COMPRESSED)) && !svcn;
> -	u32 name_size = QuadAlign(name_len * sizeof(short));
> +	u32 name_size = ALIGN(name_len * sizeof(short), 8);
>  	u32 name_off = is_ext ? SIZEOF_NONRESIDENT_EX : SIZEOF_NONRESIDENT;
>  	u32 run_off = name_off + name_size;
>  	u32 run_size, asize;
> @@ -1403,7 +1403,7 @@ int ni_insert_nonresident(struct ntfs_inode *ni, enum ATTR_TYPE type,
>  	if (err < 0)
>  		goto out;
> 
> -	run_size = QuadAlign(err);
> +	run_size = ALIGN(err, 8);
> 
>  	if (plen < len) {
>  		err = -EINVAL;
> @@ -1463,8 +1463,8 @@ int ni_insert_resident(struct ntfs_inode *ni, u32 data_size,
>  		       struct ATTRIB **new_attr, struct mft_inode **mi)
>  {
>  	int err;
> -	u32 name_size = QuadAlign(name_len * sizeof(short));
> -	u32 asize = SIZEOF_RESIDENT + name_size + QuadAlign(data_size);
> +	u32 name_size = ALIGN(name_len * sizeof(short), 8);
> +	u32 asize = SIZEOF_RESIDENT + name_size + ALIGN(data_size, 8);
>  	struct ATTRIB *attr;
> 
>  	err = ni_insert_attr(ni, type, name, name_len, asize, SIZEOF_RESIDENT,
> @@ -2849,7 +2849,7 @@ static bool ni_update_parent(struct ntfs_inode *ni, struct NTFS_DUP_INFO *dup,
>  		} else if (!attr->non_res) {
>  			u32 data_size = le32_to_cpu(attr->res.data_size);
> 
> -			dup->alloc_size = cpu_to_le64(QuadAlign(data_size));
> +			dup->alloc_size = cpu_to_le64(ALIGN(data_size, 8));
>  			dup->data_size = cpu_to_le64(data_size);
>  		} else {
>  			u64 new_valid = ni->i_valid;
> diff --git a/fs/ntfs3/fslog.c b/fs/ntfs3/fslog.c
> index 397ba6a956e7..5a118c351441 100644
> --- a/fs/ntfs3/fslog.c
> +++ b/fs/ntfs3/fslog.c
> @@ -456,7 +456,7 @@ static inline bool is_rst_page_hdr_valid(u32 file_off,
>  		return false;
> 
>  	ro = le16_to_cpu(rhdr->ra_off);
> -	if (!IsQuadAligned(ro) || ro > sys_page)
> +	if (!IS_ALIGNED(ro, 8) || ro > sys_page)
>  		return false;
> 
>  	end_usa = ((sys_page >> SECTOR_SHIFT) + 1) * sizeof(short);
> @@ -488,7 +488,7 @@ static inline bool is_rst_area_valid(const struct RESTART_HDR *rhdr)
> 
>  	off = le16_to_cpu(ra->client_off);
> 
> -	if (!IsQuadAligned(off) || ro + off > SECTOR_SIZE - sizeof(short))
> +	if (!IS_ALIGNED(off, 8) || ro + off > SECTOR_SIZE - sizeof(short))
>  		return false;
> 
>  	off += cl * sizeof(struct CLIENT_REC);
> @@ -526,8 +526,8 @@ static inline bool is_rst_area_valid(const struct RESTART_HDR *rhdr)
>  	}
> 
>  	/* The log page data offset and record header length must be quad-aligned */
> -	if (!IsQuadAligned(le16_to_cpu(ra->data_off)) ||
> -	    !IsQuadAligned(le16_to_cpu(ra->rec_hdr_len)))
> +	if (!IS_ALIGNED(le16_to_cpu(ra->data_off), 8) ||
> +	    !IS_ALIGNED(le16_to_cpu(ra->rec_hdr_len), 8))
>  		return false;
> 
>  	return true;
> @@ -1355,9 +1355,9 @@ static void log_create(struct ntfs_log *log, u32 l_size, const u64 last_lsn,
>  		log->l_flags |= NTFSLOG_MULTIPLE_PAGE_IO;
> 
>  	/* Compute the log page values */
> -	log->data_off = QuadAlign(
> +	log->data_off = ALIGN(
>  		offsetof(struct RECORD_PAGE_HDR, fixups) +
> -		sizeof(short) * ((log->page_size >> SECTOR_SHIFT) + 1));
> +		sizeof(short) * ((log->page_size >> SECTOR_SHIFT) + 1), 8);
>  	log->data_size = log->page_size - log->data_off;
>  	log->record_header_len = sizeof(struct LFS_RECORD_HDR);
> 
> @@ -1365,9 +1365,9 @@ static void log_create(struct ntfs_log *log, u32 l_size, const u64 last_lsn,
>  	log->reserved = log->data_size - log->record_header_len;
> 
>  	/* Compute the restart page values. */
> -	log->ra_off = QuadAlign(
> +	log->ra_off = ALIGN(
>  		offsetof(struct RESTART_HDR, fixups) +
> -		sizeof(short) * ((log->sys_page_size >> SECTOR_SHIFT) + 1));
> +		sizeof(short) * ((log->sys_page_size >> SECTOR_SHIFT) + 1), 8);
>  	log->restart_size = log->sys_page_size - log->ra_off;
>  	log->ra_size = struct_size(log->ra, clients, 1);
>  	log->current_openlog_count = open_log_count;
> @@ -1496,7 +1496,7 @@ static int next_log_lsn(struct ntfs_log *log, const struct LFS_RECORD_HDR *rh,
> 
>  		vbo = hdr_off + log->data_off;
>  	} else {
> -		vbo = QuadAlign(end);
> +		vbo = ALIGN(end, 8);
>  	}
> 
>  	/* Compute the lsn based on the file offset and the sequence count */
> @@ -2982,7 +2982,7 @@ static struct ATTRIB *attr_create_nonres_log(struct ntfs_sb_info *sbi,
>  					     __le16 flags)
>  {
>  	struct ATTRIB *attr;
> -	u32 name_size = QuadAlign(name_len * sizeof(short));
> +	u32 name_size = ALIGN(name_len * sizeof(short), 8);
>  	bool is_ext = flags & (ATTR_FLAG_COMPRESSED | ATTR_FLAG_SPARSED);
>  	u32 asize = name_size +
>  		    (is_ext ? SIZEOF_NONRESIDENT_EX : SIZEOF_NONRESIDENT);
> @@ -3220,7 +3220,7 @@ static int do_action(struct ntfs_log *log, struct OPEN_ATTR_ENRTY *oe,
>  			goto dirty_vol;
> 
>  		memmove(attr, attr2, dlen);
> -		rec->used = cpu_to_le32(QuadAlign(roff + dlen));
> +		rec->used = cpu_to_le32(ALIGN(roff + dlen, 8));
> 
>  		mi->dirty = true;
>  		break;
> @@ -3231,7 +3231,7 @@ static int do_action(struct ntfs_log *log, struct OPEN_ATTR_ENRTY *oe,
>  		used = le32_to_cpu(rec->used);
> 
>  		if (!check_if_attr(rec, lrh) || dlen < SIZEOF_RESIDENT ||
> -		    !IsQuadAligned(asize) ||
> +		    !IS_ALIGNED(asize, 8) ||
>  		    Add2Ptr(attr2, asize) > Add2Ptr(lrh, rec_len) ||
>  		    dlen > record_size - used) {
>  			goto dirty_vol;
> @@ -3296,7 +3296,7 @@ static int do_action(struct ntfs_log *log, struct OPEN_ATTR_ENRTY *oe,
>  		if (nsize > asize && nsize - asize > record_size - used)
>  			goto dirty_vol;
> 
> -		nsize = QuadAlign(nsize);
> +		nsize = ALIGN(nsize, 8);
>  		data_off = le16_to_cpu(attr->res.data_off);
> 
>  		if (nsize < asize) {
> @@ -3341,7 +3341,7 @@ static int do_action(struct ntfs_log *log, struct OPEN_ATTR_ENRTY *oe,
>  			goto dirty_vol;
>  		}
> 
> -		nsize = QuadAlign(nsize);
> +		nsize = ALIGN(nsize, 8);
> 
>  		memmove(Add2Ptr(attr, nsize), Add2Ptr(attr, asize),
>  			used - le16_to_cpu(lrh->record_off) - asize);
> @@ -5103,8 +5103,8 @@ int log_replay(struct ntfs_inode *ni, bool *initialized)
>  	rh->sys_page_size = cpu_to_le32(log->page_size);
>  	rh->page_size = cpu_to_le32(log->page_size);
> 
> -	t16 = QuadAlign(offsetof(struct RESTART_HDR, fixups) +
> -			sizeof(short) * t16);
> +	t16 = ALIGN(offsetof(struct RESTART_HDR, fixups) +
> +		    sizeof(short) * t16, 8);
>  	rh->ra_off = cpu_to_le16(t16);
>  	rh->minor_ver = cpu_to_le16(1); // 0x1A:
>  	rh->major_ver = cpu_to_le16(1); // 0x1C:
> diff --git a/fs/ntfs3/fsntfs.c b/fs/ntfs3/fsntfs.c
> index 92140050fb6c..488e47c7f670 100644
> --- a/fs/ntfs3/fsntfs.c
> +++ b/fs/ntfs3/fsntfs.c
> @@ -1944,7 +1944,7 @@ int ntfs_security_init(struct ntfs_sb_info *sbi)
>  	sbi->security.next_id = SECURITY_ID_FIRST;
>  	/* Always write new security at the end of bucket */
>  	sbi->security.next_off =
> -		Quad2Align(sds_size - SecurityDescriptorsBlockSize);
> +			ALIGN(sds_size - SecurityDescriptorsBlockSize, 16);
> 
>  	cnt = 0;
>  	off = 0;
> @@ -2099,7 +2099,7 @@ int ntfs_insert_security(struct ntfs_sb_info *sbi,
>  	struct NTFS_DE_SII sii_e;
>  	struct SECURITY_HDR *d_security;
>  	u32 new_sec_size = size_sd + SIZEOF_SECURITY_HDR;
> -	u32 aligned_sec_size = Quad2Align(new_sec_size);
> +	u32 aligned_sec_size = ALIGN(new_sec_size, 16);
>  	struct SECURITY_KEY hash_key;
>  	struct ntfs_fnd *fnd_sdh = NULL;
>  	const struct INDEX_ROOT *root_sdh;
> diff --git a/fs/ntfs3/index.c b/fs/ntfs3/index.c
> index 6aa9540ece47..3fdec8871fab 100644
> --- a/fs/ntfs3/index.c
> +++ b/fs/ntfs3/index.c
> @@ -701,7 +701,7 @@ static struct NTFS_DE *hdr_find_e(const struct ntfs_index *indx,
> 
>  	if (max_idx >= nslots) {
>  		u16 *ptr;
> -		int new_slots = QuadAlign(2 * nslots);
> +		int new_slots = ALIGN(2 * nslots, 8);
> 
>  		ptr = ntfs_malloc(sizeof(u16) * new_slots);
>  		if (ptr)
> @@ -958,7 +958,7 @@ static struct indx_node *indx_new(struct ntfs_index *indx,
>  	index->rhdr.fix_num = cpu_to_le16(fn);
>  	index->vbn = cpu_to_le64(vbn);
>  	hdr = &index->ihdr;
> -	eo = QuadAlign(sizeof(struct INDEX_BUFFER) + fn * sizeof(short));
> +	eo = ALIGN(sizeof(struct INDEX_BUFFER) + fn * sizeof(short), 8);
>  	hdr->de_off = cpu_to_le32(eo);
> 
>  	e = Add2Ptr(hdr, eo);
> diff --git a/fs/ntfs3/inode.c b/fs/ntfs3/inode.c
> index bf51e294432e..36f673857ebd 100644
> --- a/fs/ntfs3/inode.c
> +++ b/fs/ntfs3/inode.c
> @@ -1336,7 +1336,7 @@ struct inode *ntfs_create_inode(struct user_namespace *mnt_userns,
>  	fname->dup.ea_size = fname->dup.reparse = 0;
> 
>  	dsize = le16_to_cpu(new_de->key_size);
> -	asize = QuadAlign(SIZEOF_RESIDENT + dsize);
> +	asize = ALIGN(SIZEOF_RESIDENT + dsize, 8);
> 
>  	attr->type = ATTR_NAME;
>  	attr->size = cpu_to_le32(asize);
> @@ -1350,7 +1350,7 @@ struct inode *ntfs_create_inode(struct user_namespace *mnt_userns,
> 
>  	if (security_id == SECURITY_ID_INVALID) {
>  		/* Insert security attribute */
> -		asize = SIZEOF_RESIDENT + QuadAlign(sd_size);
> +		asize = SIZEOF_RESIDENT + ALIGN(sd_size, 8);
> 
>  		attr->type = ATTR_SECURE;
>  		attr->size = cpu_to_le32(asize);
> @@ -1473,7 +1473,7 @@ struct inode *ntfs_create_inode(struct user_namespace *mnt_userns,
>  		attr->id = cpu_to_le16(aid++);
> 
>  		/* resident or non resident? */
> -		asize = QuadAlign(SIZEOF_RESIDENT + nsize);
> +		asize = ALIGN(SIZEOF_RESIDENT + nsize, 8);
>  		t16 = PtrOffset(rec, attr);
> 
>  		if (asize + t16 + 8 > sbi->record_size) {
> @@ -1509,7 +1509,7 @@ struct inode *ntfs_create_inode(struct user_namespace *mnt_userns,
>  				goto out5;
>  			}
> 
> -			asize = SIZEOF_NONRESIDENT + QuadAlign(err);
> +			asize = SIZEOF_NONRESIDENT + ALIGN(err, 8);
>  			inode->i_size = nsize;
>  		} else {
>  			attr->res.data_off = SIZEOF_RESIDENT_LE;
> diff --git a/fs/ntfs3/namei.c b/fs/ntfs3/namei.c
> index b1ccd66172f2..0626844e6bdc 100644
> --- a/fs/ntfs3/namei.c
> +++ b/fs/ntfs3/namei.c
> @@ -57,7 +57,7 @@ int fill_name_de(struct ntfs_sb_info *sbi, void *buf, const struct qstr *name,
>  	fname->type = FILE_NAME_POSIX;
>  	data_size = fname_full_size(fname);
> 
> -	e->size = cpu_to_le16(QuadAlign(data_size) + sizeof(struct NTFS_DE));
> +	e->size = cpu_to_le16(ALIGN(data_size, 8) + sizeof(struct NTFS_DE));
>  	e->key_size = cpu_to_le16(data_size);
>  	e->flags = 0;
>  	e->res = 0;
> diff --git a/fs/ntfs3/ntfs.h b/fs/ntfs3/ntfs.h
> index 40398e6c39c9..d6480845ce1f 100644
> --- a/fs/ntfs3/ntfs.h
> +++ b/fs/ntfs3/ntfs.h
> @@ -390,8 +390,8 @@ static inline u64 attr_ondisk_size(const struct ATTRIB *attr)
>  	return attr->non_res ? ((attr->flags &
>  				 (ATTR_FLAG_COMPRESSED | ATTR_FLAG_SPARSED)) ?
>  					le64_to_cpu(attr->nres.total_size) :
> -					le64_to_cpu(attr->nres.alloc_size)) :
> -			       QuadAlign(le32_to_cpu(attr->res.data_size));
> +					le64_to_cpu(attr->nres.alloc_size))
> +			     : ALIGN(le32_to_cpu(attr->res.data_size), 8);
>  }
> 
>  static inline u64 attr_size(const struct ATTRIB *attr)
> @@ -527,8 +527,8 @@ static_assert(sizeof(struct ATTR_LIST_ENTRY) == 0x20);
> 
>  static inline u32 le_size(u8 name_len)
>  {
> -	return QuadAlign(offsetof(struct ATTR_LIST_ENTRY, name) +
> -			 name_len * sizeof(short));
> +	return ALIGN(offsetof(struct ATTR_LIST_ENTRY, name) +
> +		     name_len * sizeof(short), 8);
>  }
> 
>  /* returns 0 if 'attr' has the same type and name */
> @@ -689,10 +689,10 @@ static inline bool de_has_vcn_ex(const struct NTFS_DE *e)
>  							sizeof(__le64)));
>  }
> 
> -#define MAX_BYTES_PER_NAME_ENTRY					       \
> -	QuadAlign(sizeof(struct NTFS_DE) +				       \
> -		  offsetof(struct ATTR_FILE_NAME, name) +		       \
> -		  NTFS_NAME_LEN * sizeof(short))
> +#define MAX_BYTES_PER_NAME_ENTRY \
> +	ALIGN(sizeof(struct NTFS_DE) + \
> +	      offsetof(struct ATTR_FILE_NAME, name) + \
> +	      NTFS_NAME_LEN * sizeof(short), 8)
> 
>  struct INDEX_HDR {
>  	__le32 de_off;	// 0x00: The offset from the start of this structure
> diff --git a/fs/ntfs3/ntfs_fs.h b/fs/ntfs3/ntfs_fs.h
> index 0c3ac89c3115..69bf1edc561e 100644
> --- a/fs/ntfs3/ntfs_fs.h
> +++ b/fs/ntfs3/ntfs_fs.h
> @@ -897,7 +897,7 @@ static inline bool run_is_empty(struct runs_tree *run)
>  /* NTFS uses quad aligned bitmaps */
>  static inline size_t bitmap_size(size_t bits)
>  {
> -	return QuadAlign((bits + 7) >> 3);
> +	return ALIGN((bits + 7) >> 3, 8);
>  }
> 
>  #define _100ns2seconds 10000000
> diff --git a/fs/ntfs3/record.c b/fs/ntfs3/record.c
> index 0d4a6251bddc..721c14f83e2b 100644
> --- a/fs/ntfs3/record.c
> +++ b/fs/ntfs3/record.c
> @@ -206,7 +206,7 @@ struct ATTRIB *mi_enum_attr(struct mft_inode *mi, struct ATTRIB *attr)
>  			return NULL;
> 
>  		if (off >= used || off < MFTRECORD_FIXUP_OFFSET_1 ||
> -		    !IsDwordAligned(off)) {
> +		    !IS_ALIGNED(off, 4)) {
>  			return NULL;
>  		}
> 
> @@ -235,7 +235,7 @@ struct ATTRIB *mi_enum_attr(struct mft_inode *mi, struct ATTRIB *attr)
> 
>  	/* Can we use the first field (attr->type) */
>  	if (off + 8 > used) {
> -		static_assert(QuadAlign(sizeof(enum ATTR_TYPE)) == 8);
> +		static_assert(ALIGN(sizeof(enum ATTR_TYPE), 8) == 8);
>  		return NULL;
>  	}
> 
> @@ -539,7 +539,7 @@ bool mi_resize_attr(struct mft_inode *mi, struct ATTRIB *attr, int bytes)
>  	next = Add2Ptr(attr, asize);
> 
>  	if (bytes > 0) {
> -		dsize = QuadAlign(bytes);
> +		dsize = ALIGN(bytes, 8);
>  		if (used + dsize > total)
>  			return false;
>  		nsize = asize + dsize;
> @@ -549,7 +549,7 @@ bool mi_resize_attr(struct mft_inode *mi, struct ATTRIB *attr, int bytes)
>  		used += dsize;
>  		rsize += dsize;
>  	} else {
> -		dsize = QuadAlign(-bytes);
> +		dsize = ALIGN(-bytes, 8);
>  		if (dsize > asize)
>  			return false;
>  		nsize = asize - dsize;
> @@ -596,7 +596,7 @@ int mi_pack_runs(struct mft_inode *mi, struct ATTRIB *attr,
>  		return err;
>  	}
> 
> -	new_run_size = QuadAlign(err);
> +	new_run_size = ALIGN(err, 8);
> 
>  	memmove(next + new_run_size - run_size, next + dsize, tail);
> 
> diff --git a/fs/ntfs3/super.c b/fs/ntfs3/super.c
> index 6be13e256c1a..1f4784f8bfaf 100644
> --- a/fs/ntfs3/super.c
> +++ b/fs/ntfs3/super.c
> @@ -808,9 +808,9 @@ static int ntfs_init_from_boot(struct super_block *sb, u32 sector_size,
>  	sbi->attr_size_tr = (5 * record_size >> 4); // ~320 bytes
> 
>  	sbi->max_bytes_per_attr =
> -		record_size - QuadAlign(MFTRECORD_FIXUP_OFFSET_1) -
> -		QuadAlign(((record_size >> SECTOR_SHIFT) * sizeof(short))) -
> -		QuadAlign(sizeof(enum ATTR_TYPE));
> +		record_size - ALIGN(MFTRECORD_FIXUP_OFFSET_1, 8) -
> +		ALIGN(((record_size >> SECTOR_SHIFT) * sizeof(short)), 8) -
> +		ALIGN(sizeof(enum ATTR_TYPE), 8);
> 
>  	sbi->index_size = boot->index_size < 0
>  				  ? 1u << (-boot->index_size)
> @@ -858,9 +858,9 @@ static int ntfs_init_from_boot(struct super_block *sb, u32 sector_size,
>  	rec->rhdr.fix_off = cpu_to_le16(MFTRECORD_FIXUP_OFFSET_1);
>  	fn = (sbi->record_size >> SECTOR_SHIFT) + 1;
>  	rec->rhdr.fix_num = cpu_to_le16(fn);
> -	ao = QuadAlign(MFTRECORD_FIXUP_OFFSET_1 + sizeof(short) * fn);
> +	ao = ALIGN(MFTRECORD_FIXUP_OFFSET_1 + sizeof(short) * fn, 8);
>  	rec->attr_off = cpu_to_le16(ao);
> -	rec->used = cpu_to_le32(ao + QuadAlign(sizeof(enum ATTR_TYPE)));
> +	rec->used = cpu_to_le32(ao + ALIGN(sizeof(enum ATTR_TYPE), 8));
>  	rec->total = cpu_to_le32(sbi->record_size);
>  	((struct ATTRIB *)Add2Ptr(rec, ao))->type = ATTR_END;
> 
> diff --git a/fs/ntfs3/xattr.c b/fs/ntfs3/xattr.c
> index 98871c895e77..6b17d46b9506 100644
> --- a/fs/ntfs3/xattr.c
> +++ b/fs/ntfs3/xattr.c
> @@ -26,9 +26,9 @@
>  static inline size_t unpacked_ea_size(const struct EA_FULL *ea)
>  {
>  	return ea->size ? le32_to_cpu(ea->size)
> -			: DwordAlign(struct_size(
> -				  ea, name,
> -				  1 + ea->name_len + le16_to_cpu(ea->elength)));
> +			: ALIGN(struct_size(
> +			      ea, name,
> +			      1 + ea->name_len + le16_to_cpu(ea->elength)), 4);
>  }
> 
>  static inline size_t packed_ea_size(const struct EA_FULL *ea)
> @@ -289,7 +289,7 @@ static noinline int ntfs_set_ea(struct inode *inode, const char *name,
>  		goto out;
>  	}
> 
> -	add = DwordAlign(struct_size(ea_all, name, 1 + name_len + val_size));
> +	add = ALIGN(struct_size(ea_all, name, 1 + name_len + val_size), 4);
> 
>  	err = ntfs_read_ea(ni, &ea_all, add, &info);
>  	if (err)
> --
> 2.25.1

Thanks, applied!

Best regards





[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux