Re: [PATCH v2 2/2] rdma: Autoload netlink client modules

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 15, 2017 at 10:57:37AM -0600, Jason Gunthorpe wrote:
> On Tue, Aug 15, 2017 at 06:58:43PM +0300, Leon Romanovsky wrote:
> > On Tue, Aug 15, 2017 at 09:35:23AM -0600, Jason Gunthorpe wrote:
> > > On Tue, Aug 15, 2017 at 12:56:51PM +0300, Leon Romanovsky wrote:
> > >
> > > > Can it be something like that (untested)?
> > > > #define MODULE_ALIAS_RDMA_NETLINK(_index)
> > >
> > > No, MODULE_ALIAS does something like
> > >
> > > static const char __UNIQUE_ID(name)[]
> > >   __used __attribute__((section(".modinfo"), unused, aligned(1))) = __stringify(tag) "=" info;
> > >
> > > So it must accept a string produced by the pre-processor, not by code.
> >
> > But the netlink indexes were defined as anonymous enum, can we safely
> > convert them to be defines?
>
> Technically no, we shouldn't, the difference could break things.. But
> due to the limited use of the header we probably could change to
> defines.
>
> Either way doesn't really matter..

Can this be a solution?
http://elixir.free-electrons.com/linux/latest/source/include/uapi/linux/in.h#L27

Thanks

>
> Jason
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux