Re: [PATCH] OCFS2: use OCFS2_SEC_BITS in macro

From: Joseph Qi
Date: Sat Jan 25 2020 - 20:47:30 EST




On 20/1/21 11:37, Alex Shi wrote:
> This macros should be used as it.
>
> Signed-off-by: Alex Shi <alex.shi@xxxxxxxxxxxxxxxxx>
> Cc: Mark Fasheh <mark@xxxxxxxxxx>
> Cc: Joel Becker <jlbec@xxxxxxxxxxxx>
> Cc: Joseph Qi <joseph.qi@xxxxxxxxxxxxxxxxx>
> Cc: ocfs2-devel@xxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx

Acked-by: Joseph Qi <joseph.qi@xxxxxxxxxxxxxxxxx>
> ---
> fs/ocfs2/dlmglue.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ocfs2/dlmglue.c b/fs/ocfs2/dlmglue.c
> index cda1027d0819..0d6daf49b72a 100644
> --- a/fs/ocfs2/dlmglue.c
> +++ b/fs/ocfs2/dlmglue.c
> @@ -2133,7 +2133,7 @@ static void ocfs2_downconvert_on_unlock(struct ocfs2_super *osb,
> }
>
> #define OCFS2_SEC_BITS 34
> -#define OCFS2_SEC_SHIFT (64 - 34)
> +#define OCFS2_SEC_SHIFT (64 - OCFS2_SEC_BITS)
> #define OCFS2_NSEC_MASK ((1ULL << OCFS2_SEC_SHIFT) - 1)
>
> /* LVB only has room for 64 bits of time here so we pack it for
>