All of lore.kernel.org
 help / color / mirror / Atom feed
* ARCMSR - Notify sense data report - Lost patch?
@ 2014-11-24 11:47 Markus Stockhausen
  2014-11-24 13:23 ` Christoph Hellwig
  0 siblings, 1 reply; 9+ messages in thread
From: Markus Stockhausen @ 2014-11-24 11:47 UTC (permalink / raw)
  To: ching2048, thenzl; +Cc: linux-scsi

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

Hi,

sorting out some diffs between the official Arcea driver and 
the newest 3.18 patches I found a patch that might got lost:

https://lkml.org/lkml/2014/2/20/956

Could you check if that is desired?

Thanks in advance.

Markus
=

[-- Attachment #2: InterScan_Disclaimer.txt --]
[-- Type: text/plain, Size: 1650 bytes --]

****************************************************************************
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail
irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und
vernichten Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte
Weitergabe dieser Mail ist nicht gestattet.

Über das Internet versandte E-Mails können unter fremden Namen erstellt oder
manipuliert werden. Deshalb ist diese als E-Mail verschickte Nachricht keine
rechtsverbindliche Willenserklärung.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

Vorstand:
Kadir Akin
Dr. Michael Höhnerbach

Vorsitzender des Aufsichtsrates:
Hans Kristian Langva

Registergericht: Amtsgericht Köln
Registernummer: HRB 52 497

This e-mail may contain confidential and/or privileged information. If you
are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail. Any
unauthorized copying, disclosure or distribution of the material in this
e-mail is strictly forbidden.

e-mails sent over the internet may have been written under a wrong name or
been manipulated. That is why this message sent as an e-mail is not a
legally binding declaration of intention.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

executive board:
Kadir Akin
Dr. Michael Höhnerbach

President of the supervisory board:
Hans Kristian Langva

Registry office: district court Cologne
Register number: HRB 52 497

****************************************************************************

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

* Re: ARCMSR - Notify sense data report - Lost patch?
  2014-11-24 11:47 ARCMSR - Notify sense data report - Lost patch? Markus Stockhausen
@ 2014-11-24 13:23 ` Christoph Hellwig
  2014-11-24 13:45   ` AW: " Markus Stockhausen
  0 siblings, 1 reply; 9+ messages in thread
From: Christoph Hellwig @ 2014-11-24 13:23 UTC (permalink / raw)
  To: Markus Stockhausen; +Cc: ching2048, thenzl, linux-scsi

On Mon, Nov 24, 2014 at 11:47:02AM +0000, Markus Stockhausen wrote:
> Hi,
> 
> sorting out some diffs between the official Arcea driver and 
> the newest 3.18 patches I found a patch that might got lost:
> 
> https://lkml.org/lkml/2014/2/20/956
> 
> Could you check if that is desired?

Looks sensible to me.  Did you test it?


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

* AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-24 13:23 ` Christoph Hellwig
@ 2014-11-24 13:45   ` Markus Stockhausen
  2014-11-24 14:47     ` Tomas Henzl
  0 siblings, 1 reply; 9+ messages in thread
From: Markus Stockhausen @ 2014-11-24 13:45 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: ching2048, thenzl, linux-scsi

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

> Von: linux-scsi-owner@vger.kernel.org [linux-scsi-owner@vger.kernel.org]" im Auftrag von "Christoph Hellwig [hch@infradead.org]
> Gesendet: Montag, 24. November 2014 14:23
> An: Markus Stockhausen
> Cc: ching2048@areca.com.tw; thenzl@redhat.com; linux-scsi@vger.kernel.org
> Betreff: Re: ARCMSR - Notify sense data report - Lost patch?
> 
> On Mon, Nov 24, 2014 at 11:47:02AM +0000, Markus Stockhausen wrote:
> > Hi,
> >
> > sorting out some diffs between the official Arcea driver and
> > the newest 3.18 patches I found a patch that might got lost:
> >
> > https://lkml.org/lkml/2014/2/20/956
> >
> > Could you check if that is desired?
> 
> Looks sensible to me.  Did you test it?

