mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jean Delvare <jdelvare@suse.de>
To: Matt Porter <mporter@kernel.crashing.org>,
	Alexandre Bounine <alexandre.bounine@idt.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: RapidIO subsystem and modularity
Date: Wed, 22 May 2013 14:40:30 +0200	[thread overview]
Message-ID: <1369226430.4718.123.camel@chaos.site> (raw)

Hi Matt, Alexandre, Andrew,

At the moment all pieces of the RapidIO subsystem are defined as bool
and thus cannot be built as modules. This is a problem for distribution
kernels, which are left with only two choices: disable RapidIO
altogether (too bad if any user actually needed it), or enable it all
(and waste memory and initialization time for every user without the
hardware - most users as I understand it.)

I don't even know what RapidIO is good for. I read about it on the web
but I have to admit I still have no clear idea, what systems use this
technology.

This leads me to two questions:

1* Is there a fundamental reason why (at least the device-specific parts
of) RapidIO can't be modularized?

FWIW I tried making the device drivers (tsi721) modular and it builds
and links OK, but I suppose one would have implement a proper remove
function, otherwise module unloading will break.

I also tried making the switch drivers (tsi57x etc.) modular, all build
just fine without any change, but there seems to be some initialization
magic which may not work when the code is built as modules.

2* What systems are expected to use RapidIO? It is enabled in the i386,
x86_64, ppc and ppc64 generic openSUSE kernel configurations. If someone
tells me that some or all of these generic kernels would never be used
on systems which need RapidIO support, I would be happy to disable
RapidIO support from these kernels, to make them smaller and decrease
boot times.

Thanks for any insight,
-- 
Jean Delvare
Suse L3


             reply	other threads:[~2013-05-22 12:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-22 12:40 Jean Delvare [this message]
2013-05-22 13:24 ` Bounine, Alexandre
2013-05-22 15:17   ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1369226430.4718.123.camel@chaos.site \
    --to=jdelvare@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=alexandre.bounine@idt.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mporter@kernel.crashing.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®