[PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
From: Ashish Kalra Date: Wed, 29 Jul 2009 21:15:49 +0530 Fix for non-ncq & ncq commands causing timeouts when both are issued simultaneously to the same device. Signed-off-by: Ashish Kalra --- drivers/ata/sata_fsl.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 5a88b44..a33f130 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c @@ -1262,6 +1262,7 @@ static struct scsi_host_template sata_fsl_sht = { static struct ata_port_operations sata_fsl_ops = { .inherits = &sata_pmp_port_ops, + .qc_defer = ata_std_qc_defer; .qc_prep = sata_fsl_qc_prep, .qc_issue = sata_fsl_qc_issue, .qc_fill_rtf = sata_fsl_qc_fill_rtf, -- 1.6.0 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
On 07/29/2009 10:03 AM, ashish kalra wrote: From: Ashish Kalra Date: Wed, 29 Jul 2009 21:15:49 +0530 Fix for non-ncq & ncq commands causing timeouts when both are issued simultaneously to the same device. Signed-off-by: Ashish Kalra --- drivers/ata/sata_fsl.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 5a88b44..a33f130 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c @@ -1262,6 +1262,7 @@ static struct scsi_host_template sata_fsl_sht = { static struct ata_port_operations sata_fsl_ops = { .inherits = &sata_pmp_port_ops, + .qc_defer = ata_std_qc_defer; .qc_prep = sata_fsl_qc_prep, .qc_issue = sata_fsl_qc_issue, .qc_fill_rtf = sata_fsl_qc_fill_rtf, This doesn't look like it should change anything. sata_fsl_ops inherits from sata_pmp_port_ops, which inherits from sata_port_ops, which already sets qc_defer to ata_std_qc_defer. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
Hello, >> Signed-off-by: Ashish Kalra >> --- >> drivers/ata/sata_fsl.c | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c >> index 5a88b44..a33f130 100644 >> --- a/drivers/ata/sata_fsl.c >> +++ b/drivers/ata/sata_fsl.c >> @@ -1262,6 +1262,7 @@ static struct scsi_host_template sata_fsl_sht = { >> static struct ata_port_operations sata_fsl_ops = { >> .inherits = &sata_pmp_port_ops, >> >> + .qc_defer = ata_std_qc_defer; >> .qc_prep = sata_fsl_qc_prep, >> .qc_issue = sata_fsl_qc_issue, >> .qc_fill_rtf = sata_fsl_qc_fill_rtf, > This doesn't look like it should change anything. sata_fsl_ops inherits > from sata_pmp_port_ops, which inherits from sata_port_ops, which already > sets qc_defer to ata_std_qc_defer. Oh, yes. Actually this patch was for older kernels where there inheritence was not there. As you mentioned, this patch is not required now. Thanks, Ashish ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
On Jul 30, 2009, at 8:23 AM, Kalra Ashish-B00888 wrote: Hello, >> Signed-off-by: Ashish Kalra >> --- >> drivers/ata/sata_fsl.c | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c >> index 5a88b44..a33f130 100644 >> --- a/drivers/ata/sata_fsl.c >> +++ b/drivers/ata/sata_fsl.c >> @@ -1262,6 +1262,7 @@ static struct scsi_host_template sata_fsl_sht = { >> static struct ata_port_operations sata_fsl_ops = { >> .inherits = &sata_pmp_port_ops, >> >> + .qc_defer = ata_std_qc_defer; >> .qc_prep = sata_fsl_qc_prep, >> .qc_issue = sata_fsl_qc_issue, >> .qc_fill_rtf = sata_fsl_qc_fill_rtf, > This doesn't look like it should change anything. sata_fsl_ops inherits > from sata_pmp_port_ops, which inherits from sata_port_ops, which already > sets qc_defer to ata_std_qc_defer. Oh, yes. Actually this patch was for older kernels where there inheritence was not there. As you mentioned, this patch is not required now. How old? Should we be asking for this to be applied to some of the stable kernel series? - k ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
Hello Kumar, >> > This doesn't look like it should change anything. sata_fsl_ops >> inherits >> > from sata_pmp_port_ops, which inherits from sata_port_ops, which >> already >> > sets qc_defer to ata_std_qc_defer. >> Oh, yes. Actually this patch was for older kernels where there >> inheritence was not there. As you mentioned, this patch is not >> required now. >> >How old? Should we be asking for this to be applied to some of the >stable kernel series? I believe that the inheritence stuff was added in 2.6.26, thus, this patch is required for quite old kernels ( < 2.6.26 ). Thanks, Ashish ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
On Jul 30, 2009, at 8:58 AM, Kalra Ashish-B00888 wrote: Hello Kumar, >> > This doesn't look like it should change anything. sata_fsl_ops >> inherits >> > from sata_pmp_port_ops, which inherits from sata_port_ops, which >> already >> > sets qc_defer to ata_std_qc_defer. >> Oh, yes. Actually this patch was for older kernels where there >> inheritence was not there. As you mentioned, this patch is not >> required now. >> >How old? Should we be asking for this to be applied to some of the >stable kernel series? I believe that the inheritence stuff was added in 2.6.26, thus, this patch is required for quite old kernels ( < 2.6.26 ). Ok, than I'm not going to worry about it. - k ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH][sata_fsl] Defer non-ncq commands when ncq commands active
On 07/29/2009 12:03 PM, ashish kalra wrote: From: Ashish Kalra Date: Wed, 29 Jul 2009 21:15:49 +0530 Fix for non-ncq & ncq commands causing timeouts when both are issued simultaneously to the same device. Signed-off-by: Ashish Kalra --- drivers/ata/sata_fsl.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/ata/sata_fsl.c b/drivers/ata/sata_fsl.c index 5a88b44..a33f130 100644 --- a/drivers/ata/sata_fsl.c +++ b/drivers/ata/sata_fsl.c @@ -1262,6 +1262,7 @@ static struct scsi_host_template sata_fsl_sht = { static struct ata_port_operations sata_fsl_ops = { .inherits = &sata_pmp_port_ops, + .qc_defer = ata_std_qc_defer; Applied version with obvious s/;/,/ fix... Jeff ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev