linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] MMC fixes for v5.13-rc8
@ 2021-06-24 12:32 Ulf Hansson
  2021-06-24 18:00 ` Linus Torvalds
  2021-06-24 18:01 ` pr-tracker-bot
  0 siblings, 2 replies; 5+ messages in thread
From: Ulf Hansson @ 2021-06-24 12:32 UTC (permalink / raw)
  To: Linus, linux-mmc, linux-kernel; +Cc: Ulf Hansson

Hi Linus,

Here's a PR with an MMC fix intended for v5.13-rc8. Details about the
highlights are as usual found in the signed tag.

Please pull this in!

Kind regards
Ulf Hansson


The following changes since commit 009c9aa5be652675a06d5211e1640e02bbb1c33d:

  Linux 5.13-rc6 (2021-06-13 14:43:10 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git tags/mmc-v5.13-rc6

for you to fetch changes up to 103a5348c22c3fca8b96c735a9e353b8a0801842:

  mmc: meson-gx: use memcpy_to/fromio for dram-access-quirk (2021-06-14 14:02:33 +0200)

----------------------------------------------------------------
MMC host:
 - meson-gx: Use memcpy_to/fromio for dram-access-quirk

----------------------------------------------------------------
Neil Armstrong (1):
      mmc: meson-gx: use memcpy_to/fromio for dram-access-quirk

 drivers/mmc/host/meson-gx-mmc.c | 50 ++++++++++++++++++++++++++++++++++++-----
 1 file changed, 45 insertions(+), 5 deletions(-)

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] MMC fixes for v5.13-rc8
  2021-06-24 12:32 [GIT PULL] MMC fixes for v5.13-rc8 Ulf Hansson
@ 2021-06-24 18:00 ` Linus Torvalds
  2021-06-28  7:49   ` Ulf Hansson
  2021-06-24 18:01 ` pr-tracker-bot
  1 sibling, 1 reply; 5+ messages in thread
From: Linus Torvalds @ 2021-06-24 18:00 UTC (permalink / raw)
  To: Ulf Hansson; +Cc: linux-mmc, Linux Kernel Mailing List

On Thu, Jun 24, 2021 at 5:32 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> Neil Armstrong (1):
>       mmc: meson-gx: use memcpy_to/fromio for dram-access-quirk

Ugh. How horribly ugly.

Why is that 'host->dram_access_quirk' test _inside_ the loop, rather
than be something like

        if (host->dram_access_quirk)
                return sg_copy_to_buffer(data->sg, data->sg_len,
                                host->bounce_buf, xfer_bytes);

at the top of the function, with meson_mmc_copy_buffer() then only
handling the mmio case?

No, I don't know this code, I'm just looking at the patch and going
"that looks really ugly".

Anyway, I've pulled it, but I thought I'd voice my reaction to it..

              Linus

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] MMC fixes for v5.13-rc8
  2021-06-24 12:32 [GIT PULL] MMC fixes for v5.13-rc8 Ulf Hansson
  2021-06-24 18:00 ` Linus Torvalds
@ 2021-06-24 18:01 ` pr-tracker-bot
  1 sibling, 0 replies; 5+ messages in thread
From: pr-tracker-bot @ 2021-06-24 18:01 UTC (permalink / raw)
  To: Ulf Hansson; +Cc: Linus, linux-mmc, linux-kernel, Ulf Hansson

The pull request you sent on Thu, 24 Jun 2021 14:32:51 +0200:

> git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git tags/mmc-v5.13-rc6

has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/4a09d388f2ab382f217a764e6a152b3f614246f6

Thank you!

-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/prtracker.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] MMC fixes for v5.13-rc8
  2021-06-24 18:00 ` Linus Torvalds
@ 2021-06-28  7:49   ` Ulf Hansson
  2021-06-28  8:28     ` Neil Armstrong
  0 siblings, 1 reply; 5+ messages in thread
