From: Toshi Kani <toshi.kani@hp.com>
To: "Dixit, Ashutosh" <ashutosh.dixit@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Cc: "Dutt, Sudeep" <sudeep.dutt@intel.com>,
"Rao, Nikhil" <nikhil.rao@intel.com>,
"Williams, Dan J" <dan.j.williams@intel.com>
Subject: Re: Regression in v4.2-rc1: vmalloc_to_page with ioremap
Date: Mon, 20 Jul 2015 09:59:02 -0600 [thread overview]
Message-ID: <1437407942.3214.159.camel@hp.com> (raw)
In-Reply-To: <1437407695.3214.156.camel@hp.com>
On Mon, 2015-07-20 at 09:54 -0600, Toshi Kani wrote:
> On Sat, 2015-07-18 at 18:04 +0000, Dixit, Ashutosh wrote:
> > vmalloc_to_page with ioremap'd memory used to work correctly till
> > v4.1. In
> > v4.2-rc1 when ioremap is done using huge pages vmalloc_to_page on
> > ioremap'd
> > memory crashes. Are there plans to fix this?
> >
> > An example of the use of vmalloc_to_page with ioremap is in the
> > Intel
> > MIC SCIF
> > driver (drivers/misc/mic/scif/scif_nodeqp.c) which dma map's the
> > ioremap'd range
> > of one device to a second device.
>
> Yes, vmalloc_to_page() assumes 4KB mappings. But ioremap with huge
> pages was enabled in v4.1, while v4.2-rc1 has update for the check
> with
> MTRRs. Can you send me outputs of the following files? If the
> driver
> fails to load in v4.2-rc1, you can obtain the info in v4.1.
>
> /proc/mtrr
> /proc/iomem
> /proc/vmallocinfo
> /sys/kernel/debug/kernel_page_tables (need CONFIG_X86_PTDUMP set)
>
> Also, does the driver map a regular memory range with ioremap? If
> not, how does 'struct page' get allocated for the range (since
> vmalloc_to_page returns a page pointer)?
Can you also try the 'nohugeiomap' kernel option to the 4.2-rc1 kernel
to see if the problem goes away?
Thanks,
-Toshi
next prev parent reply other threads:[~2015-07-20 16:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-18 18:04 Regression in v4.2-rc1: vmalloc_to_page with ioremap Dixit, Ashutosh
2015-07-20 15:54 ` Toshi Kani
2015-07-20 15:59 ` Toshi Kani [this message]
2015-07-20 18:33 ` Ashutosh Dixit
2015-07-20 19:21 ` Toshi Kani
2015-07-21 15:17 ` Ashutosh Dixit
2015-07-21 20:39 ` Toshi Kani
2015-07-21 20:43 ` Toshi Kani
2015-07-22 7:13 ` Ashutosh Dixit
2015-07-22 14:54 ` Toshi Kani
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1437407942.3214.159.camel@hp.com \
--to=toshi.kani@hp.com \
--cc=ashutosh.dixit@intel.com \
--cc=dan.j.williams@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=nikhil.rao@intel.com \
--cc=sudeep.dutt@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox