| Message ID | 20091125170948.5446.32754.sendpatchset@localhost |
|---|---|
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
On Wed, 25 Nov 2009 18:09:48 +0100 Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote: > From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > Subject: [PATCH] pata_pdc202xx_old: document known issues > > Document known issues with the driver to aid distribution makers, > users and developers in making informed decisions instead of wasting > their time needlessly. > > Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > --- > drivers/ata/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > Index: b/drivers/ata/Kconfig > =================================================================== > --- a/drivers/ata/Kconfig > +++ b/drivers/ata/Kconfig > @@ -583,6 +583,10 @@ config PATA_PDC_OLD > This option enables support for the Promise 20246, 20262, 20263, > 20265 and 20267 adapters. > > + Known issues: > + - UDMA transfers fail mysteriously on some chipsets > + - ATAPI DMA is unsupported currently Not sure this is useful, because the reports of UDMA failures are lower than most other reports. Should IPV6 have "known issues, mysterious timer list corruption" for example which occurs far more. Not do we list 'no atapi dma' in the help for the IDE SII driver ? Would be useful to fix (may actually just work now) but comments belong in the driver code. -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/25/2009 12:46 PM, Alan Cox wrote: > On Wed, 25 Nov 2009 18:09:48 +0100 > Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: > >> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> >> Subject: [PATCH] pata_pdc202xx_old: document known issues >> >> Document known issues with the driver to aid distribution makers, >> users and developers in making informed decisions instead of wasting >> their time needlessly. >> >> Signed-off-by: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> >> --- >> drivers/ata/Kconfig | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> Index: b/drivers/ata/Kconfig >> =================================================================== >> --- a/drivers/ata/Kconfig >> +++ b/drivers/ata/Kconfig >> @@ -583,6 +583,10 @@ config PATA_PDC_OLD >> This option enables support for the Promise 20246, 20262, 20263, >> 20265 and 20267 adapters. >> >> + Known issues: >> + - UDMA transfers fail mysteriously on some chipsets >> + - ATAPI DMA is unsupported currently > > Not sure this is useful, because the reports of UDMA failures are lower > than most other reports. Should IPV6 have "known issues, mysterious timer > list corruption" for example which occurs far more. Not do we list 'no > atapi dma' in the help for the IDE SII driver ? If the chip can support ATAPI DMI, but the driver does not, that deserves a comment, even if it's "hardware bugs prevent ATAPI DMA" or "ATAPI DMA would require much more code to support, so we did not bother for now" Ditto for things like useful ideas ("consider PIO-over-DMA in SiI 311x") and other would-be-nice-to-have ideas. These can serve as projects for newbies, or reminders for old-timers. Space in the driver source code comment header is cheap. Use liberally :) Write a Shakespearean soliloquy on a chip's hardware bugs, if you'd like. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 25 November 2009 06:50:26 pm Jeff Garzik wrote: > On 11/25/2009 12:46 PM, Alan Cox wrote: > > On Wed, 25 Nov 2009 18:09:48 +0100 > > Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: > > > >> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> > >> Subject: [PATCH] pata_pdc202xx_old: document known issues > >> > >> Document known issues with the driver to aid distribution makers, > >> users and developers in making informed decisions instead of wasting > >> their time needlessly. > >> > >> Signed-off-by: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> > >> --- > >> drivers/ata/Kconfig | 4 ++++ > >> 1 file changed, 4 insertions(+) > >> > >> Index: b/drivers/ata/Kconfig > >> =================================================================== > >> --- a/drivers/ata/Kconfig > >> +++ b/drivers/ata/Kconfig > >> @@ -583,6 +583,10 @@ config PATA_PDC_OLD > >> This option enables support for the Promise 20246, 20262, 20263, > >> 20265 and 20267 adapters. > >> > >> + Known issues: > >> + - UDMA transfers fail mysteriously on some chipsets > >> + - ATAPI DMA is unsupported currently > > > > Not sure this is useful, because the reports of UDMA failures are lower > > than most other reports. Should IPV6 have "known issues, mysterious timer > > list corruption" for example which occurs far more. Not do we list 'no > > atapi dma' in the help for the IDE SII driver ? > > If the chip can support ATAPI DMI, but the driver does not, that > deserves a comment, even if it's "hardware bugs prevent ATAPI DMA" or > "ATAPI DMA would require much more code to support, so we did not bother > for now" > > Ditto for things like useful ideas ("consider PIO-over-DMA in SiI 311x") > and other would-be-nice-to-have ideas. These can serve as projects for > newbies, or reminders for old-timers. The problem is that the old driver supported ATAPI DMA so people may have quite different expectations than in case of never-ever-implemented-ideas. -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/25/2009 12:52 PM, Bartlomiej Zolnierkiewicz wrote: > On Wednesday 25 November 2009 06:50:26 pm Jeff Garzik wrote: >> On 11/25/2009 12:46 PM, Alan Cox wrote: >>> On Wed, 25 Nov 2009 18:09:48 +0100 >>> Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: >>> >>>> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> >>>> Subject: [PATCH] pata_pdc202xx_old: document known issues >>>> >>>> Document known issues with the driver to aid distribution makers, >>>> users and developers in making informed decisions instead of wasting >>>> their time needlessly. >>>> >>>> Signed-off-by: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> >>>> --- >>>> drivers/ata/Kconfig | 4 ++++ >>>> 1 file changed, 4 insertions(+) >>>> >>>> Index: b/drivers/ata/Kconfig >>>> =================================================================== >>>> --- a/drivers/ata/Kconfig >>>> +++ b/drivers/ata/Kconfig >>>> @@ -583,6 +583,10 @@ config PATA_PDC_OLD >>>> This option enables support for the Promise 20246, 20262, 20263, >>>> 20265 and 20267 adapters. >>>> >>>> + Known issues: >>>> + - UDMA transfers fail mysteriously on some chipsets >>>> + - ATAPI DMA is unsupported currently >>> >>> Not sure this is useful, because the reports of UDMA failures are lower >>> than most other reports. Should IPV6 have "known issues, mysterious timer >>> list corruption" for example which occurs far more. Not do we list 'no >>> atapi dma' in the help for the IDE SII driver ? >> >> If the chip can support ATAPI DMI, but the driver does not, that >> deserves a comment, even if it's "hardware bugs prevent ATAPI DMA" or >> "ATAPI DMA would require much more code to support, so we did not bother >> for now" >> >> Ditto for things like useful ideas ("consider PIO-over-DMA in SiI 311x") >> and other would-be-nice-to-have ideas. These can serve as projects for >> newbies, or reminders for old-timers. > > The problem is that the old driver supported ATAPI DMA so people may have > quite different expectations than in case of never-ever-implemented-ideas. This is not rocket science :) Have one section "known issues" and another section "fun ideas to explore." This is English code comments, you may set any level of expectations. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> Space in the driver source code comment header is cheap. Use liberally > :) Write a Shakespearean soliloquy on a chip's hardware bugs, if you'd > like. [Don't tempt me, I've sneaked other little ditties and bits of filk into the kernel before now, but "Lost Andre" is probably a bit too long and lost on non-Hawkwind fans] This isnt the driver source code comment at the moment. Alan -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 25 November 2009 07:04:20 pm Jeff Garzik wrote: > On 11/25/2009 12:52 PM, Bartlomiej Zolnierkiewicz wrote: > > On Wednesday 25 November 2009 06:50:26 pm Jeff Garzik wrote: > >> On 11/25/2009 12:46 PM, Alan Cox wrote: > >>> On Wed, 25 Nov 2009 18:09:48 +0100 > >>> Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: > >>> > >>>> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> > >>>> Subject: [PATCH] pata_pdc202xx_old: document known issues > >>>> > >>>> Document known issues with the driver to aid distribution makers, > >>>> users and developers in making informed decisions instead of wasting > >>>> their time needlessly. > >>>> > >>>> Signed-off-by: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> > >>>> --- > >>>> drivers/ata/Kconfig | 4 ++++ > >>>> 1 file changed, 4 insertions(+) > >>>> > >>>> Index: b/drivers/ata/Kconfig > >>>> =================================================================== > >>>> --- a/drivers/ata/Kconfig > >>>> +++ b/drivers/ata/Kconfig > >>>> @@ -583,6 +583,10 @@ config PATA_PDC_OLD > >>>> This option enables support for the Promise 20246, 20262, 20263, > >>>> 20265 and 20267 adapters. > >>>> > >>>> + Known issues: > >>>> + - UDMA transfers fail mysteriously on some chipsets > >>>> + - ATAPI DMA is unsupported currently > >>> > >>> Not sure this is useful, because the reports of UDMA failures are lower > >>> than most other reports. Should IPV6 have "known issues, mysterious timer > >>> list corruption" for example which occurs far more. Not do we list 'no > >>> atapi dma' in the help for the IDE SII driver ? > >> > >> If the chip can support ATAPI DMI, but the driver does not, that > >> deserves a comment, even if it's "hardware bugs prevent ATAPI DMA" or > >> "ATAPI DMA would require much more code to support, so we did not bother > >> for now" > >> > >> Ditto for things like useful ideas ("consider PIO-over-DMA in SiI 311x") > >> and other would-be-nice-to-have ideas. These can serve as projects for > >> newbies, or reminders for old-timers. > > > > The problem is that the old driver supported ATAPI DMA so people may have > > quite different expectations than in case of never-ever-implemented-ideas. > > This is not rocket science :) Have one section "known issues" and > another section "fun ideas to explore." This is English code comments, > you may set any level of expectations. I think it is the best to leave up to the driver maintainer (once we find out who this person is, MAINTAINERS file still lacks info about PATA drivers in -rc8). -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/25/2009 01:29 PM, Bartlomiej Zolnierkiewicz wrote: > I think it is the best to leave up to the driver maintainer > (once we find out who this person is, MAINTAINERS file still > lacks info about PATA drivers in -rc8). The important thing is taking two minutes to write a relevant comment, not determining who is wearing a crown on his head. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 25 November 2009 09:38:27 pm Jeff Garzik wrote: > On 11/25/2009 01:29 PM, Bartlomiej Zolnierkiewicz wrote: > > I think it is the best to leave up to the driver maintainer > > (once we find out who this person is, MAINTAINERS file still > > lacks info about PATA drivers in -rc8). > > > The important thing is taking two minutes to write a relevant comment, If you want something changed in the code of pata_pdc202xx_old feel free to make said changes. I'm neither the author nor the maintainer of this driver. It will only take two minutes of your time anyway. > not determining who is wearing a crown on his head. From the fact that it takes years to make a tiny clarification for PATA support maintainer I'm guessing that by "a crown" you've actually meant "a crown of thorns".. -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/25/2009 05:02 PM, Bartlomiej Zolnierkiewicz wrote: > On Wednesday 25 November 2009 09:38:27 pm Jeff Garzik wrote: >> On 11/25/2009 01:29 PM, Bartlomiej Zolnierkiewicz wrote: >>> I think it is the best to leave up to the driver maintainer >>> (once we find out who this person is, MAINTAINERS file still >>> lacks info about PATA drivers in -rc8). >> >> >> The important thing is taking two minutes to write a relevant comment, > > If you want something changed in the code of pata_pdc202xx_old feel free to > make said changes. I'm neither the author nor the maintainer of this driver. You don't need to be the author or maintainer of a driver to submit changes. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thursday 26 November 2009 04:14:58 am Jeff Garzik wrote: > On 11/25/2009 05:02 PM, Bartlomiej Zolnierkiewicz wrote: > > On Wednesday 25 November 2009 09:38:27 pm Jeff Garzik wrote: > >> On 11/25/2009 01:29 PM, Bartlomiej Zolnierkiewicz wrote: > >>> I think it is the best to leave up to the driver maintainer > >>> (once we find out who this person is, MAINTAINERS file still > >>> lacks info about PATA drivers in -rc8). > >> > >> > >> The important thing is taking two minutes to write a relevant comment, > > > > If you want something changed in the code of pata_pdc202xx_old feel free to > > make said changes. I'm neither the author nor the maintainer of this driver. > > You don't need to be the author or maintainer of a driver to submit changes. I have neither any obligation nor any motivation to do it and you've already wasted much more than said two minutes for everybody. -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Index: b/drivers/ata/Kconfig =================================================================== --- a/drivers/ata/Kconfig +++ b/drivers/ata/Kconfig @@ -583,6 +583,10 @@ config PATA_PDC_OLD This option enables support for the Promise 20246, 20262, 20263, 20265 and 20267 adapters. + Known issues: + - UDMA transfers fail mysteriously on some chipsets + - ATAPI DMA is unsupported currently + If unsure, say N. config PATA_QDI