From: Ulf Hansson @ 2021-06-28  7:49 UTC (permalink / raw)
  To: Linus Torvalds, Neil Armstrong
  Cc: linux-mmc, Linux Kernel Mailing List, Robin Murphy

+ Neil, Robin

On Thu, 24 Jun 2021 at 20:00, Linus Torvalds
<torvalds@linux-foundation.org> wrote:
>
> On Thu, Jun 24, 2021 at 5:32 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
> >
> > Neil Armstrong (1):
> >       mmc: meson-gx: use memcpy_to/fromio for dram-access-quirk
>
> Ugh. How horribly ugly.
>
> Why is that 'host->dram_access_quirk' test _inside_ the loop, rather
> than be something like
>
>         if (host->dram_access_quirk)
>                 return sg_copy_to_buffer(data->sg, data->sg_len,
>                                 host->bounce_buf, xfer_bytes);
>
> at the top of the function, with meson_mmc_copy_buffer() then only
> handling the mmio case?
>
> No, I don't know this code, I'm just looking at the patch and going
> "that looks really ugly".
>
> Anyway, I've pulled it, but I thought I'd voice my reaction to it..

Thanks, good point!

Robin and Neil did discuss some cleanup that could be made on top [1],
but it looks like that was targeting a different part.

In any case, Neil, would you mind sending a cleanup according to the
comment from Linus?

Kind regards
Uffe

[1]
https://www.spinics.net/lists/arm-kernel/msg901045.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [GIT PULL] MMC fixes for v5.13-rc8
  2021-06-28  7:49   ` Ulf Hansson
@ 2021-06-28  8:28     ` Neil Armstrong
  0 siblings, 0 replies; 5+ messages in thread
From: Neil Armstrong @ 2021-06-28  8:28 UTC (permalink / raw)
  To: Ulf Hansson, Linus Torvalds
  Cc: linux-mmc, Linux Kernel Mailing List, Robin Murphy

Hi,

On 28/06/2021 09:49, Ulf Hansson wrote:
> + Neil, Robin
> 
> On Thu, 24 Jun 2021 at 20:00, Linus Torvalds
> <torvalds@linux-foundation.org> wrote:
>>
>> On Thu, Jun 24, 2021 at 5:32 AM Ulf Hansson <ulf.hansson@linaro.org> wrote:
>>>
>>> Neil Armstrong (1):
>>>       mmc: meson-gx: use memcpy_to/fromio for dram-access-quirk
>>
>> Ugh. How horribly ugly.
>>
>> Why is that 'host->dram_access_quirk' test _inside_ the loop, rather
>> than be something like
>>
>>         if (host->dram_access_quirk)
>>                 return sg_copy_to_buffer(data->sg, data->sg_len,
>>                                 host->bounce_buf, xfer_bytes);
>>
>> at the top of the function, with meson_mmc_copy_buffer() then only
>> handling the mmio case?
>>
>> No, I don't know this code, I'm just looking at the patch and going
>> "that looks really ugly".
>>
>> Anyway, I've pulled it, but I thought I'd voice my reaction to it..
> 
> Thanks, good point!
> 
> Robin and Neil did discuss some cleanup that could be made on top [1],
> but it looks like that was targeting a different part.

Yes this was a quick hack (and obviously not the optimal way to) to address the fatal system crash.

> 
> In any case, Neil, would you mind sending a cleanup according to the
> comment from Linus?

Sure, as discussed with Robin, the plan is to introduce sg_copy_to_buffer_to/fromio
and remove this crappy hack, but this requires multiple subsystems involved.

> 
> Kind regards
> Uffe

Thanks,
Neil

> 
> [1]
> https://www.spinics.net/lists/arm-kernel/msg901045.html
> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-06-28  8:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-24 12:32 [GIT PULL] MMC fixes for v5.13-rc8 Ulf Hansson
2021-06-24 18:00 ` Linus Torvalds
2021-06-28  7:49   ` Ulf Hansson
2021-06-28  8:28     ` Neil Armstrong
2021-06-24 18:01 ` pr-tracker-bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).