Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1578159

Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup

From Dan Williams <dan.j.williams@intel.com>
Newsgroups linux.kernel
Subject Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup
Date 2017-02-10 04:30 +0100
Message-ID <t9a4h-6rR-7@gated-at.bofh.it> (permalink)
References (4 earlier) <t7JSy-vs-5@gated-at.bofh.it> <t81FM-3Jw-7@gated-at.bofh.it> <t86P7-71B-9@gated-at.bofh.it> <t86P7-71B-7@gated-at.bofh.it> <t8Ktc-7a8-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

On Wed, Feb 8, 2017 at 4:08 PM, James Bottomley
<James.Bottomley@hansenpartnership.com> wrote:
> On Mon, 2017-02-06 at 21:42 -0800, Dan Williams wrote:
[..]
>> ...but it reproduces on current mainline with the same config. I
>> haven't spotted what makes scsi_debug behave like this.
>
> Looking at the config, it's a static debug with report luns enabled.
>  Is it as simple as the fact that we probe lun 0 manually to see if the
> target exists, but then we don't account for the fact that we already
> did this, so if it turns up again in the report lun scan, we'll probe
> it again leading to a double add.  If that theory is correct, this may
> be the fix (compile tested only).
>
> James
>
> ---
>
> diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c
> index 6f7128f..ba4be08 100644
> --- a/drivers/scsi/scsi_scan.c
> +++ b/drivers/scsi/scsi_scan.c
> @@ -1441,6 +1441,10 @@ static int scsi_report_lun_scan(struct scsi_target *starget, int bflags,
>         for (lunp = &lun_data[1]; lunp <= &lun_data[num_luns]; lunp++) {
>                 lun = scsilun_to_int(lunp);
>
> +               if (lun == 0)
> +                       /* already scanned LUN 0 */
> +                       continue;
> +
>                 if (lun > sdev->host->max_lun) {
>                         sdev_printk(KERN_WARNING, sdev,
>                                     "lun%llu has a LUN larger than"

I gave this a shot on top of linux-next, but still hit the failure.
Log attached.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup Dan Williams <dan.j.williams@intel.com> - 2017-02-04 21:40 +0100
  Re: [lkp-robot] [scsi, block] 0dba1314d4:  WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-02-04 22:10 +0100
    Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup Dan Williams <dan.j.williams@intel.com> - 2017-02-04 22:10 +0100
  Re: [lkp-robot] [scsi, block] 0dba1314d4:         WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup Christoph Hellwig <hch@lst.de> - 2017-02-05 10:20 +0100
    Re: [lkp-robot] [scsi, block] 0dba1314d4:  WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-02-07 01:20 +0100
      Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup Dan Williams <dan.j.williams@intel.com> - 2017-02-07 06:50 +0100
        Re: [lkp-robot] [scsi, block] 0dba1314d4:  WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-02-09 01:10 +0100
          Re: [lkp-robot] [scsi, block] 0dba1314d4: WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup Dan Williams <dan.j.williams@intel.com> - 2017-02-10 04:30 +0100
      Re: [lkp-robot] [scsi, block] 0dba1314d4:  WARNING:at_fs/sysfs/dir.c:#sysfs_warn_dup James Bottomley <James.Bottomley@HansenPartnership.com> - 2017-02-11 17:10 +0100

csiph-web