No not really, as I don't know how. We had a controller crash the last
week and I was wondering about the differences between vendor and
upstream version. So I basically did some code comparison between 
the two source. Besides a lot of layout differences and other hacks 
this was one of the first things I noticed and wondered what difference 
those flags could make. Especially regarding stability.

Hopefully Ching and/or Tomas know better as they seem repsonsible 
for the rest of the 3.18 arcmsr patches.

Markus
=

[-- Attachment #2: InterScan_Disclaimer.txt --]
[-- Type: text/plain, Size: 1650 bytes --]

****************************************************************************
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail
irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und
vernichten Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte
Weitergabe dieser Mail ist nicht gestattet.

Über das Internet versandte E-Mails können unter fremden Namen erstellt oder
manipuliert werden. Deshalb ist diese als E-Mail verschickte Nachricht keine
rechtsverbindliche Willenserklärung.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

Vorstand:
Kadir Akin
Dr. Michael Höhnerbach

Vorsitzender des Aufsichtsrates:
Hans Kristian Langva

Registergericht: Amtsgericht Köln
Registernummer: HRB 52 497

This e-mail may contain confidential and/or privileged information. If you
are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail. Any
unauthorized copying, disclosure or distribution of the material in this
e-mail is strictly forbidden.

e-mails sent over the internet may have been written under a wrong name or
been manipulated. That is why this message sent as an e-mail is not a
legally binding declaration of intention.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

executive board:
Kadir Akin
Dr. Michael Höhnerbach

President of the supervisory board:
Hans Kristian Langva

Registry office: district court Cologne
Register number: HRB 52 497

****************************************************************************

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

* Re: AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-24 13:45   ` AW: " Markus Stockhausen
@ 2014-11-24 14:47     ` Tomas Henzl
  2014-11-24 18:25       ` AW: " Markus Stockhausen
  0 siblings, 1 reply; 9+ messages in thread
From: Tomas Henzl @ 2014-11-24 14:47 UTC (permalink / raw)
  To: Markus Stockhausen, Christoph Hellwig; +Cc: ching2048, linux-scsi

On 11/24/2014 02:45 PM, Markus Stockhausen wrote:
>> Von: linux-scsi-owner@vger.kernel.org [linux-scsi-owner@vger.kernel.org]" im Auftrag von "Christoph Hellwig [hch@infradead.org]
>> Gesendet: Montag, 24. November 2014 14:23
>> An: Markus Stockhausen
>> Cc: ching2048@areca.com.tw; thenzl@redhat.com; linux-scsi@vger.kernel.org
>> Betreff: Re: ARCMSR - Notify sense data report - Lost patch?
>>
>> On Mon, Nov 24, 2014 at 11:47:02AM +0000, Markus Stockhausen wrote:
>>> Hi,
>>>
>>> sorting out some diffs between the official Arcea driver and
>>> the newest 3.18 patches I found a patch that might got lost:
>>>
>>> https://lkml.org/lkml/2014/2/20/956
>>>
>>> Could you check if that is desired?
>> Looks sensible to me.  Did you test it?
> No not really, as I don't know how. We had a controller crash the last
> week and I was wondering about the differences between vendor and
> upstream version. So I basically did some code comparison between 
> the two source. Besides a lot of layout differences and other hacks 
> this was one of the first things I noticed and wondered what difference 
> those flags could make. Especially regarding stability.
>
> Hopefully Ching and/or Tomas know better as they seem repsonsible 
> for the rest of the 3.18 arcmsr patches.

I don't know why Ching hasn't re-posted this in later series.
The Areca in-house driver works stable?

>
> Markus


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

* AW: AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-24 14:47     ` Tomas Henzl
@ 2014-11-24 18:25       ` Markus Stockhausen
       [not found]         ` <CAKiknE87ac0yKzrvgiRj5fnq7OvdJSeVgYBLjt=rFYkjfS1yZg@mail.gmail.com>
  0 siblings, 1 reply; 9+ messages in thread
From: Markus Stockhausen @ 2014-11-24 18:25 UTC (permalink / raw)
  To: Tomas Henzl, Christoph Hellwig, ching2048; +Cc: linux-scsi

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

> Von: Tomas Henzl [thenzl@redhat.com]
> Gesendet: Montag, 24. November 2014 15:47
> An: Markus Stockhausen; Christoph Hellwig
> Cc: ching2048@areca.com.tw; linux-scsi@vger.kernel.org
> Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch?
> 
> On 11/24/2014 02:45 PM, Markus Stockhausen wrote:
> >> Von: linux-scsi-owner@vger.kernel.org [linux-scsi-owner@vger.kernel.org]&quot; im Auftrag von &quot;Christoph Hellwig [hch@infradead.org]
> >> Gesendet: Montag, 24. November 2014 14:23
> >> An: Markus Stockhausen
> >> Cc: ching2048@areca.com.tw; thenzl@redhat.com; linux-scsi@vger.kernel.org
> >> Betreff: Re: ARCMSR - Notify sense data report - Lost patch?
> >>
> >> On Mon, Nov 24, 2014 at 11:47:02AM +0000, Markus Stockhausen wrote:
> >>> Hi,
> >>>
> >>> sorting out some diffs between the official Arcea driver and
> >>> the newest 3.18 patches I found a patch that might got lost:
> >>>
> >>> https://lkml.org/lkml/2014/2/20/956
> >>>
> >>> Could you check if that is desired?
> >> Looks sensible to me.  Did you test it?
> > No not really, as I don't know how. We had a controller crash the last
> > week and I was wondering about the differences between vendor and
> > upstream version. So I basically did some code comparison between
> > the two source. Besides a lot of layout differences and other hacks
> > this was one of the first things I noticed and wondered what difference
> > those flags could make. Especially regarding stability.
> >
> > Hopefully Ching and/or Tomas know better as they seem repsonsible
> > for the rest of the 3.18 arcmsr patches.
> 
> I don't know why Ching hasn't re-posted this in later series.
> The Areca in-house driver works stable?

Sorry - No idea. We are on a stock kernel (3.10 longterm no vendor driver) 
and have to decide which way to go ... Sadly quite limited test possibilites. 
None of the two options feels 100% comfortable.

@Ching: Does any of the patches fit for stable?

Markus=

[-- Attachment #2: InterScan_Disclaimer.txt --]
[-- Type: text/plain, Size: 1650 bytes --]

****************************************************************************
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail
irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und
vernichten Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte
Weitergabe dieser Mail ist nicht gestattet.

Über das Internet versandte E-Mails können unter fremden Namen erstellt oder
manipuliert werden. Deshalb ist diese als E-Mail verschickte Nachricht keine
rechtsverbindliche Willenserklärung.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

Vorstand:
Kadir Akin
Dr. Michael Höhnerbach

Vorsitzender des Aufsichtsrates:
Hans Kristian Langva

Registergericht: Amtsgericht Köln
Registernummer: HRB 52 497

This e-mail may contain confidential and/or privileged information. If you
are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail. Any
unauthorized copying, disclosure or distribution of the material in this
e-mail is strictly forbidden.

e-mails sent over the internet may have been written under a wrong name or
been manipulated. That is why this message sent as an e-mail is not a
legally binding declaration of intention.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

executive board:
Kadir Akin
Dr. Michael Höhnerbach

President of the supervisory board:
Hans Kristian Langva

Registry office: district court Cologne
Register number: HRB 52 497

****************************************************************************

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

* AW: AW: ARCMSR - Notify sense data report - Lost patch?
       [not found]         ` <CAKiknE87ac0yKzrvgiRj5fnq7OvdJSeVgYBLjt=rFYkjfS1yZg@mail.gmail.com>
@ 2014-11-26 17:46           ` Markus Stockhausen
  2014-11-27 11:43             ` 黃清隆
  0 siblings, 1 reply; 9+ messages in thread
From: Markus Stockhausen @ 2014-11-26 17:46 UTC (permalink / raw)
  To: 黃清隆; +Cc: Tomas Henzl, Christoph Hellwig, linux-scsi

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

> Von: 黃清隆 [ching2048@areca.com.tw]
> Gesendet: Mittwoch, 26. November 2014 13:11
> An: Markus Stockhausen
> Cc: Tomas Henzl; Christoph Hellwig; linux-scsi@vger.kernel.org
> 
> Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch?
>
> Hi Markus,
> 
> Which version of arcmsr driver you are using?
> What model number of adapter you are using?

We are using a Areca 1261ML controller with the 3.12 stock drivers. 
So not the official Areca ones. From reading the change logs for 
3.18 I anticipate that the following fixes are important for stability
and might therefore be valid for -stable.

arcmsr: fix command timeout under heavy load
arcmsr: limit max. number of SCSI command request
arcmsr: return status of abort command

BTW: The same seems to apply for the lost patch about notify
sense data.

> Regards,
> 
> Ching



[-- Attachment #2: InterScan_Disclaimer.txt --]
[-- Type: text/plain, Size: 1650 bytes --]

****************************************************************************
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail
irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und
vernichten Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte
Weitergabe dieser Mail ist nicht gestattet.

Über das Internet versandte E-Mails können unter fremden Namen erstellt oder
manipuliert werden. Deshalb ist diese als E-Mail verschickte Nachricht keine
rechtsverbindliche Willenserklärung.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

Vorstand:
Kadir Akin
Dr. Michael Höhnerbach

Vorsitzender des Aufsichtsrates:
Hans Kristian Langva

Registergericht: Amtsgericht Köln
Registernummer: HRB 52 497

This e-mail may contain confidential and/or privileged information. If you
are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail. Any
unauthorized copying, disclosure or distribution of the material in this
e-mail is strictly forbidden.

e-mails sent over the internet may have been written under a wrong name or
been manipulated. That is why this message sent as an e-mail is not a
legally binding declaration of intention.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

executive board:
Kadir Akin
Dr. Michael Höhnerbach

President of the supervisory board:
Hans Kristian Langva

Registry office: district court Cologne
Register number: HRB 52 497

****************************************************************************

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

* Re: AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-26 17:46           ` Markus Stockhausen
@ 2014-11-27 11:43             ` 黃清隆
  2014-11-27 16:05               ` AW: " Markus Stockhausen
  0 siblings, 1 reply; 9+ messages in thread
From: 黃清隆 @ 2014-11-27 11:43 UTC (permalink / raw)
  To: Markus Stockhausen; +Cc: Tomas Henzl, Christoph Hellwig, linux-scsi

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

Hi Markus,

The attached source code has patched the notify sense data.
It has been tested from kernel 3.0 ~ 3.18.

Thanks,
Ching

2014-11-27 1:46 GMT+08:00 Markus Stockhausen <stockhausen@collogia.de>:
>> Von: 黃清隆 [ching2048@areca.com.tw]
>> Gesendet: Mittwoch, 26. November 2014 13:11
>> An: Markus Stockhausen
>> Cc: Tomas Henzl; Christoph Hellwig; linux-scsi@vger.kernel.org
>>
>> Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch?
>>
>> Hi Markus,
>>
>> Which version of arcmsr driver you are using?
>> What model number of adapter you are using?
>
> We are using a Areca 1261ML controller with the 3.12 stock drivers.
> So not the official Areca ones. From reading the change logs for
> 3.18 I anticipate that the following fixes are important for stability
> and might therefore be valid for -stable.
>
> arcmsr: fix command timeout under heavy load
> arcmsr: limit max. number of SCSI command request
> arcmsr: return status of abort command
>
> BTW: The same seems to apply for the lost patch about notify
> sense data.
>
>> Regards,
>>
>> Ching
>
>

[-- Attachment #2: arcmsr-20141127.zip --]
[-- Type: application/zip, Size: 34537 bytes --]

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

* AW: AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-27 11:43             ` 黃清隆
@ 2014-11-27 16:05               ` Markus Stockhausen
  2014-11-28  2:36                 ` 黃清隆
  0 siblings, 1 reply; 9+ messages in thread
From: Markus Stockhausen @ 2014-11-27 16:05 UTC (permalink / raw)
  To: 黃清隆; +Cc: Tomas Henzl, Christoph Hellwig, linux-scsi

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

> Von: 黃清隆 [ching2048@areca.com.tw]
> Gesendet: Donnerstag, 27. November 2014 12:43
> An: Markus Stockhausen
> Cc: Tomas Henzl; Christoph Hellwig; linux-scsi@vger.kernel.org
> Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch?
> 
> Hi Markus,
> 
> The attached source code has patched the notify sense data.
> It has been tested from kernel 3.0 ~ 3.18.
> 
> Thanks,
> Ching

Thanks for providing the source code. Nevertheless I would like to see
the patch https://lkml.org/lkml/2014/2/20/956 resubmitted to linux
standard kernel.

Will that be possible?

Markus


[-- Attachment #2: InterScan_Disclaimer.txt --]
[-- Type: text/plain, Size: 1650 bytes --]

****************************************************************************
Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte
Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail
irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und
vernichten Sie diese Mail. Das unerlaubte Kopieren sowie die unbefugte
Weitergabe dieser Mail ist nicht gestattet.

Über das Internet versandte E-Mails können unter fremden Namen erstellt oder
manipuliert werden. Deshalb ist diese als E-Mail verschickte Nachricht keine
rechtsverbindliche Willenserklärung.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

Vorstand:
Kadir Akin
Dr. Michael Höhnerbach

Vorsitzender des Aufsichtsrates:
Hans Kristian Langva

Registergericht: Amtsgericht Köln
Registernummer: HRB 52 497

This e-mail may contain confidential and/or privileged information. If you
are not the intended recipient (or have received this e-mail in error)
please notify the sender immediately and destroy this e-mail. Any
unauthorized copying, disclosure or distribution of the material in this
e-mail is strictly forbidden.

e-mails sent over the internet may have been written under a wrong name or
been manipulated. That is why this message sent as an e-mail is not a
legally binding declaration of intention.

Collogia
Unternehmensberatung AG
Ubierring 11
D-50678 Köln

executive board:
Kadir Akin
Dr. Michael Höhnerbach

President of the supervisory board:
Hans Kristian Langva

Registry office: district court Cologne
Register number: HRB 52 497

****************************************************************************

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

* Re: AW: ARCMSR - Notify sense data report - Lost patch?
  2014-11-27 16:05               ` AW: " Markus Stockhausen
@ 2014-11-28  2:36                 ` 黃清隆
  0 siblings, 0 replies; 9+ messages in thread
From: 黃清隆 @ 2014-11-28  2:36 UTC (permalink / raw)
  To: Markus Stockhausen; +Cc: Tomas Henzl, Christoph Hellwig, linux-scsi

Of course.
I shall submit this patch latter.

Thanks,
Ching

2014-11-28 0:05 GMT+08:00 Markus Stockhausen <stockhausen@collogia.de>:
>> Von: 黃清隆 [ching2048@areca.com.tw]
>> Gesendet: Donnerstag, 27. November 2014 12:43
>> An: Markus Stockhausen
>> Cc: Tomas Henzl; Christoph Hellwig; linux-scsi@vger.kernel.org
>> Betreff: Re: AW: ARCMSR - Notify sense data report - Lost patch?
>>
>> Hi Markus,
>>
>> The attached source code has patched the notify sense data.
>> It has been tested from kernel 3.0 ~ 3.18.
>>
>> Thanks,
>> Ching
>
> Thanks for providing the source code. Nevertheless I would like to see
> the patch https://lkml.org/lkml/2014/2/20/956 resubmitted to linux
> standard kernel.
>
> Will that be possible?
>
> Markus
>
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

end of thread, other threads:[~2014-11-28  2:36 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-11-24 11:47 ARCMSR - Notify sense data report - Lost patch? Markus Stockhausen
2014-11-24 13:23 ` Christoph Hellwig
2014-11-24 13:45   ` AW: " Markus Stockhausen
2014-11-24 14:47     ` Tomas Henzl
2014-11-24 18:25       ` AW: " Markus Stockhausen
     [not found]         ` <CAKiknE87ac0yKzrvgiRj5fnq7OvdJSeVgYBLjt=rFYkjfS1yZg@mail.gmail.com>
2014-11-26 17:46           ` Markus Stockhausen
2014-11-27 11:43             ` 黃清隆
2014-11-27 16:05               ` AW: " Markus Stockhausen
2014-11-28  2:36                 ` 黃清隆

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.