linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
@ 2013-11-25 22:01 Justin Piszcz
  2013-11-28 12:59 ` Martin K. Petersen
  0 siblings, 1 reply; 8+ messages in thread
From: Justin Piszcz @ 2013-11-25 22:01 UTC (permalink / raw)
  To: open list, linux-ext4

Hello,

Using 3.12.0 and ext4fs with 2 x SSDs in a RAID-1 configuration on a
3ware HW RAID card, no md/dm, I noticed the following recently:
[178339.353565] sda2: WRITE SAME failed. Manually zeroing.

It seems to be similar to this issue here:
http://permalink.gmane.org/gmane.linux.kernel/1494512

Relevant logs:
[178339.353519] 3w-sas: scsi0: ERROR: (0x03:0x0101): Invalid command
opcode:opcode=0x41.
[178339.353565] sda2: WRITE SAME failed. Manually zeroing.

Is this something I should be worried about?
I am running a database on the host, could that be the cause of it?
(in the related post, he was also running a DB).

Justin.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-11-25 22:01 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver Justin Piszcz
@ 2013-11-28 12:59 ` Martin K. Petersen
  2013-11-28 13:07   ` Justin Piszcz
  0 siblings, 1 reply; 8+ messages in thread
From: Martin K. Petersen @ 2013-11-28 12:59 UTC (permalink / raw)
  To: Justin Piszcz; +Cc: open list, linux-ext4, linux-scsi

>>>>> "Justin" == Justin Piszcz <jpiszcz@lucidpixels.com> writes:

Justin> Relevant logs: [178339.353519] 3w-sas: scsi0: ERROR:
Justin> (0x03:0x0101): Invalid command opcode:opcode=0x41.
Justin> [178339.353565] sda2: WRITE SAME failed. Manually zeroing.

Justin> Is this something I should be worried about? 

No, it just means that your 3ware controller does not handle the WRITE
SAME command and we fall back to manually writing out zeroes.

I posted a patch a while back that prevents WRITE SAME being issued for
controllers that do not pass the commands through to disk directly:

	http://marc.info/?l=linux-scsi&m=138252394614920&w=2

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-11-28 12:59 ` Martin K. Petersen
@ 2013-11-28 13:07   ` Justin Piszcz
  2013-11-28 13:13     ` Martin K. Petersen
  0 siblings, 1 reply; 8+ messages in thread
From: Justin Piszcz @ 2013-11-28 13:07 UTC (permalink / raw)
  To: 'Martin K. Petersen'; +Cc: 'open list', linux-ext4, linux-scsi



-----Original Message-----
From: Martin K. Petersen [mailto:martin.petersen@oracle.com] 
Sent: Thursday, November 28, 2013 8:00 AM
To: Justin Piszcz
Cc: open list; linux-ext4@vger.kernel.org; linux-scsi@vger.kernel.org
Subject: Re: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx
driver

>>>>> "Justin" == Justin Piszcz <jpiszcz@lucidpixels.com> writes:

Justin> Relevant logs: [178339.353519] 3w-sas: scsi0: ERROR:
Justin> (0x03:0x0101): Invalid command opcode:opcode=0x41.
Justin> [178339.353565] sda2: WRITE SAME failed. Manually zeroing.

Justin> Is this something I should be worried about? 

No, it just means that your 3ware controller does not handle the WRITE
SAME command and we fall back to manually writing out zeroes.

I posted a patch a while back that prevents WRITE SAME being issued for
controllers that do not pass the commands through to disk directly:

	http://marc.info/?l=linux-scsi&m=138252394614920&w=2

[ .. ]

Awesome, thank you!  This patch is over a month old, do you know if is
currently in mainline?  I'll also update my case with 3ware/LSI.

Justin.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-11-28 13:07   ` Justin Piszcz
@ 2013-11-28 13:13     ` Martin K. Petersen
  2013-11-28 21:52       ` Justin Piszcz
  0 siblings, 1 reply; 8+ messages in thread
From: Martin K. Petersen @ 2013-11-28 13:13 UTC (permalink / raw)
  To: Justin Piszcz
  Cc: 'Martin K. Petersen', 'open list',
	linux-ext4, linux-scsi

>>>>> "Justin" == Justin Piszcz <jpiszcz@lucidpixels.com> writes:

Martin> I posted a patch a while back that prevents WRITE SAME being
Martin> issued for controllers that do not pass the commands through to
Martin> disk directly:

Martin>	http://marc.info/?l=linux-scsi&m=138252394614920&w=2

Justin> Awesome, thank you!  This patch is over a month old, do you know
Justin> if is currently in mainline? 

Nope. - James?

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-11-28 13:13     ` Martin K. Petersen
@ 2013-11-28 21:52       ` Justin Piszcz
  2013-12-05 23:43         ` Justin Piszcz
  0 siblings, 1 reply; 8+ messages in thread
From: Justin Piszcz @ 2013-11-28 21:52 UTC (permalink / raw)
  To: 'Martin K. Petersen'
  Cc: 'open list', linux-ext4, linux-scsi, gregkh



