Re: [Bug #13663] suspend to ram regression (IDE related)

Jeff Chua
2009-07-01T17:28:31Z

On Thu, Jul 2, 2009 at 12:29 AM, BartlomiejZolnierkiewicz<bzolnier@gmail.com> wrote:> Here is the more complete version, also taking into the account changes> in ide_intr() and ide_timer_expiry():
This works great for. Survived STR, STD. I just applied on top vanillalatest Linus's git pull. Nothing else to revert.
Thanks,Jeff.

> --->  drivers/ide/ide-io.c |   15 ++++++++++----->  1 file changed, 10 insertions(+), 5 deletions(-)>> Index: b/drivers/ide/ide-io.c> ===================================================================> --- a/drivers/ide/ide-io.c> +++ b/drivers/ide/ide-io.c> @@ -532,7 +532,8 @@ repeat:>>                if (startstop == ide_stopped) {>                        rq = hwif->rq;> -                       hwif->rq = NULL;> +                       if ((drive->dev_flags & IDE_DFLAG_BLOCKED) == 0)> +                               hwif->rq = NULL;>                        goto repeat;>                }>        } else> @@ -679,8 +680,10 @@ void ide_timer_expiry (unsigned long dat>                spin_lock_irq(&hwif->lock);>                enable_irq(hwif->irq);>                if (startstop == ide_stopped && hwif->polling == 0) {> -                       rq_in_flight = hwif->rq;> -                       hwif->rq = NULL;> +                       if ((drive->dev_flags & IDE_DFLAG_BLOCKED) == 0) {> +                               rq_in_flight = hwif->rq;> +                               hwif->rq = NULL;> +                       }>                        ide_unlock_port(hwif);>                        plug_device = 1;>                }> @@ -856,8 +859,10 @@ irqreturn_t ide_intr (int irq, void *dev>         */>        if (startstop == ide_stopped && hwif->polling == 0) {>                BUG_ON(hwif->handler);> -               rq_in_flight = hwif->rq;> -               hwif->rq = NULL;> +               if ((drive->dev_flags & IDE_DFLAG_BLOCKED) == 0) {> +                       rq_in_flight = hwif->rq;> +                       hwif->rq = NULL;> +               }>                ide_unlock_port(hwif);>                plug_device = 1;>        }>ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