Re: [PATCH] drivers/scsi/emcctd: drivers/scsi/emcctd: Client driver implementation for EMC-Symmetrix GuestOS emulated Cut-Through Device

Hi Maneesh,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.4 next-20160119]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Singhal-Maneesh/drivers-scsi-emcctd-drivers-scsi-emcctd-Client-driver-implementation-for-EMC-Symmetrix-GuestOS-emulated-Cut-Through-Device/20160119-200115
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: i386-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/scsi/emcctd/emcctd.c: In function 'ctd_proc_show':
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 3 has type 'int' [-Wformat=]
       "number interrupts: %ld\n"
       ^
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 5 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 6 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 7 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 8 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 9 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 10 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c:239:4: warning: format '%ld' expects argument of type 'long int', but argument 11 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c: In function 'ctd_scsi_response_sanity_check':
   drivers/scsi/emcctd/emcctd.c:655:10: warning: format '%ld' expects argument of type 'long int', but argument 7 has type 'int' [-Wformat=]
      ctd_dprintk_crit(
             ^
   drivers/scsi/emcctd/emcctd.c:655:10: warning: format '%ld' expects argument of type 'long int', but argument 8 has type 'int' [-Wformat=]
   drivers/scsi/emcctd/emcctd.c: In function 'ctd_handle_scsi_response':

quoted

drivers/scsi/emcctd/emcctd.c:878:30: warning: format '%lx' expects argument of type 'long unsigned int', but argument 7 has type 'int' [-Wformat=]

   drivers/scsi/emcctd/emcctd.c: In function 'ctd_pci_probe':

quoted

drivers/scsi/emcctd/emcctd.c:2588:10: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t {aka unsigned int}' [-Wformat=]

      ctd_dprintk_crit(
             ^
   drivers/scsi/emcctd/emcctd.c:2616:10: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
      ctd_dprintk_crit(
             ^
   drivers/scsi/emcctd/emcctd.c:2638:10: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type 'resource_size_t {aka unsigned int}' [-Wformat=]
      ctd_dprintk_crit(
             ^

vim +878 drivers/scsi/emcctd/emcctd.c

   862	
   863			memcpy(cmnd->sense_buffer, sense_data, sense_data_length);
   864	
   865			set_driver_byte(cmnd, DRIVER_SENSE);
   866			if (ctd_debug) {
   867				scsi_print_command(cmnd);
   868				scsi_print_result(cmnd, "emcctd sense", SUCCESS);
   869				scsi_print_sense(cmnd);
   870			}
   871		}
   872	
   873		if ((io_response->emc_ctd_v010_scsi_response_status ==
   874							SAM_STAT_TASK_SET_FULL) ||
   875			(io_response->emc_ctd_v010_scsi_response_status ==
   876								SAM_STAT_BUSY)) {
   877	
 > 878			ctd_dprintk(
   879				"QUEUE DEPTH change for channel:%d id:%d lun:%lld] active io count = %lx\n",
   880				cmnd->device->channel, cmnd->device->id,
   881				cmnd->device->lun,
   882				ctd_private->hw_stats.active_io_count.counter);
   883	
   884			scsi_track_queue_full(cmnd->device,
   885				ctd_private->hw_stats.active_io_count.counter);
   886		}

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