mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2025-01-18 18:06:59 +07:00
orangefs: use ARRAY_SIZE
Using the ARRAY_SIZE macro improves the readability of the code. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Signed-off-by: Jérémy Lefaure <jeremy.lefaure@lse.epita.fr> Signed-off-by: Mike Marshall <hubcap@omnibond.com>
This commit is contained in:
parent
933f7ac1a1
commit
296200d3bb
@ -15,8 +15,10 @@
|
|||||||
|
|
||||||
#ifdef __KERNEL__
|
#ifdef __KERNEL__
|
||||||
#include <linux/types.h>
|
#include <linux/types.h>
|
||||||
|
#include <linux/kernel.h>
|
||||||
#else
|
#else
|
||||||
#include <stdint.h>
|
#include <stdint.h>
|
||||||
|
#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
#define GOSSIP_NO_DEBUG (__u64)0
|
#define GOSSIP_NO_DEBUG (__u64)0
|
||||||
@ -88,6 +90,6 @@ static struct __keyword_mask_s s_kmod_keyword_mask_map[] = {
|
|||||||
};
|
};
|
||||||
|
|
||||||
static const int num_kmod_keyword_mask_map = (int)
|
static const int num_kmod_keyword_mask_map = (int)
|
||||||
(sizeof(s_kmod_keyword_mask_map) / sizeof(struct __keyword_mask_s));
|
(ARRAY_SIZE(s_kmod_keyword_mask_map));
|
||||||
|
|
||||||
#endif /* __ORANGEFS_DEBUG_H */
|
#endif /* __ORANGEFS_DEBUG_H */
|
||||||
|
@ -4,6 +4,7 @@
|
|||||||
*
|
*
|
||||||
* See COPYING in top-level directory.
|
* See COPYING in top-level directory.
|
||||||
*/
|
*/
|
||||||
|
#include <linux/kernel.h>
|
||||||
#include "protocol.h"
|
#include "protocol.h"
|
||||||
#include "orangefs-kernel.h"
|
#include "orangefs-kernel.h"
|
||||||
#include "orangefs-dev-proto.h"
|
#include "orangefs-dev-proto.h"
|
||||||
@ -606,7 +607,7 @@ int orangefs_normalize_to_errno(__s32 error_code)
|
|||||||
/* Convert ORANGEFS encoded errno values into regular errno values. */
|
/* Convert ORANGEFS encoded errno values into regular errno values. */
|
||||||
} else if ((-error_code) & ORANGEFS_ERROR_BIT) {
|
} else if ((-error_code) & ORANGEFS_ERROR_BIT) {
|
||||||
i = (-error_code) & ~(ORANGEFS_ERROR_BIT|ORANGEFS_ERROR_CLASS_BITS);
|
i = (-error_code) & ~(ORANGEFS_ERROR_BIT|ORANGEFS_ERROR_CLASS_BITS);
|
||||||
if (i < sizeof(PINT_errno_mapping)/sizeof(*PINT_errno_mapping))
|
if (i < ARRAY_SIZE(PINT_errno_mapping))
|
||||||
error_code = -PINT_errno_mapping[i];
|
error_code = -PINT_errno_mapping[i];
|
||||||
else
|
else
|
||||||
error_code = -EINVAL;
|
error_code = -EINVAL;
|
||||||
|
Loading…
Reference in New Issue
Block a user