-----Original Message-----
From: Martin K. Petersen [mailto:martin.petersen@oracle.com] 
Sent: Thursday, November 28, 2013 8:13 AM

[ .. ]

Martin>	http://marc.info/?l=linux-scsi&m=138252394614920&w=2

Justin> Awesome, thank you!  This patch is over a month old, do you know
Justin> if is currently in mainline? 

Nope. - James?

[ .. ]

Could it possibly get added to 3.12.x -stable as well? I've patched against
3.12.0 for now -- I'll let you know if there are any issues-- so far so
good, thanks again.

Justin.


^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-11-28 21:52       ` Justin Piszcz
@ 2013-12-05 23:43         ` Justin Piszcz
  2013-12-06  0:02           ` Greg KH
  2013-12-06  0:35           ` Martin K. Petersen
  0 siblings, 2 replies; 8+ messages in thread
From: Justin Piszcz @ 2013-12-05 23:43 UTC (permalink / raw)
  To: 'Martin K. Petersen'
  Cc: 'open list', linux-ext4, linux-scsi, gregkh



> -----Original Message-----
> From: Martin K. Petersen [mailto:martin.petersen@oracle.com]
> Sent: Thursday, November 28, 2013 8:13 AM
> 
> 
> Martin>	http://marc.info/?l=linux-scsi&m=138252394614920&w=2
> 
> Justin> Awesome, thank you!  This patch is over a month old, do you know
> Justin> if is currently in mainline?
> 
> Nope. - James?
> 
> Could it possibly get added to 3.12.x -stable as well? I've patched
against
> 3.12.0 for now -- I'll let you know if there are any issues-- so far so
> good, thanks again.
> 
> Justin.

Update:
System uptime with the patch is 7+ days under extensive load at times and
I've had no issues to date.
I just checked the current linux git repo (3.13-rc2) and found this patch
has not been included.

Please advise if any further testing or validation needs to be completed to
help get this into -stable/-mainline.

Justin.




^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-12-05 23:43         ` Justin Piszcz
@ 2013-12-06  0:02           ` Greg KH
  2013-12-06  0:35           ` Martin K. Petersen
  1 sibling, 0 replies; 8+ messages in thread
From: Greg KH @ 2013-12-06  0:02 UTC (permalink / raw)
  To: Justin Piszcz
  Cc: 'Martin K. Petersen', 'open list',
	linux-ext4, linux-scsi

On Thu, Dec 05, 2013 at 06:43:39PM -0500, Justin Piszcz wrote:
> 
> 
> > -----Original Message-----
> > From: Martin K. Petersen [mailto:martin.petersen@oracle.com]
> > Sent: Thursday, November 28, 2013 8:13 AM
> > 
> > 
> > Martin>	http://marc.info/?l=linux-scsi&m=138252394614920&w=2
> > 
> > Justin> Awesome, thank you!  This patch is over a month old, do you know
> > Justin> if is currently in mainline?
> > 
> > Nope. - James?
> > 
> > Could it possibly get added to 3.12.x -stable as well? I've patched
> against
> > 3.12.0 for now -- I'll let you know if there are any issues-- so far so
> > good, thanks again.
> > 
> > Justin.
> 
> Update:
> System uptime with the patch is 7+ days under extensive load at times and
> I've had no issues to date.
> I just checked the current linux git repo (3.13-rc2) and found this patch
> has not been included.
> 
> Please advise if any further testing or validation needs to be completed to
> help get this into -stable/-mainline.

It needs to get into Linus's tree first, after that, let
stable@vger.kernel.org know the git commit id and I'll be glad to queue
it up for a stable kernel release.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver
  2013-12-05 23:43         ` Justin Piszcz
  2013-12-06  0:02           ` Greg KH
@ 2013-12-06  0:35           ` Martin K. Petersen
  1 sibling, 0 replies; 8+ messages in thread
From: Martin K. Petersen @ 2013-12-06  0:35 UTC (permalink / raw)
  To: Justin Piszcz
  Cc: 'Martin K. Petersen', 'open list',
	linux-ext4, linux-scsi, gregkh

>>>>> "Justin" == Justin Piszcz <jpiszcz@lucidpixels.com> writes:

Justin> Update: System uptime with the patch is 7+ days under extensive
Justin> load at times and I've had no issues to date.  I just checked
Justin> the current linux git repo (3.13-rc2) and found this patch has
Justin> not been included.

James has it queued in scsi/fixes. It'll make it to Linus on the next
pull and it has been tagged for inclusion in stable.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2013-12-06  0:35 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-25 22:01 3.12.0: sda2: WRITE SAME failed. Manually zeroing. with 3w-xxxx driver Justin Piszcz
2013-11-28 12:59 ` Martin K. Petersen
2013-11-28 13:07   ` Justin Piszcz
2013-11-28 13:13     ` Martin K. Petersen
2013-11-28 21:52       ` Justin Piszcz
2013-12-05 23:43         ` Justin Piszcz
2013-12-06  0:02           ` Greg KH
2013-12-06  0:35           ` Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).