linux-kernel - Re: [PATCH] mmc: core: apply SD quirks earlier during probe

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240820231858.GA439714@sony.com>
Date: Wed, 21 Aug 2024 08:18:58 +0900
From: Keita Aihara <keita.aihara@...y.com>
To: Ulf Hansson <ulf.hansson@...aro.org>
Cc: Jonathan Bell <jonathan@...pberrypi.com>, Tim.Bird@...y.com,
	Shingo.Takeuchi@...y.com, Masaya.Takahashi@...y.com,
	keita.aihara@...y.com, linux-mmc@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mmc: core: apply SD quirks earlier during probe

On Tue, Aug 20, 2024 at 12:10:36PM +0200, Ulf Hansson wrote:
> On Tue, 6 Aug 2024 at 03:36, Keita Aihara <keita.aihara@...y.com> wrote:
> >
> > On Mon, Aug 05, 2024 at 12:14:25PM +0200, Ulf Hansson wrote:
> > > On Fri, 2 Aug 2024 at 05:21, Keita Aihara <keita.aihara@...y.com> wrote:
> > > >
> > > > Applying MMC_QUIRK_BROKEN_SD_CACHE is broken, as the card's extended
> > > > registers are parsed prior to the quirk being applied in mmc_blk.
> > >
> > > In what way is it a problem to read the extended registers first?
> >
> > SD quirks are referenced by mmc_card_broken_sd_cache() in
> > sd_parse_ext_reg_perf(). If the quirk is set, SD_EXT_PERF_CACHE is not
> > set to card->ext_perf.feature_support and the cache support will not be
> > enabled.
> >
> > Therefore, SD quirks should be initialized before parsing the extension
> > registers.
>
> Makes perfect sense! Please include some of this information in the
> commit message to make this clear.

Sure.

>
> >
> > >
> > > >
> > > > Split this out into an SD-specific list of quirks and apply in
> > > > mmc_sd_init_card instead.
> > > >
> > > > Fixes: c467c8f08185 ("mmc: Add MMC_QUIRK_BROKEN_SD_CACHE for Kingston Canvas Go Plus from 11/2019")
> > > > Authored-by: Jonathan Bell <jonathan@...pberrypi.com>
>
> This tag isn't normally what we use. I suggest you change the author
> of the patch to Jonathan and keep his sob-tag.
>
> Then add yourself with a "Co-developed-by" tag and keep your sob-tag.

Thank you for your suggestion.
Removed "Authored-by" tag and added "Co-developed-by" tag to right after
the first "Signed-off-by" tag by Jonathan.

[PATCH v2] mmc: core: apply SD quirks earlier during probe

I hope the v2 patch meets the format to treat Jonathan as the patch
author.

>
> > > > Signed-off-by: Jonathan Bell <jonathan@...pberrypi.com>
> > > > Signed-off-by: Keita Aihara <keita.aihara@...y.com>
>
> [...]
>
> Kind regards
> Uffe

Best regards,
Keita Aihara