All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michel Meyers <steltek@tcnnet.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>,
	Mike Christie <mchristi@redhat.com>
Cc: linux-scsi@vger.kernel.org
Subject: Re: [PATCH 1/1] scsi: add Synology DSM 6.0 and higher to 1024 sector blacklist
Date: Wed, 20 Apr 2016 10:46:45 +0200	[thread overview]
Message-ID: <571741F5.7060305@tcnnet.com> (raw)
In-Reply-To: <yq137qqzl6f.fsf@sermon.lab.mkp.net>

On 2016-04-12 21:22, Martin K. Petersen wrote:
>>>>>> "Mike" == Mike Christie <mchristi@redhat.com> writes:
> 
>>> Mike: Any preference?
> 
> Mike> I think quirking SYNOLOGY is best. It will be the safest route and
> Mike> work for all devices.
> 
> Michel: Please spin a new version of the patch.

Apologies for not getting back earlier. SYNOLOGY have released a new
version of their OS indicating that they have worked to fix the iSCSI
problem (and another one to actually make that update installable):

https://www.synology.com/en-global/releaseNote/RS10613xs+
- "Fixed multiple issues that might cause iSCSI service to hang under
heavy loading and performing Vmware VAAI commands."
- "Fixed an issue where DSM 6.0-7321 Update 1 could not be normally
installed with Auto-Update."

(They have not updated my support ticket on this case thus far.)

Both my SYNOLOGY appliances now seem to show correct vpd data when queried:

root@server:/# sg_vpd -p bl /dev/sdb
Block limits VPD page (SBC):
  Write same non-zero (WSNZ): 1
  Maximum compare and write length: 1 blocks
  Optimal transfer length granularity: 128 blocks
  Maximum transfer length: 8192 blocks
  Optimal transfer length: 1152 blocks
  Maximum prefetch length: 0 blocks

root@server:/# sg_vpd -p bl /dev/sdd
Block limits VPD page (SBC):
  Write same non-zero (WSNZ): 1
  Maximum compare and write length: 1 blocks
  Optimal transfer length granularity: 128 blocks
  Maximum transfer length: 8192 blocks
  Optimal transfer length: 512 blocks
  Maximum prefetch length: 0 blocks

I am currently running tests with an unmodified Debian 4.4.0-1-amd64
kernel, so unless I can reproduce the problem, I suggest we leave the
current blacklist as is (thereby only quirking the pre-6.0 SYNOLOGY
volumes that are affected) and discard the new patch.

--
Michel


  reply	other threads:[~2016-04-20  8:47 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20160114003437.354D28EE1D4@bedivere.hansenpartnership.com>
2016-04-11  8:43 ` [PATCH 1/1] scsi: add Synology DSM 6.0 and higher to 1024 sector blacklist Michel Meyers
2016-04-12  2:20   ` Martin K. Petersen
2016-04-12  8:28     ` Michel Meyers
2016-04-12 18:42       ` Martin K. Petersen
2016-04-12 19:18         ` Mike Christie
2016-04-12 19:22           ` Martin K. Petersen
2016-04-20  8:46             ` Michel Meyers [this message]
2016-04-26  2:12               ` Martin K. Petersen

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=571741F5.7060305@tcnnet.com \
    --to=steltek@tcnnet.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --cc=mchristi@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.