mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: + net-uninline-xfrm_selector_match.patch added to -mm tree
       [not found] <200611031934.kA3JYCjF030732@shell0.pdx.osdl.net>
@ 2006-11-06 22:38 ` David Miller
  2006-11-06 22:49   ` Andrew Morton
  0 siblings, 1 reply; 3+ messages in thread
From: David Miller @ 2006-11-06 22:38 UTC (permalink / raw)
  To: linux-kernel, akpm; +Cc: acme

From: akpm@osdl.org
Date: Fri, 03 Nov 2006 11:34:12 -0800

> Subject: net: uninline xfrm_selector_match()
> From: Andrew Morton <akpm@osdl.org>
> 
> Six callsites, huge.
> 
> Cc: Arnaldo Carvalho de Melo <acme@mandriva.com>
> Signed-off-by: Andrew Morton <akpm@osdl.org>

You can't implement it like this :-)

xfrm_user.c is a bad place to put the uninlined version because
this can be built modular, whereas the callsites in places such
as xfrm_policy.c will be built statically into the kernel.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: + net-uninline-xfrm_selector_match.patch added to -mm tree
  2006-11-06 22:38 ` + net-uninline-xfrm_selector_match.patch added to -mm tree David Miller
@ 2006-11-06 22:49   ` Andrew Morton
  2006-11-06 22:51     ` David Miller
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2006-11-06 22:49 UTC (permalink / raw)
  To: David Miller; +Cc: linux-kernel, acme

On Mon, 06 Nov 2006 14:38:31 -0800 (PST)
David Miller <davem@davemloft.net> wrote:

> From: akpm@osdl.org
> Date: Fri, 03 Nov 2006 11:34:12 -0800
> 
> > Subject: net: uninline xfrm_selector_match()
> > From: Andrew Morton <akpm@osdl.org>
> > 
> > Six callsites, huge.
> > 
> > Cc: Arnaldo Carvalho de Melo <acme@mandriva.com>
> > Signed-off-by: Andrew Morton <akpm@osdl.org>
> 
> You can't implement it like this :-)
> 
> xfrm_user.c is a bad place to put the uninlined version because
> this can be built modular, whereas the callsites in places such
> as xfrm_policy.c will be built statically into the kernel.

I would have found that out when I got around to compiling it ;)

Where should the out-of-line function be placed, or should I just drop it? 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: + net-uninline-xfrm_selector_match.patch added to -mm tree
  2006-11-06 22:49   ` Andrew Morton
@ 2006-11-06 22:51     ` David Miller
  0 siblings, 0 replies; 3+ messages in thread
From: David Miller @ 2006-11-06 22:51 UTC (permalink / raw)
  To: akpm; +Cc: linux-kernel, acme

From: Andrew Morton <akpm@osdl.org>
Date: Mon, 6 Nov 2006 14:49:32 -0800

> I would have found that out when I got around to compiling it ;)
> 
> Where should the out-of-line function be placed, or should I just drop it? 

Either xfrm_policy.c or xfrm_state.c would work fine.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2006-11-06 22:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <200611031934.kA3JYCjF030732@shell0.pdx.osdl.net>
2006-11-06 22:38 ` + net-uninline-xfrm_selector_match.patch added to -mm tree David Miller
2006-11-06 22:49   ` Andrew Morton
2006-11-06 22:51     ` David Miller

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®