Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1215351

Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

From Shawn Lin <shawn.lin@rock-chips.com>
Newsgroups linux.kernel
Subject Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done
Date 2015-08-28 16:00 +0200
Message-ID <q2smf-545-23@gated-at.bofh.it> (permalink)
References (1 earlier) <q2iwx-7Ro-1@gated-at.bofh.it> <q2nFU-6Jv-13@gated-at.bofh.it> <q2oBY-87d-25@gated-at.bofh.it> <q2oLE-6n-37@gated-at.bofh.it> <q2p50-t4-17@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


在 2015/8/28 18:22, Jialing Fu 写道:
>
> [...]
>
>>>> Hi, ulf
>>>>
>>>> We find this bug on Intel-C3230RK platform for very small probability.
>>>>
>>>> Whereas I can easily reproduce this case if I add a mdelay(1) or
>>>> longer delay as Jialing did.
>>>>
>>>> This patch seems useful to me. Should we push it forward? :)
>>>
>>>
>>> It seems like a very good idea!
>>>
>>> Should we add a fixes tag to it?
>>
>>
>> That's cool, but how to add a fixes tag?
>>
>> [Fixes] mmc: core: fix race condition in mmc_wait_data_done ?   :)
>>
>
> A fixes tag points to an old commit which introduced the bug. If we can't find one, we can add a Cc tag to "stable". Just search the git log and you will find examples.
>
> Like add one line as below?
> Fixes: 2220eedfd7ae ("mmc: fix async request mechanism for sequential read scenarios")
>

That's it, Jialing. From my git blame, seems this bug has been 
introduced for a long time, but I feel strange that no one had captured 
it before you did.

Anyway, I will add a fixes tag and send v2 ASAP. :)

>
> Kind regards
> Uffe
>


-- 
Best Regards
Shawn Lin

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done Shawn Lin <shawn.lin@rock-chips.com> - 2015-08-28 05:20 +0200
  Re: [RESEND PATCH] mmc: core: fix race condition in  mmc_wait_data_done Shawn Lin <shawn.lin@rock-chips.com> - 2015-08-28 05:30 +0200
    Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done Ulf Hansson <ulf.hansson@linaro.org> - 2015-08-28 11:00 +0200
      Re: [RESEND PATCH] mmc: core: fix race condition in  mmc_wait_data_done Shawn Lin <shawn.lin@rock-chips.com> - 2015-08-28 12:00 +0200
        Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done Ulf Hansson <ulf.hansson@linaro.org> - 2015-08-28 12:10 +0200
          RE: [RESEND PATCH] mmc: core: fix race condition in  mmc_wait_data_done Jialing Fu <jlfu@marvell.com> - 2015-08-28 12:30 +0200
            Re: [RESEND PATCH] mmc: core: fix race condition in  mmc_wait_data_done Shawn Lin <shawn.lin@rock-chips.com> - 2015-08-28 16:00 +0200
              ~RE: [RESEND PATCH] mmc: core: fix race condition in  mmc_wait_data_done Jialing Fu <jlfu@marvell.com> - 2015-08-31 04:10 +0200

csiph-web