[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6153f9d247214d58b5eb7a76cb1258b3@hisilicon.com>
Date: Wed, 24 Mar 2021 07:33:23 +0000
From: "Song Bao Hua (Barry Song)" <song.bao.hua@...ilicon.com>
To: Christoph Hellwig <hch@....de>,
"tiantao (H)" <tiantao6@...ilicon.com>
CC: "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"peterz@...radead.org" <peterz@...radead.org>,
"paulmck@...nel.org" <paulmck@...nel.org>,
"ast@...nel.org" <ast@...nel.org>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"rostedt@...dmis.org" <rostedt@...dmis.org>,
"m.szyprowski@...sung.com" <m.szyprowski@...sung.com>,
"iommu@...ts.linux-foundation.org" <iommu@...ts.linux-foundation.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] dma-mapping: make map_benchmark compile into module
> -----Original Message-----
> From: Christoph Hellwig [mailto:hch@....de]
> Sent: Wednesday, March 24, 2021 8:13 PM
> To: tiantao (H) <tiantao6@...ilicon.com>
> Cc: akpm@...ux-foundation.org; peterz@...radead.org; paulmck@...nel.org;
> ast@...nel.org; tglx@...utronix.de; rostedt@...dmis.org; hch@....de;
> m.szyprowski@...sung.com; Song Bao Hua (Barry Song)
> <song.bao.hua@...ilicon.com>; iommu@...ts.linux-foundation.org;
> linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] dma-mapping: make map_benchmark compile into module
>
> On Wed, Mar 24, 2021 at 10:17:38AM +0800, Tian Tao wrote:
> > under some scenarios, it is necessary to compile map_benchmark
> > into module to test iommu, so this patch changed Kconfig and
> > export_symbol to implement map_benchmark compiled into module.
> >
> > On the other hand, map_benchmark is a driver, which is supposed
> > to be able to run as a module.
> >
> > Signed-off-by: Tian Tao <tiantao6@...ilicon.com>
>
> Nope, we're not going to export more kthread internals for a test
> module.
The requirement comes from an colleague who is frequently changing
the map-bench code for some customized test purpose. and he doesn't
want to build kernel image and reboot every time. So I moved the
requirement to Tao Tian.
Right now, kthread_bind() is exported, kthread_bind_mask() seems
to be a little bit "internal" as you said, maybe a wrapper like
kthread_bind_node() won't be that "internal", comparing to exposing
the cpumask?
Anyway, we don't find other driver users for this, hardly I can
convince you it is worth.
Thanks
Barry
Powered by blists - more mailing lists