Hi Franck,

On Wed, Jul 24 2013, [email protected] wrote:
> From: Franck Jullien <[email protected]>
>
> A previous commit (fdfa20c1631210d0) reordered the
> shutdown sequence in mmc_blk_remove_req. However,
> mmc_cleanup_queue is now called before we get the
> card pointer and, sadly, mmc_cleanup_queue set
> mq->card to NULL.
>
> This patch moves the card pointer assignment before
> mmc_cleanup_queue.
>
> Signed-off-by: Franck Jullien <[email protected]>
> ---
>  drivers/mmc/card/block.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c
> index cd0b7f4..f4a0bea 100644
> --- a/drivers/mmc/card/block.c
> +++ b/drivers/mmc/card/block.c
> @@ -2191,10 +2191,10 @@ static void mmc_blk_remove_req(struct mmc_blk_data 
> *md)
>                * is freeing the queue that stops new requests
>                * from being accepted.
>                */
> +             card = md->queue.card;
>               mmc_cleanup_queue(&md->queue);
>               if (md->flags & MMC_BLK_PACKED_CMD)
>                       mmc_packed_clean(&md->queue);
> -             card = md->queue.card;
>               if (md->disk->flags & GENHD_FL_UP) {
>                       device_remove_file(disk_to_dev(md->disk), 
> &md->force_ro);
>                       if ((md->area_type & MMC_BLK_DATA_AREA_BOOT) &&

Thanks for the patch, pushed to mmc-next for 3.12.

- Chris.
-- 
Chris Ball   <[email protected]>   <http://printf.net/>
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to