Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 09:36:55PM +0100, Levente Kurusa wrote: > >1. > >Countering your nodisable comment, would it be valid to do: > >libata.force=2:disable libata.force=2.02:nodisable > >To disable all of port 2 except device 2? > > I think that makes sense and I support having 'nodisable'. I really don't want to. Let's please add whatever is necessary for fringe use cases but no more. It's not like this thing is generally useful. > >2. One of my friends wondered if it would be worthwhile to add force > >keywords for other HORKAGE bits, and if so, should the > >ata_lflag/ata_link force bits also be presented? > > I don't think so. Most of the other HORKAGEs are automatically > recognized and applied by the code. I think the only ones > which can cause trouble if not detected at first are the ones that are > currently in the list. Again, just whatever is immediately necessary, please. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 09:36:55PM +0100, Levente Kurusa wrote: 1. Countering your nodisable comment, would it be valid to do: libata.force=2:disable libata.force=2.02:nodisable To disable all of port 2 except device 2? I think that makes sense and I support having 'nodisable'. I really don't want to. Let's please add whatever is necessary for fringe use cases but no more. It's not like this thing is generally useful. 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? I don't think so. Most of the other HORKAGEs are automatically recognized and applied by the code. I think the only ones which can cause trouble if not detected at first are the ones that are currently in the list. Again, just whatever is immediately necessary, please. Thanks. -- tejun -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 09:36:55PM +0100, Levente Kurusa wrote: > > 2. One of my friends wondered if it would be worthwhile to add force > > keywords for other HORKAGE bits, and if so, should the > > ata_lflag/ata_link force bits also be presented? > I don't think so. Most of the other HORKAGEs are automatically > recognized and applied by the code. I think the only ones > which can cause trouble if not detected at first are the ones that are > currently in the list. His logic was thinking that it will aid debugging/testing on new buggy devices if the options are available at boot. I'd think of the following as candidates for that: ATA_HORKAGE_NODMA ATA_HORKAGE_MAX_SEC_128 ATA_HORKAGE_DIAGNOSTIC ATA_HORKAGE_BROKEN_HPA ATA_HORKAGE_DISABLE ATA_HORKAGE_HPA_SIZE ATA_HORKAGE_IVB ATA_HORKAGE_STUCK_ERR (only set by code presently, not by blacklist) ATA_HORKAGE_BRIDGE_OK ATA_HORKAGE_ATAPI_MOD16_DMA ATA_HORKAGE_NOSETXFER ATA_HORKAGE_MAX_SEC_LBA48 -- Robin Hugh Johnson Gentoo Linux: Developer, Infrastructure Lead E-Mail : robb...@gentoo.org GnuPG FP : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
Hi, Robin. 2013-12-12 21:22, Robin H. Johnson: On Thu, Dec 12, 2013 at 08:39:35AM -0500, Tejun Heo wrote: Hello, Robin. On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: + { "disable", .horkage_on = ATA_HORKAGE_DISABLE }, + { "nodisable",.horkage_off= ATA_HORKAGE_DISABLE }, Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need "nodisable". Let's just add "disable" for now. Can you please update the patch and resend? Before I do so, I have two questions: 1. Countering your nodisable comment, would it be valid to do: libata.force=2:disable libata.force=2.02:nodisable To disable all of port 2 except device 2? I think that makes sense and I support having 'nodisable'. 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? I don't think so. Most of the other HORKAGEs are automatically recognized and applied by the code. I think the only ones which can cause trouble if not detected at first are the ones that are currently in the list. -- Regards, Levente Kurusa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 08:39:35AM -0500, Tejun Heo wrote: > Hello, Robin. > > On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: > > + { "disable",.horkage_on = ATA_HORKAGE_DISABLE }, > > + { "nodisable", .horkage_off= ATA_HORKAGE_DISABLE }, > Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need > "nodisable". Let's just add "disable" for now. Can you please update > the patch and resend? Before I do so, I have two questions: 1. Countering your nodisable comment, would it be valid to do: libata.force=2:disable libata.force=2.02:nodisable To disable all of port 2 except device 2? 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? There are only 3 HORKAGE bits presently available in libata.force: ATA_HORKAGE_NONCQ ATA_HORKAGE_DUMP_ID ATA_HORKAGE_ATAPI_DMADIR And 3 ata_link flags: ATA_LFLAG_NO_HRST ATA_LFLAG_NO_SRST ATA_LFLAG_RST_ONCE -- Robin Hugh Johnson Gentoo Linux: Developer, Infrastructure Lead E-Mail : robb...@gentoo.org GnuPG FP : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
Hello, Robin. On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: > + { "disable",.horkage_on = ATA_HORKAGE_DISABLE }, > + { "nodisable", .horkage_off= ATA_HORKAGE_DISABLE }, Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need "nodisable". Let's just add "disable" for now. Can you please update the patch and resend? Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
Hello, Robin. On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: + { disable,.horkage_on = ATA_HORKAGE_DISABLE }, + { nodisable, .horkage_off= ATA_HORKAGE_DISABLE }, Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need nodisable. Let's just add disable for now. Can you please update the patch and resend? Thanks. -- tejun -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 08:39:35AM -0500, Tejun Heo wrote: Hello, Robin. On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: + { disable,.horkage_on = ATA_HORKAGE_DISABLE }, + { nodisable, .horkage_off= ATA_HORKAGE_DISABLE }, Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need nodisable. Let's just add disable for now. Can you please update the patch and resend? Before I do so, I have two questions: 1. Countering your nodisable comment, would it be valid to do: libata.force=2:disable libata.force=2.02:nodisable To disable all of port 2 except device 2? 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? There are only 3 HORKAGE bits presently available in libata.force: ATA_HORKAGE_NONCQ ATA_HORKAGE_DUMP_ID ATA_HORKAGE_ATAPI_DMADIR And 3 ata_link flags: ATA_LFLAG_NO_HRST ATA_LFLAG_NO_SRST ATA_LFLAG_RST_ONCE -- Robin Hugh Johnson Gentoo Linux: Developer, Infrastructure Lead E-Mail : robb...@gentoo.org GnuPG FP : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
Hi, Robin. 2013-12-12 21:22, Robin H. Johnson: On Thu, Dec 12, 2013 at 08:39:35AM -0500, Tejun Heo wrote: Hello, Robin. On Sat, Dec 07, 2013 at 04:56:27PM -0800, Robin H. Johnson wrote: + { disable, .horkage_on = ATA_HORKAGE_DISABLE }, + { nodisable,.horkage_off= ATA_HORKAGE_DISABLE }, Given the current usage of ATA_HORKAGE_DISABLE, I don't think we need nodisable. Let's just add disable for now. Can you please update the patch and resend? Before I do so, I have two questions: 1. Countering your nodisable comment, would it be valid to do: libata.force=2:disable libata.force=2.02:nodisable To disable all of port 2 except device 2? I think that makes sense and I support having 'nodisable'. 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? I don't think so. Most of the other HORKAGEs are automatically recognized and applied by the code. I think the only ones which can cause trouble if not detected at first are the ones that are currently in the list. -- Regards, Levente Kurusa -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] libata: provide the ability to disable a disk via the params.
On Thu, Dec 12, 2013 at 09:36:55PM +0100, Levente Kurusa wrote: 2. One of my friends wondered if it would be worthwhile to add force keywords for other HORKAGE bits, and if so, should the ata_lflag/ata_link force bits also be presented? I don't think so. Most of the other HORKAGEs are automatically recognized and applied by the code. I think the only ones which can cause trouble if not detected at first are the ones that are currently in the list. His logic was thinking that it will aid debugging/testing on new buggy devices if the options are available at boot. I'd think of the following as candidates for that: ATA_HORKAGE_NODMA ATA_HORKAGE_MAX_SEC_128 ATA_HORKAGE_DIAGNOSTIC ATA_HORKAGE_BROKEN_HPA ATA_HORKAGE_DISABLE ATA_HORKAGE_HPA_SIZE ATA_HORKAGE_IVB ATA_HORKAGE_STUCK_ERR (only set by code presently, not by blacklist) ATA_HORKAGE_BRIDGE_OK ATA_HORKAGE_ATAPI_MOD16_DMA ATA_HORKAGE_NOSETXFER ATA_HORKAGE_MAX_SEC_LBA48 -- Robin Hugh Johnson Gentoo Linux: Developer, Infrastructure Lead E-Mail : robb...@gentoo.org GnuPG FP : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/