linux-kernel - Re: [PATCH v2] dma-buf: move dma-buf symbols into the DMA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 25 Oct 2021 14:52:32 +0200
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Sumit Semwal <sumit.semwal@...aro.org>
Cc:     Christian König <christian.koenig@....com>,
        Alex Deucher <alexander.deucher@....com>,
        David Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>,
        "Pan, Xinhui" <Xinhui.Pan@....com>,
        LKML <linux-kernel@...r.kernel.org>,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Maxime Ripard <mripard@...nel.org>,
        Thomas Zimmermann <tzimmermann@...e.de>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        DRI mailing list <dri-devel@...ts.freedesktop.org>,
        Daniel Vetter <daniel.vetter@...ll.ch>,
        Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH v2] dma-buf: move dma-buf symbols into the DMA_BUF module
 namespace

On Mon, Oct 25, 2021 at 05:43:49PM +0530, Sumit Semwal wrote:
> Hi Greg,
> 
> On Mon, 25 Oct 2021 at 16:29, Christian König <christian.koenig@....com> wrote:
> >
> > Am 25.10.21 um 12:36 schrieb Greg Kroah-Hartman:
> > > On Sun, Oct 10, 2021 at 02:46:28PM +0200, Greg Kroah-Hartman wrote:
> > >> In order to better track where in the kernel the dma-buf code is used,
> > >> put the symbols in the namespace DMA_BUF and modify all users of the
> > >> symbols to properly import the namespace to not break the build at the
> > >> same time.
> > >>
> > >> Now the output of modinfo shows the use of these symbols, making it
> > >> easier to watch for users over time:
> > >>
> > >> $ modinfo drivers/misc/fastrpc.ko | grep import
> > >> import_ns:      DMA_BUF
> > >>
> > >> Cc: "Pan, Xinhui" <Xinhui.Pan@....com>
> > >> Cc: David Airlie <airlied@...ux.ie>
> > >> Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>
> > >> Cc: Maxime Ripard <mripard@...nel.org>
> > >> Cc: Thomas Zimmermann <tzimmermann@...e.de>
> > >> Cc: Mauro Carvalho Chehab <mchehab@...nel.org>
> > >> Cc: dri-devel@...ts.freedesktop.org
> > >> Acked-by: Daniel Vetter <daniel.vetter@...ll.ch>
> > >> Acked-by: Christian König <christian.koenig@....com>
> > >> Acked-by: Arnd Bergmann <arnd@...db.de>
> > >> Acked-by: Sumit Semwal <sumit.semwal@...aro.org>
> > >> Acked-by: Alex Deucher <alexander.deucher@....com>
> > >> Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > >> ---
> > >> v2: added lots of acks
> > >>      added 2 more drivers that needed the change, as found by Arnd
> > > Ping?  Any ideas on what needs to happen to get this into the tree?
> > >
> > > Or can I take it through my char-misc tree?  I seem to have a bunch of
> > > acks on it by the respective maintainers...
> >
> > I could push that upstream through the drm-misc-next tree if you like,
> > but honestly char-misc sounds like the better approach since this
> > touches a lot of drivers outside of drm as well.
> 
> I agree with Christian here - char-misc might be a better way for this.

Good idea, I'll take it through this tree as it will need to add the
symbol to the habanalabs driver at the same time as it has picked up
support for these symbols in my tree.

thanks,

greg k-h