Re: [PATCH 3/4] libata: Improve timeout handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
> From: Alan Cox <alan@xxxxxxxxxx>
>
> On a timeout call a device specific handler early in the recovery so that
> we can complete and process successful commands which timed out due to IRQ
> loss or the like rather more elegantly.
>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxx>
> ---
[...]
> diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
> index a93247c..757b956 100644
> --- a/drivers/ata/libata-eh.c
> +++ b/drivers/ata/libata-eh.c
[...]
> @@ -2871,6 +2914,7 @@ EXPORT_SYMBOL_GPL(ata_sff_qc_issue);
>  EXPORT_SYMBOL_GPL(ata_sff_qc_fill_rtf);
>  EXPORT_SYMBOL_GPL(ata_sff_host_intr);
>  EXPORT_SYMBOL_GPL(ata_sff_interrupt);
> +EXPORT_SYMBOL_GPL(ata_sff_lost_interrupt);
[...]
> diff --git a/include/linux/libata.h b/include/linux/libata.h
> index 7bb1013..75dbb35 100644
> --- a/include/linux/libata.h
> +++ b/include/linux/libata.h
[...]
> @@ -1514,6 +1515,7 @@ extern bool ata_sff_qc_fill_rtf(struct ata_queued_cmd *qc);
>  extern unsigned int ata_sff_host_intr(struct ata_port *ap,
>  				      struct ata_queued_cmd *qc);
>  extern irqreturn_t ata_sff_interrupt(int irq, void *dev_instance);
> +extern void ata_sff_lost_interrupt(struct ata_port *ap);

Actually, this one isn't used anywhere outside libata-sff.c (yet?).

Also, perhaps we cannot agree on the style of multiple line comments,
but please check this patch for whitespace issues, there is even one
that can easily be spotted without running checkpatch.pl.

Regards,

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

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux