Hi,

On Saturday 12 May 2007, Lee Trager wrote:
> Currently when system which have HPA require HPA to be detected and
> disabled upon resume from RAM or disk. The current IDE drivers do not do
> this nor does libata(obviously it since it doesn't support HPA yet). I
> have implemented this into the current IDE drivers and it has been
> tested by many others since 7/15/2006 in bug number
> 6840(http://bugzilla.kernel.org/show_bug.cgi?id=6840) and it has been
> confirmed to work fine with no problems.

Big thanks for working on this bug.

The patch looks good, applied.

> --- linux-2.6.21.1-old/include/linux/ide.h    2007-05-01 02:54:12.000000000 
> -0400
> +++ linux-2.6.21.1/include/linux/ide.h        2007-04-28 01:06:20.000000000 
> -0400
> @@ -1005,6 +1005,7 @@
>       int             (*probe)(ide_drive_t *);
>       void            (*remove)(ide_drive_t *);
>       void            (*shutdown)(ide_drive_t *);
> +     void            (*resume)(ide_drive_t *);
>  } ide_driver_t;

ide_driver_t changed a bit in 2.6.22 due to /proc/ide/ rework so this
chunk (and one chunk in ide-disk.c) rejected to apply.  Fixed them by hand.

> @@ -1279,7 +1281,12 @@
>       rqpm.pm_step = ide_pm_state_start_resume;
>       rqpm.pm_state = PM_EVENT_ON;
>  
> -     return ide_do_drive_cmd(drive, &rq, ide_head_wait);
> +     err = ide_do_drive_cmd(drive, &rq, ide_head_wait);
> +
> +     if (err == 0 && drv->resume)
> +             drv->resume(drive);
> +

Added extra drv != NULL check (there may be no IDE device driver et all).
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to