Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1579795

Re: [PATCH 3/8] mm: cma: Export a few symbols

From Maxime Ripard <maxime.ripard@free-electrons.com>
Newsgroups linux.kernel
Subject Re: [PATCH 3/8] mm: cma: Export a few symbols
Date 2017-02-13 14:50 +0100
Message-ID <tapaW-4mz-9@gated-at.bofh.it> (permalink)
References <t904V-5k-7@gated-at.bofh.it> <t904W-5k-17@gated-at.bofh.it> <t92zM-1JF-19@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

Hi Michal,

On Thu, Feb 09, 2017 at 08:20:47PM +0100, Michal Hocko wrote:
> [CC CMA people]
> 
> On Thu 09-02-17 17:39:17, Maxime Ripard wrote:
> > Modules might want to check their CMA pool size and address for debugging
> > and / or have additional checks.
> > 
> > The obvious way to do this would be through dev_get_cma_area and
> > cma_get_base and cma_get_size, that are currently not exported, which
> > results in a build failure.
> > 
> > Export them to prevent such a failure.
> 
> Who actually uses those exports. None of the follow up patches does
> AFAICS.

This is for the ARM Mali GPU driver that is out of tree, unfortunately.

In one case (using the legacy fbdev API), the driver wants to (and
probably should) validate that the buffer as indeed been allocated
from the memory allocation pool.

Rob suggested that instead of hardcoding it to cover the whole RAM
(which defeats the purpose of that check in the first place), we used
the memory-region bindings in the DT and follow that, which does work
great, but we still have to retrieve the base address and size of that
region, hence why this patches are needed.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 3/8] mm: cma: Export a few symbols Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-09 17:50 +0100
  Re: [PATCH 3/8] mm: cma: Export a few symbols Michal Hocko <mhocko@kernel.org> - 2017-02-09 20:30 +0100
    Re: [PATCH 3/8] mm: cma: Export a few symbols Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-13 14:50 +0100
      Re: [PATCH 3/8] mm: cma: Export a few symbols Michal Hocko <mhocko@kernel.org> - 2017-02-20 13:40 +0100
        Re: [PATCH 3/8] mm: cma: Export a few symbols Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-02-24 00:10 +0100

csiph-web