linux-kernel - RE: [PATCH V4 2/4] mmc: cqhci: DMA Configuration prior to CQE

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]

Date:   Thu, 20 Dec 2018 21:01:18 +0000
From:   Sowjanya Komatineni <skomatineni@...dia.com>
To:     Adrian Hunter <adrian.hunter@...el.com>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "mark.rutland@....com" <mark.rutland@....com>,
        "Mikko Perttunen" <mperttunen@...dia.com>,
        "thierry.reding@...il.com" <thierry.reding@...il.com>,
        Jonathan Hunter <jonathanh@...dia.com>,
        "ulf.hansson@...aro.org" <ulf.hansson@...aro.org>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-tegra@...r.kernel.org" <linux-tegra@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>
Subject: RE: [PATCH V4 2/4] mmc: cqhci: DMA Configuration prior to CQE

Hi Adrian,

Thank you for the feedback.

> This doesn't seem to relate to the host controller implementation.  
> "The device" means the eMMC.

Yes, setting block size of 512B before enabling command queue is a device specific 
requirement not host specific. So thought to update in cqhci driver to follow this 
device specific sequence requirement. This also serves tegra sdhci host strictly
following this device specific requirement.

> We don't want to disable and re-enable in the request function,
> so that change is not good for controllers that don't have your problem.
> Another thing to consider is that the block size register may not need to be changed
> - for example when cqhci is halted to allow a manual discard, the block size register is
> not updated, so I would expect its value to be unchanged.

Once block size is set prior to enabling CQE, it stays same till command queue is 
disabled and I don’t think it needs reconfiguration.

> There are ways you can solve this in your driver.  
> You could look at using SDHCI I/O accessors, and/or implement your own ->enable() 
> instead of calling sdhci_cqe_enable() directly.  Would that be feasible?

Sure, will provide updated patch that takes care of this inside tegra sdhci host once you 
confirm that we don’t plan to fix this device specific sequence requirement in cqhci driver.

-Sowjanya