[PATCH][Trusty/Utopic/Vivid] mmc: card: Don't access RPMB partitions for normal read/write

[Adam Lee]

On Wed, May 13, 2015 at 03:15:22PM +0100, Andy Whitcroft wrote:
> On Wed, May 13, 2015 at 03:21:15PM +0800, Adam Lee wrote:
> > From: Chuanxiao Dong <chuanxiao.dong at intel.com>
> > 
> > During kernel boot, it will try to read some logical sectors
> > of each block device node for the possible partition table.
> > 
> > But since RPMB partition is special and can not be accessed
> > by normal eMMC read / write CMDs, it will cause below error
> > messages during kernel boot:
> > ...
> >  mmc0: Got data interrupt 0x00000002 even though no data operation was in progress.
> >  mmcblk0rpmb: error -110 transferring data, sector 0, nr 32, cmd response 0x900, card status 0xb00
> >  mmcblk0rpmb: retrying using single block read
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  mmcblk0rpmb: timed out sending r/w cmd command, card status 0x400900
> >  end_request: I/O error, dev mmcblk0rpmb, sector 0
> >  Buffer I/O error on device mmcblk0rpmb, logical block 0
> >  end_request: I/O error, dev mmcblk0rpmb, sector 8
> >  Buffer I/O error on device mmcblk0rpmb, logical block 1
> >  end_request: I/O error, dev mmcblk0rpmb, sector 16
> >  Buffer I/O error on device mmcblk0rpmb, logical block 2
> >  end_request: I/O error, dev mmcblk0rpmb, sector 24
> >  Buffer I/O error on device mmcblk0rpmb, logical block 3
> > ...
> > 
> > This patch will discard the access request in eMMC queue if
> > it is RPMB partition access request. By this way, it avoids
> > trigger above error messages.
> > 
> > Fixes: 090d25fe224c ("mmc: core: Expose access to RPMB partition")
> > Signed-off-by: Yunpeng Gao <yunpeng.gao at intel.com>
> > Signed-off-by: Chuanxiao Dong <chuanxiao.dong at intel.com>
> > Tested-by: Michael Shigorin <mike at altlinux.org>
> > Signed-off-by: Ulf Hansson <ulf.hansson at linaro.org>
> > (cherry picked from commit 4e93b9a6abc0d028daf3c8a00cb77b679d8a4df4)
> > Signed-off-by: Adam Lee <adam.lee at canonical.com>
> 
> This seems to just quiet a small number of benign errors on boot?  Do we
> really need to worry about these??  It does not feel like they are
> harmful?
> 
> -apw

I forgot the BugLink in the first patch of this thread.

http://bugs.launchpad.net/bugs/1454013

It's not just some errors, the booting stage will take very long time
without this fix.

-- 
Adam Lee