Re: resource sanity fails on ioremap

[Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>]

On Fri, Feb 05, 2016 at 11:28:57AM -0800, Jarkko Sakkinen wrote:
> On Fri, Feb 05, 2016 at 10:42:58AM -0700, Jason Gunthorpe wrote:
> > On Fri, Feb 05, 2016 at 08:47:04AM -0800, Jarkko Sakkinen wrote:
> > 
> > > Right, thanks for the explanation and I'm able to reproduce it on my own
> > > work laptop now:
> > > 
> > > [    0.708272] resource sanity check: requesting [mem
> > > 0xfed10000-0xfed15fff], which spans more than pnp 00:01 [mem
> > > 0xfed10000-0xfed13fff]
> > > [    0.708302]  [<ffffffff8108548a>] ? iomem_map_sanity_check+0xba/0xd0
> > > 
> > > Luckily we have this fixed (in two commits) in 4.5.
> > 
> > We do? I don't recall seeing a commit that changes the 0x1000
> 
> Ignore our fixes do not do anything to this. And it does not even occur
> my laptop (that was a different case). Should think before pressing.
> Apologies. I'll push a fix as soon to master.

Actually this does make things right, doesn't it?

https://github.com/jsakkine/linux-tpmdd/commit/1bd047be37d95bf65a219f4931215f71878ac060

struct resource new_res = {
	.start	= start,
	.end	= start + size - 1,
	.flags	= IORESOURCE_MEM,
};

/Jarkko

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140