linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h>.
@ 2009-12-30  0:32 Robert P. J. Day
  2009-12-30  0:56 ` Stephen Rothwell
  0 siblings, 1 reply; 4+ messages in thread
From: Robert P. J. Day @ 2009-12-30  0:32 UTC (permalink / raw)
  To: Linux Kernel Mailing List


Since nothing in the entire tree makes use of this macro, remove it.
This removal should eventually propagate to all of the arch-specific
param.h files.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

  from the earlier discussion, no one provided any compelling reason
to retain this macro.  i'll eventually simplify the rest of the
param.h files similarly.  the fact that this macro is currently
exported to user space doesn't seem like any argument to keep it.

diff --git a/include/asm-generic/param.h b/include/asm-generic/param.h
index cdf8251..47939d8 100644
--- a/include/asm-generic/param.h
+++ b/include/asm-generic/param.h
@@ -19,6 +19,4 @@
 #define NOGROUP		(-1)
 #endif

-#define MAXHOSTNAMELEN	64	/* max length of hostname */
-
 #endif /* __ASM_GENERIC_PARAM_H */

rday
--

========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

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

* Re: [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h>.
  2009-12-30  0:32 [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h> Robert P. J. Day
@ 2009-12-30  0:56 ` Stephen Rothwell
  2009-12-30  1:03   ` Robert P. J. Day
  0 siblings, 1 reply; 4+ messages in thread
From: Stephen Rothwell @ 2009-12-30  0:56 UTC (permalink / raw)
  To: Robert P. J. Day; +Cc: Linux Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 531 bytes --]

Hi Robert,

On Tue, 29 Dec 2009 19:32:15 -0500 (EST) "Robert P. J. Day" <rpjday@crashcourse.ca> wrote:
>
> Since nothing in the entire tree makes use of this macro, remove it.
> This removal should eventually propagate to all of the arch-specific
> param.h files.

Except if the only place this is picked up by glibc is the kernel header
files, then it will disappear from user mode and some packages may break.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

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

* Re: [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h>.
  2009-12-30  0:56 ` Stephen Rothwell
@ 2009-12-30  1:03   ` Robert P. J. Day
  2009-12-30  2:05     ` Vikram Dhillon
  0 siblings, 1 reply; 4+ messages in thread
From: Robert P. J. Day @ 2009-12-30  1:03 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: Linux Kernel Mailing List

On Wed, 30 Dec 2009, Stephen Rothwell wrote:

> Hi Robert,
>
> On Tue, 29 Dec 2009 19:32:15 -0500 (EST) "Robert P. J. Day" <rpjday@crashcourse.ca> wrote:
> >
> > Since nothing in the entire tree makes use of this macro, remove
> > it. This removal should eventually propagate to all of the
> > arch-specific param.h files.
>
> Except if the only place this is picked up by glibc is the kernel
> header files, then it will disappear from user mode and some
> packages may break.

  true, but this still takes me back to my original observation -- why
is this macro being defined in a kernel header file if the kernel
itself has absolutely no interest in it and doesn't use it in any way?

  if glibc wants it, it might as well define it.

rday
--


========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

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

* Re: [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h>.
  2009-12-30  1:03   ` Robert P. J. Day
@ 2009-12-30  2:05     ` Vikram Dhillon
  0 siblings, 0 replies; 4+ messages in thread
From: Vikram Dhillon @ 2009-12-30  2:05 UTC (permalink / raw)
  To: Robert P. J. Day; +Cc: Stephen Rothwell, Linux Kernel Mailing List

I agree with you, the only place this macro is being used in glibc,
there are some really nice packages like MATLAB that depend on it, but
in distributions like Solaris there are conflicts in the newer version
and glibc so the trend seems to be moving away from glibc dependency
because of those conflicts. We might more of packages moving away from
glibc so let's wait and see what happens next :D

Regards,
Vikram Dhillon

~~~
There are lots of Linux users who don't care how the kernel works, but
only want to use it. That is a tribute to how good Linux is.
-- Linus Torvalds



On Tue, Dec 29, 2009 at 8:03 PM, Robert P. J. Day <rpjday@crashcourse.ca> wrote:
> On Wed, 30 Dec 2009, Stephen Rothwell wrote:
>
>> Hi Robert,
>>
>> On Tue, 29 Dec 2009 19:32:15 -0500 (EST) "Robert P. J. Day" <rpjday@crashcourse.ca> wrote:
>> >
>> > Since nothing in the entire tree makes use of this macro, remove
>> > it. This removal should eventually propagate to all of the
>> > arch-specific param.h files.
>>
>> Except if the only place this is picked up by glibc is the kernel
>> header files, then it will disappear from user mode and some
>> packages may break.
>
>  true, but this still takes me back to my original observation -- why
> is this macro being defined in a kernel header file if the kernel
> itself has absolutely no interest in it and doesn't use it in any way?
>
>  if glibc wants it, it might as well define it.
>
> rday
> --
>
>
> ========================================================================
> Robert P. J. Day                               Waterloo, Ontario, CANADA
>
>            Linux Consulting, Training and Kernel Pedantry.
>
> Web page:                                          http://crashcourse.ca
> Twitter:                                       http://twitter.com/rpjday
> ========================================================================
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

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

end of thread, other threads:[~2009-12-30  2:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-12-30  0:32 [PATCH] Remove macro MAXHOSTNAMELEN from <asm-generic/param.h> Robert P. J. Day
2009-12-30  0:56 ` Stephen Rothwell
2009-12-30  1:03   ` Robert P. J. Day
2009-12-30  2:05     ` Vikram Dhillon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).