Re: [PATCH] scsi: replace broken specification URL
- Original Message - > From: "Martin K. Petersen" > To: "Michael Opdenacker" > Cc: cor...@lwn.net, j...@linux.vnet.ibm.com, "martin petersen" > , > 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 > >>>>> 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
- 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
> "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
[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
- 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-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
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
> "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
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
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
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
[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. This replaces the now broken link with another public resource where the specifications can be found. Signed-off-by: Michael Opdenacker --- Documentation/DocBook/scsi.tmpl | 2 +- drivers/scsi/scsicam.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/DocBook/scsi.tmpl b/Documentation/DocBook/scsi.tmpl index 4b9b9b286cea..41ad3086b7a2 100644 --- a/Documentation/DocBook/scsi.tmpl +++ b/Documentation/DocBook/scsi.tmpl @@ -160,7 +160,7 @@ drivers/scsi/scsicam.c - SCSI + SCSI Common Access Method support functions, for use with HDIO_GETGEO, etc. diff --git a/drivers/scsi/scsicam.c b/drivers/scsi/scsicam.c index 910f4a7a3924..8144618fe2de 100644 --- a/drivers/scsi/scsicam.c +++ b/drivers/scsi/scsicam.c @@ -207,7 +207,7 @@ EXPORT_SYMBOL(scsi_partsize); * * WORKINGX3T9.2 * DRAFT792D - * see http://www.t10.org/ftp/t10/drafts/cam/cam-r12b.pdf + * see ftp://ftp.uni-duisburg.de/Hardware/SCSI/cam-r12b.pdf * *Revision 6 * 10-MAR-94 -- 2.7.4 -- 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
