[lustre-devel] [PATCH 3/4] staging: lustre: workitem.c: replace IS_PO2 with is_power_of_2
Greg KH
gregkh at linuxfoundation.org
Fri Oct 16 22:40:49 PDT 2015
On Sat, Oct 17, 2015 at 12:06:59AM +0200, Aya Mahfouz wrote:
> Replaces IS_PO2 by is_power_of_2. IS_PO2 is used with several debug
> macros. In this case, it is CDEBUG. Note that the replacement changes
> the types involved, because the parameter of IS_PO2 is of type long long
> and the return type is int, while the parameter of is_power_of_2 is of
> type long and the return type is bool. This, however, has no impact,
> because the actual argument is always of type int, and the return value
> is always used as a boolean.
>
> Signed-off-by: Aya Mahfouz <mahfouz.saif.elyazal at gmail.com>
> ---
> drivers/staging/lustre/lustre/libcfs/workitem.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/lustre/lustre/libcfs/workitem.c b/drivers/staging/lustre/lustre/libcfs/workitem.c
> index e1143a5..377e1ea 100644
> --- a/drivers/staging/lustre/lustre/libcfs/workitem.c
> +++ b/drivers/staging/lustre/lustre/libcfs/workitem.c
> @@ -41,6 +41,8 @@
>
> #define DEBUG_SUBSYSTEM S_LNET
>
> +#include <linux/log2.h>
> +
> #include "../../include/linux/libcfs/libcfs.h"
>
> #define CFS_WS_NAME_LEN 16
> @@ -325,7 +327,7 @@ cfs_wi_sched_destroy(struct cfs_wi_sched *sched)
>
> spin_lock(&cfs_wi_data.wi_glock);
> while (sched->ws_nthreads > 0) {
> - CDEBUG(IS_PO2(++i) ? D_WARNING : D_NET,
> + CDEBUG(is_power_of_2(++i) ? D_WARNING : D_NET,
Same question about 0 here as well.
thanks,
greg k-h
More information about the lustre-devel
mailing list