Re: [PATCH] scsi: replace broken specification URL

[Laurence Oberman]

- Original Message -
> From: "Martin K. Petersen" <martin.peter...@oracle.com>
> To: "Michael Opdenacker" <michael.opdenac...@free-electrons.com>
> Cc: cor...@lwn.net, j...@linux.vnet.ibm.com, "martin petersen" 
> <martin.peter...@oracle.com>,
> linux-doc@vger.kernel.org, linux-ker...@vger.kernel.org, 
> linux-s...@vger.kernel.org
> Sent: Thursday, September 15, 2016 10:06:19 AM
> Subject: Re: [PATCH] scsi: replace broken specification URL
> 
> >>>>> "Michael" == Michael Opdenacker <michael.opdenac...@free-electrons.com>
> >>>>> writes:
> 
> Michael> + * 'cam-r12b.pdf' document on http://www.t10.org/t10docs.htm
> Michael> + * (registration required)
> 
> That link really should be http://www.t10.org/drafts.htm. You can't look
> up draft specifications using the proposal document search form.
> 
> --
> Martin K. PetersenOracle Linux Engineering
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
I looked up the original URL when reviewing, and missed the Proposal Heading.
Checked the one Martin referred to and agree with Martin's change as well.



--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Laurence Oberman]

- Original Message -
> From: "Michael Opdenacker" 
> To: cor...@lwn.net, j...@linux.vnet.ibm.com, "martin petersen" 
> 
> Cc: linux-doc@vger.kernel.org, linux-ker...@vger.kernel.org, 
> linux-s...@vger.kernel.org, "Michael Opdenacker"
> 
> Sent: Thursday, September 15, 2016 9:03:05 AM
> Subject: [PATCH] scsi: replace broken specification URL
> 
> The t10.org website containing SCSI-2 draft specifications now requires
> to be from a member company to access the documents.
> 
> Signed-off-by: Michael Opdenacker 
> ---
>  Documentation/DocBook/scsi.tmpl | 6 +++---
>  drivers/scsi/scsicam.c  | 3 ++-
>  2 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/DocBook/scsi.tmpl
> b/Documentation/DocBook/scsi.tmpl
> index 4b9b9b286cea..b8b646426321 100644
> --- a/Documentation/DocBook/scsi.tmpl
> +++ b/Documentation/DocBook/scsi.tmpl
> @@ -160,9 +160,9 @@
>
>  drivers/scsi/scsicam.c
>  
> -   url='http://www.t10.org/ftp/t10/drafts/cam/cam-r12b.pdf'>SCSI
> -  Common Access Method support functions, for use with
> -  HDIO_GETGEO, etc.
> +  SCSI Common Access
> +   Method support functions ('cam-r12b.pdf' document,
> +   registration required), for use with HDIO_GETGEO, etc.
>  
>  !Edrivers/scsi/scsicam.c
>
> diff --git a/drivers/scsi/scsicam.c b/drivers/scsi/scsicam.c
> index 910f4a7a3924..5c446d9ef468 100644
> --- a/drivers/scsi/scsicam.c
> +++ b/drivers/scsi/scsicam.c
> @@ -207,7 +207,8 @@ EXPORT_SYMBOL(scsi_partsize);
>   *
>   * WORKINGX3T9.2
>   * DRAFT792D
> - * see http://www.t10.org/ftp/t10/drafts/cam/cam-r12b.pdf
> + * 'cam-r12b.pdf' document on http://www.t10.org/t10docs.htm
> + * (registration required)
>   *
>   *Revision 6
>   * 10-MAR-94
> --
> 2.7.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
Looks right matching what James wanted as well.
Reviewed-by: Laurence Oberman 
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Martin K. Petersen]

> "Michael" == Michael Opdenacker  
> writes:

Michael> + * 'cam-r12b.pdf' document on http://www.t10.org/t10docs.htm
Michael> + * (registration required)

That link really should be http://www.t10.org/drafts.htm. You can't look
up draft specifications using the proposal document search form.

-- 
Martin K. Petersen  Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Michael Opdenacker]

Hi Martin,

On 14/09/2016 19:00, Martin K. Petersen wrote:


Michael> So, should we only that the cam-r12b document can be found from
Michael> http://www.t10.org/t10docs.htm (registration required)?, and
Michael> tell that a copy can be found on
Michael> 
http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/?

As inconvenient as the T10/ANSI restrictions are, we should not be
linking to illegitimate spec repositories.


I agree with you. The documents do not allow copy without permission, so 
the copies we found are illegitimate indeed.


Now I know what patch to send, to get rid of the broken link :)

Thanks,

Michael.

--
Michael Opdenacker, CEO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Martin K. Petersen]

> "Michael" == Michael Opdenacker  
> writes:

Michael> So, should we only that the cam-r12b document can be found from
Michael> http://www.t10.org/t10docs.htm (registration required)?, and
Michael> tell that a copy can be found on
Michael> 
http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/?

As inconvenient as the T10/ANSI restrictions are, we should not be
linking to illegitimate spec repositories.

-- 
Martin K. Petersen  Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Tyrel Datwyler]

On 09/09/2016 01:16 AM, Michael Opdenacker wrote:
> Hi James,
> 
> Thank you very much for your help...
> 
> On 02/07/2016 16:49, James Bottomley wrote:
>> On Sat, 2016-07-02 at 08:56 +0200, Michael Opdenacker wrote:
>>> The t10.org website containing SCSI-2 draft specifications now
>>> requires to be from a member company to access the documents.
>>>
>>> This replaces the now broken link with another public resource
>>> where the specifications can be found.
>> Just because T10 implemented a pay wall for standards, doesn't mean
>> they're not still the definitive source.
>>
>> Adding a note about where you can get free versions is a useful
>> service, please do, but we have to keep the official links.  To be
>> honest the Duisberg site doesn't seem useful because it only has the
>> CAM standard.
> 
> Understood. I found another location where all the documents seem to be 
> available:
> http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/

This link is just a blank page with the CSIT background image when I
follow it. This worked for me though:

http://www.csit-sun.pub.ro/~cpop/?dir=./Documentatie_SMP/Standarde_magistrale/SCSI

-Tyrel

>>
>> The Wayback machine is more useful because it keeps a copy of the site
>> (with the attached standards) just before the paywall went up:
>>
>> https://web.archive.org/web/20080828112749/http://t10.org/drafts.htm
> 
> However, the PDF file from 
> https://web.archive.org/web/20080828112749/http://t10.org/ftp/t10/drafts/cam/cam-r12b.pdf
>  
> fails to load at a 130810 byte limit. Other people have reported a 
> similar file size issue in the past.
> 
> So, should we only that the cam-r12b document can be found from 
> http://www.t10.org/t10docs.htm (registration required)?, and tell that a 
> copy can be found on 
> http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/?
> 
> I'm trying to fix broken links in kernel documentation, which I publish 
> on http://free-electrons.com/kerneldoc/ . I have a broken link checker 
> for the http://free-electrons.com/ website, and it finds all the broken 
> links on http://free-electrons.com/kerneldoc/ . That's a good thing, 
> isn't need, but it means I have to get rid of the broken links :)
> 
> Thanks again for your help,
> 
> Cheers,
> 
> Michael.
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[Michael Opdenacker]

Hi James,

Thank you very much for your help...

On 02/07/2016 16:49, James Bottomley wrote:

On Sat, 2016-07-02 at 08:56 +0200, Michael Opdenacker wrote:

The t10.org website containing SCSI-2 draft specifications now
requires to be from a member company to access the documents.

This replaces the now broken link with another public resource
where the specifications can be found.

Just because T10 implemented a pay wall for standards, doesn't mean
they're not still the definitive source.

Adding a note about where you can get free versions is a useful
service, please do, but we have to keep the official links.  To be
honest the Duisberg site doesn't seem useful because it only has the
CAM standard.


Understood. I found another location where all the documents seem to be 
available:

http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/


The Wayback machine is more useful because it keeps a copy of the site
(with the attached standards) just before the paywall went up:

https://web.archive.org/web/20080828112749/http://t10.org/drafts.htm


However, the PDF file from 
https://web.archive.org/web/20080828112749/http://t10.org/ftp/t10/drafts/cam/cam-r12b.pdf 
fails to load at a 130810 byte limit. Other people have reported a 
similar file size issue in the past.


So, should we only that the cam-r12b document can be found from 
http://www.t10.org/t10docs.htm (registration required)?, and tell that a 
copy can be found on 
http://www.csit-sun.pub.ro/~cpop/Documentatie_SMP/Standarde_magistrale/SCSI/?


I'm trying to fix broken links in kernel documentation, which I publish 
on http://free-electrons.com/kerneldoc/ . I have a broken link checker 
for the http://free-electrons.com/ website, and it finds all the broken 
links on http://free-electrons.com/kerneldoc/ . That's a good thing, 
isn't need, but it means I have to get rid of the broken links :)


Thanks again for your help,

Cheers,

Michael.

--
Michael Opdenacker, CEO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] scsi: replace broken specification URL

[James Bottomley]

On Sat, 2016-07-02 at 08:56 +0200, Michael Opdenacker wrote:
> The t10.org website containing SCSI-2 draft specifications now 
> requires to be from a member company to access the documents.
> 
> This replaces the now broken link with another public resource
> where the specifications can be found.

Just because T10 implemented a pay wall for standards, doesn't mean
they're not still the definitive source.

Adding a note about where you can get free versions is a useful
service, please do, but we have to keep the official links.  To be
honest the Duisberg site doesn't seem useful because it only has the
CAM standard.

The Wayback machine is more useful because it keeps a copy of the site
(with the attached standards) just before the paywall went up:

https://web.archive.org/web/20080828112749/http://t10.org/drafts.htm

But that only really has pre 2008 standards.

James

--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html