public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Don Brace <don.brace@pmcs.com>
Cc: kbuild-all@01.org, Viswas.G@pmcs.com, scott.teel@pmcs.com,
	Kevin.Barnett@pmcs.com, scott.benesh@pmcs.com,
	Mahesh.Rajashekhara@pmcs.com, hch@infradead.org,
	Justin.Lindley@pmcs.com, elliott@hpe.com,
	james.bottomley@parallels.com, linux-scsi@vger.kernel.org
Subject: Re: [PATCH v2 3/3] hpsa: add box and bay information for enclosure devices
Date: Tue, 22 Dec 2015 01:47:48 +0800	[thread overview]
Message-ID: <201512220129.CV5tgYsh%fengguang.wu@intel.com> (raw)
In-Reply-To: <20151221172146.14423.43517.stgit@brunhilda>

[-- Attachment #1: Type: text/plain, Size: 16112 bytes --]

Hi Don,

[auto build test ERROR on scsi/for-next]
[also build test ERROR on v4.4-rc6 next-20151221]

url:    https://github.com/0day-ci/linux/commits/Don-Brace/hpsa-updates/20151222-012604
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: i386-randconfig-x005-201551 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All error/warnings (new ones prefixed by >>):

   drivers/scsi/hpsa.c: In function 'hpsa_get_enclosure_info':
   drivers/scsi/hpsa.c:3201:2: error: parameter 'rc' is initialized
     int rc = -1;
     ^
   drivers/scsi/hpsa.c:3202:9: error: parameter 'c' is initialized
     struct CommandList *c = NULL;
            ^
   drivers/scsi/hpsa.c:3203:9: error: parameter 'ei' is initialized
     struct ErrorInfo *ei = NULL;
            ^
   drivers/scsi/hpsa.c:3204:9: error: parameter 'bssbp' is initialized
     struct bmic_sense_storage_box_params *bssbp = NULL;
            ^
   drivers/scsi/hpsa.c:3205:9: error: parameter 'id_phys' is initialized
     struct bmic_identify_physical_device *id_phys = NULL;
            ^
   drivers/scsi/hpsa.c:3206:9: error: parameter 'rle' is initialized
     struct ext_report_lun_entry *rle = &rlep->LUN[rle_index];
            ^
   drivers/scsi/hpsa.c:3206:38: error: 'rlep' undeclared (first use in this function)
     struct ext_report_lun_entry *rle = &rlep->LUN[rle_index];
                                         ^
   drivers/scsi/hpsa.c:3206:38: note: each undeclared identifier is reported only once for each function it appears in
   drivers/scsi/hpsa.c:3206:48: error: 'rle_index' undeclared (first use in this function)
     struct ext_report_lun_entry *rle = &rlep->LUN[rle_index];
                                                   ^
   drivers/scsi/hpsa.c:3207:2: error: parameter 'bmic_device_index' is initialized
     u16 bmic_device_index = 0;
     ^
   drivers/scsi/hpsa.c:3209:2: error: expected declaration specifiers before 'bmic_device_index'
     bmic_device_index = GET_BMIC_DRIVE_NUMBER(&rle->lunid[0]);
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
>> drivers/scsi/hpsa.c:3211:2: note: in expansion of macro 'if'
     if (bmic_device_index == 0xFF00)
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
>> drivers/scsi/hpsa.c:3211:2: note: in expansion of macro 'if'
     if (bmic_device_index == 0xFF00)
     ^
   drivers/scsi/hpsa.c:3214:2: error: expected declaration specifiers before 'bssbp'
     bssbp = kzalloc(sizeof(*bssbp), GFP_KERNEL);
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3215:2: note: in expansion of macro 'if'
     if (!bssbp)
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3215:2: note: in expansion of macro 'if'
     if (!bssbp)
     ^
   drivers/scsi/hpsa.c:3218:2: error: expected declaration specifiers before 'id_phys'
     id_phys = kzalloc(sizeof(*id_phys), GFP_KERNEL);
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3219:2: note: in expansion of macro 'if'
     if (!id_phys)
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3219:2: note: in expansion of macro 'if'
     if (!id_phys)
     ^
   drivers/scsi/hpsa.c:3222:2: error: expected declaration specifiers before 'rc'
     rc = hpsa_bmic_id_physical_device(h, scsi3addr, bmic_device_index,
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3224:2: note: in expansion of macro 'if'
     if (rc) {
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3224:2: note: in expansion of macro 'if'
     if (rc) {
     ^
   drivers/scsi/hpsa.c:3230:2: error: expected declaration specifiers before 'c'
     c = cmd_alloc(h);
     ^
   drivers/scsi/hpsa.c:3232:2: error: expected declaration specifiers before 'rc'
     rc = fill_cmd(c, BMIC_SENSE_STORAGE_BOX_PARAMS, h, bssbp,
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3235:2: note: in expansion of macro 'if'
     if (rc)
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3235:2: note: in expansion of macro 'if'
     if (rc)
     ^
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3238:2: note: in expansion of macro 'if'
     if (id_phys->phys_connector[1] == 'E')
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3238:2: note: in expansion of macro 'if'
     if (id_phys->phys_connector[1] == 'E')
     ^
   drivers/scsi/hpsa.c:3240:2: error: expected declaration specifiers before 'else'
     else
     ^
   drivers/scsi/hpsa.c:3243:2: error: expected declaration specifiers before 'rc'
     rc = hpsa_scsi_do_simple_cmd_with_retry(h, c, PCI_DMA_FROMDEVICE,
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3245:2: note: in expansion of macro 'if'
     if (rc)
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3245:2: note: in expansion of macro 'if'
     if (rc)
     ^
   drivers/scsi/hpsa.c:3248:2: error: expected declaration specifiers before 'ei'
     ei = c->err_info;
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> include/linux/compiler.h:147:2: error: expected declaration specifiers before 'if'
     if (__builtin_constant_p((cond)) ? !!(cond) :   \
     ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3249:2: note: in expansion of macro 'if'
     if (ei->CommandStatus != 0 && ei->CommandStatus != CMD_DATA_UNDERRUN) {
     ^
>> include/linux/compiler.h:161:3: error: expected declaration specifiers before ')' token
     }))
      ^
   include/linux/compiler.h:145:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   drivers/scsi/hpsa.c:3249:2: note: in expansion of macro 'if'
     if (ei->CommandStatus != 0 && ei->CommandStatus != CMD_DATA_UNDERRUN) {
     ^
   drivers/scsi/hpsa.c:3254:2: error: expected declaration specifiers before 'encl_dev'
     encl_dev->box[id_phys->active_path_number] = bssbp->phys_box_on_port;
     ^
   In file included from arch/x86/include/asm/string.h:2:0,
                    from include/linux/string.h:17,
                    from include/linux/dynamic_debug.h:111,
                    from include/linux/printk.h:277,
                    from include/linux/kernel.h:13,
                    from include/linux/list.h:8,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:
>> arch/x86/include/asm/string_32.h:182:25: error: expected declaration specifiers before '__builtin_memcpy'
    #define memcpy(t, f, n) __builtin_memcpy(t, f, n)
                            ^
>> drivers/scsi/hpsa.c:3255:2: note: in expansion of macro 'memcpy'
     memcpy(&encl_dev->phys_connector[id_phys->active_path_number],
     ^
   drivers/scsi/hpsa.c:3258:2: error: expected declaration specifiers before 'rc'
     rc = IO_OK;
     ^
   drivers/scsi/hpsa.c:3259:1: error: expected declaration specifiers before 'out'
    out:
    ^
   drivers/scsi/hpsa.c:3261:2: error: expected declaration specifiers before 'kfree'
     kfree(id_phys);
     ^
   In file included from include/uapi/linux/stddef.h:1:0,
                    from include/linux/stddef.h:4,
                    from include/uapi/linux/posix_types.h:4,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/hpsa.c:19:

vim +/if +3211 drivers/scsi/hpsa.c

  3205		struct bmic_identify_physical_device *id_phys = NULL;
  3206		struct ext_report_lun_entry *rle = &rlep->LUN[rle_index];
  3207		u16 bmic_device_index = 0;
  3208	
  3209		bmic_device_index = GET_BMIC_DRIVE_NUMBER(&rle->lunid[0]);
  3210	
> 3211		if (bmic_device_index == 0xFF00)
  3212			goto out;
  3213	
  3214		bssbp = kzalloc(sizeof(*bssbp), GFP_KERNEL);
  3215		if (!bssbp)
  3216			goto out;
  3217	
  3218		id_phys = kzalloc(sizeof(*id_phys), GFP_KERNEL);
  3219		if (!id_phys)
  3220			goto out;
  3221	
  3222		rc = hpsa_bmic_id_physical_device(h, scsi3addr, bmic_device_index,
  3223							id_phys, sizeof(*id_phys));
  3224		if (rc) {
  3225			dev_warn(&h->pdev->dev, "%s: id_phys failed %d bdi[0x%x]\n",
  3226				__func__, encl_dev->external, bmic_device_index);
  3227			goto out;
  3228		}
  3229	
  3230		c = cmd_alloc(h);
  3231	
  3232		rc = fill_cmd(c, BMIC_SENSE_STORAGE_BOX_PARAMS, h, bssbp,
  3233				sizeof(*bssbp), 0, RAID_CTLR_LUNID, TYPE_CMD);
  3234	
  3235		if (rc)
  3236			goto out;
  3237	
  3238		if (id_phys->phys_connector[1] == 'E')
  3239			c->Request.CDB[5] = id_phys->box_index;
  3240		else
  3241			c->Request.CDB[5] = 0;
  3242	
  3243		rc = hpsa_scsi_do_simple_cmd_with_retry(h, c, PCI_DMA_FROMDEVICE,
  3244							NO_TIMEOUT);
  3245		if (rc)
  3246			goto out;
  3247	
  3248		ei = c->err_info;
  3249		if (ei->CommandStatus != 0 && ei->CommandStatus != CMD_DATA_UNDERRUN) {
  3250			rc = -1;
  3251			goto out;
  3252		}
  3253	
  3254		encl_dev->box[id_phys->active_path_number] = bssbp->phys_box_on_port;
> 3255		memcpy(&encl_dev->phys_connector[id_phys->active_path_number],
  3256			bssbp->phys_connector, sizeof(bssbp->phys_connector));
  3257	
  3258		rc = IO_OK;

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 25086 bytes --]

  reply	other threads:[~2015-12-21 17:45 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-21 17:21 [PATCH v2 0/3] hpsa updates Don Brace
2015-12-21 17:21 ` [PATCH v2 1/3] hpsa: fix path_info_show Don Brace
2015-12-22  4:55   ` Matthew R. Ochs
2015-12-21 17:21 ` [PATCH v2 2/3] hpsa: change SAS transport devices to bus 0 Don Brace
2015-12-21 17:21 ` [PATCH v2 3/3] hpsa: add box and bay information for enclosure devices Don Brace
2015-12-21 17:47   ` kbuild test robot [this message]
2015-12-21 17:59   ` kbuild test robot
2015-12-22  4:58   ` Matthew R. Ochs

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201512220129.CV5tgYsh%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=Justin.Lindley@pmcs.com \
    --cc=Kevin.Barnett@pmcs.com \
    --cc=Mahesh.Rajashekhara@pmcs.com \
    --cc=Viswas.G@pmcs.com \
    --cc=don.brace@pmcs.com \
    --cc=elliott@hpe.com \
    --cc=hch@infradead.org \
    --cc=james.bottomley@parallels.com \
    --cc=kbuild-all@01.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=scott.benesh@pmcs.com \
    --cc=scott.teel@pmcs.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox