Re: [PATCH 3/5] staging: lustre: lustre: Remove unnecessary cast on void pointer

[kbuild test robot]

Hi simran,

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.10 next-20170303]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/simran-singhal/staging-nvec-Remove-unnecessary-cast-on-void-pointer/20170304-061120
config: m68k-allmodconfig (attached as .config)
compiler: m68k-linux-gcc (GCC) 4.9.0
reproduce:
        wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=m68k 

All warnings (new ones prefixed by >>):

   In file included from drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs.h:77:0,
                    from drivers/staging/lustre/lustre/mgc/../include/lprocfs_status.h:46,
                    from drivers/staging/lustre/lustre/mgc/mgc_request.c:42:
   drivers/staging/lustre/lustre/mgc/mgc_request.c: In function 'mgc_set_info_async':
   drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:216:16: warning: format '%s' expects argument of type 'char *', but argument 3 has type 'void *' [-Wformat=]
     static struct libcfs_debug_msg_data msgdata;   \
                   ^
>> drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:232:2: note: in expansion of macro '__CDEBUG'
     __CDEBUG(&cdls, mask, format, ## __VA_ARGS__);   \
     ^
   drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:236:29: note: in expansion of macro 'CDEBUG_LIMIT'
    #define CERROR(format, ...) CDEBUG_LIMIT(D_ERROR, format, ## __VA_ARGS__)
                                ^
   drivers/staging/lustre/lustre/mgc/mgc_request.c:1036:4: note: in expansion of macro 'CERROR'
       CERROR("invalid sptlrpc flavor %s to MGS\n",
       ^
   drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:216:16: warning: format '%s' expects argument of type 'char *', but argument 5 has type 'void *' [-Wformat=]
     static struct libcfs_debug_msg_data msgdata;   \
                   ^
>> drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:232:2: note: in expansion of macro '__CDEBUG'
     __CDEBUG(&cdls, mask, format, ## __VA_ARGS__);   \
     ^
   drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:243:49: note: in expansion of macro 'CDEBUG_LIMIT'
    #define LCONSOLE_ERROR_MSG(errnum, format, ...) CDEBUG_LIMIT(D_CONSOLE | D_ERROR, \
                                                    ^
   drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h:245:37: note: in expansion of macro 'LCONSOLE_ERROR_MSG'
    #define LCONSOLE_ERROR(format, ...) LCONSOLE_ERROR_MSG(0x00, format, ## __VA_ARGS__)
                                        ^
   drivers/staging/lustre/lustre/mgc/mgc_request.c:1052:4: note: in expansion of macro 'LCONSOLE_ERROR'
       LCONSOLE_ERROR("asking sptlrpc flavor %s to MGS but currently %s is in use\n",
       ^

vim +/__CDEBUG +232 drivers/staging/lustre/lustre/mgc/../include/../../include/linux/libcfs/libcfs_debug.h

d7e09d039 Peng Tao    2013-05-02  210  	return mask & D_CANTMASK ||
d7e09d039 Peng Tao    2013-05-02  211  		((libcfs_debug & mask) && (libcfs_subsystem_debug & subsystem));
d7e09d039 Peng Tao    2013-05-02  212  }
d7e09d039 Peng Tao    2013-05-02  213  
d7e09d039 Peng Tao    2013-05-02  214  #define __CDEBUG(cdls, mask, format, ...)				\
d7e09d039 Peng Tao    2013-05-02  215  do {									\
d7e09d039 Peng Tao    2013-05-02 @216  	static struct libcfs_debug_msg_data msgdata;			\
d7e09d039 Peng Tao    2013-05-02  217  									\
d7e09d039 Peng Tao    2013-05-02  218  	CFS_CHECK_STACK(&msgdata, mask, cdls);				\
d7e09d039 Peng Tao    2013-05-02  219  									\
d7e09d039 Peng Tao    2013-05-02  220  	if (cfs_cdebug_show(mask, DEBUG_SUBSYSTEM)) {			\
d7e09d039 Peng Tao    2013-05-02  221  		LIBCFS_DEBUG_MSG_DATA_INIT(&msgdata, mask, cdls);	\
d7e09d039 Peng Tao    2013-05-02  222  		libcfs_debug_msg(&msgdata, format, ## __VA_ARGS__);	\
d7e09d039 Peng Tao    2013-05-02  223  	}								\
d7e09d039 Peng Tao    2013-05-02  224  } while (0)
d7e09d039 Peng Tao    2013-05-02  225  
d7e09d039 Peng Tao    2013-05-02  226  #define CDEBUG(mask, format, ...) __CDEBUG(NULL, mask, format, ## __VA_ARGS__)
d7e09d039 Peng Tao    2013-05-02  227  
d7e09d039 Peng Tao    2013-05-02  228  #define CDEBUG_LIMIT(mask, format, ...)					\
d7e09d039 Peng Tao    2013-05-02  229  do {									\
a3ea59e0e Lisa Nguyen 2013-10-21  230  	static struct cfs_debug_limit_state cdls;			\
d7e09d039 Peng Tao    2013-05-02  231  									\
d7e09d039 Peng Tao    2013-05-02 @232  	__CDEBUG(&cdls, mask, format, ## __VA_ARGS__);			\
d7e09d039 Peng Tao    2013-05-02  233  } while (0)
d7e09d039 Peng Tao    2013-05-02  234  
d7e09d039 Peng Tao    2013-05-02  235  #define CWARN(format, ...)	CDEBUG_LIMIT(D_WARNING, format, ## __VA_ARGS__)

:::::: The code at line 232 was first introduced by commit
:::::: d7e09d0397e84eefbabfd9cb353221f3c6448d83 staging: add Lustre file system client support

:::::: TO: Peng Tao <bergwolf@xxxxxxxxx>
:::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip