[openib-general] [PATCH v2 1/7] IB/core - Add DMA mapping functions to allow device drivers to interpose
Ralph Campbell
ralph.campbel at qlogic.com
Fri Dec 1 16:27:59 PST 2006
> > Although the driver compiles on 32-bit kernels, it is unsupported
> > and never been tested. All known 64-bit systems don't define
> > CONFIG_HIGHMEM. In spite of previous emails suggesting that
> > page_address() can return NULL without CONFIG_HIGHMEM defined,
> > the code in include/linux/mm.h doesn't allow it (assuming the
> > page pointer is valid and not some random address).
> > I verified this with Andrew Morton.
>
> Hmm, is there no way to make this work on 32-bit kernels? I don't
> want to do something that we'll have to change again if we want to
> make things work on 32-bits.
>
> (And I know that qlogic has no intention of supporting the driver on
> 32-bit kernels, but we shouldn't make it impossible for someone else
> to fix it)
I don't think this is impossible to implement. I just wanted
to avoid the work unless you and others thought it was really
worth it given the reality that we already have a large
test matrix of platforms, distros, and kernel versions and
it probably won't get much testing. It is possible that
at some point 32-bit kernels will become a priority
but I don't know when that might happen.
More information about the general
mailing list