linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Pollei <stephen.pollei@gmail.com>
To: Hans Reiser <reiser@namesys.com>
Cc: Horst von Brand <vonbrand@inf.utfsm.cl>,
	Nikita Danilov <nikita@clusterfs.com>,
	Denis Vlasenko <vda@ilport.com.ua>,
	LKML <linux-kernel@vger.kernel.org>,
	ReiserFS List <reiserfs-list@namesys.com>
Subject: Re: I request inclusion of reiser4 in the mainline kernel
Date: Tue, 20 Sep 2005 15:08:21 -0700	[thread overview]
Message-ID: <feed8cdd050920150866e7925d@mail.gmail.com> (raw)
In-Reply-To: <43304A41.7080206@namesys.com>

On 9/20/05, Hans Reiser <reiser@namesys.com> wrote:
> Horst von Brand wrote:
> >Nikita Danilov <nikita@clusterfs.com> wrote:
> >It is supposed to go into the kernel, which is not exactly warning-free.

> Is that what this thread boils down to, that you guys think the compile
> should fail not warn?

I don't care if it fails or warns at compile time, but you shouldn't
misuse/abuse a warning by potentialily introducing an unrelated bug.

So if you had
#if defined(DEBUG_THIS) || defined(DEBUG_THAT) 
int znode_is_loaded(const struct znode *z);
#else
int znode_is_loaded(const struct znode *z)
  __attribute__((__warn_broken__("unavailible when not debuging")));
#endif
That would be great with me.. except __warn_broken__ or the like
doesn't exist AFAIK :-<
Closest thing is __attribute((__deprecated__)) but thats not quite right.

> >As was said before: It it is /really/ wrong, arrange for it not to compile
> >or not to link. If it isn't, well... then it wasn't that wrong anyway.

That is really true, if it is really wrong then make it so that trying
to do it simply breaks.
And if you are impatient then use a define to substitute crap into the
compile that will give you something you can't ignore as well. Make it
throw off sparks, get bonus points if you can make gcc segfault;->

#define znode_is_loaded(I_dont_care_you_are_going_to_) \
  } )die(]0now[>anyway<}}}}}}*bye*}

should stop a compile, but I don't think it's evil enough to cause gcc
to segfault.
If you didn't like my humor I'm sure you could code something more
concise that is as sick and twisted to crash a compile...
Hmmm. 
#if yadda yadda
int znode_is_loaded_yet_again(....)
#else
#define znode_is_loaded(z) ><<<>
/* break the compile if someone tries using it while not debuging */
#endif

That should break the parser as well. too bad there isn't a
_Pragma("error") or something... oh well.

Also note my opinion, doesn't really count if you grep the kernel
sources for pollei, you won't find anything.

-- 
http://dmoz.org/profiles/pollei.html
http://sourceforge.net/users/stephen_pollei/
http://www.orkut.com/Profile.aspx?uid=2455954990164098214
http://stephen_pollei.home.comcast.net/

  parent reply	other threads:[~2005-09-20 22:08 UTC|newest]

Thread overview: 143+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-16 17:05 I request inclusion of reiser4 in the mainline kernel Hans Reiser
2005-09-16 17:15 ` Christoph Hellwig
2005-09-16 17:40 ` Christoph Hellwig
2005-09-16 19:39   ` Hans Reiser
2005-09-16 19:52     ` Kyle Moffett
2005-09-17 10:51       ` Denis Vlasenko
2005-09-19  5:01         ` Hans Reiser
2005-09-16 20:50     ` Lennart Sorensen
2005-09-16 20:53       ` Lennart Sorensen
2005-09-19  5:09         ` Hans Reiser
2005-09-17  9:22     ` Christoph Hellwig
2005-09-17 10:56       ` Denis Vlasenko
2005-09-17 11:15         ` Denis Vlasenko
2005-09-18  0:34           ` Chris White
2005-09-18 10:21             ` Denis Vlasenko
2005-09-18 10:26               ` Christoph Hellwig
     [not found]                 ` <200509181406.25922.chrivers@iversen-net.dk>
2005-09-18 12:32                   ` Denis Vlasenko
2005-09-18 18:25                     ` David Masover
2005-09-19  5:16                   ` Hans Reiser
2005-09-19  5:56                     ` Valdis.Kletnieks
2005-09-19  6:53                       ` Hans Reiser
2005-09-18 17:22                 ` michael chang
2005-09-18 19:16                   ` Valdis.Kletnieks
2005-09-18 20:04                   ` Horst von Brand
2005-09-18 20:29                     ` David Masover
2005-09-18 21:43                       ` Dan Oglesby
2005-09-19  1:37                         ` PFC
2005-09-19  1:53                           ` Kyle Moffett
2005-09-19  2:48                             ` Dr.Dre
2005-09-19  4:37                           ` Marc Perkel
2005-09-18 20:33                     ` Marc Perkel
2005-09-19  5:44                     ` Hans Reiser
2005-09-19 10:39                       ` Nikita Danilov
2005-09-19 18:51                         ` Hans Reiser
2005-09-19 10:51                       ` Alan Cox
2005-09-19 23:03                       ` Horst von Brand
2005-09-20  8:00                         ` Hans Reiser
2005-09-20  7:51                       ` Pavel Machek
2005-09-20 14:41                         ` David Masover
2005-09-20 17:25                           ` Hans Reiser
2005-09-20 18:17                           ` Horst von Brand
     [not found]                           ` <20050920175727.GA17820@thunk.org>
     [not found]                             ` <1127240326.10407.22.camel@localhost>
2005-09-20 21:11                               ` Theodore Ts'o
2005-09-21  0:08                                 ` Hans Reiser
2005-09-21  9:05                                 ` Vitaly Fertman
2005-09-20 17:46                         ` Hans Reiser
     [not found]                         ` <200509202328.28501.rik@osrc.info>
2005-09-20 20:15                           ` Valdis.Kletnieks
2005-09-20 21:17                             ` Theodore Ts'o
2005-09-20 21:33                               ` Valdis.Kletnieks
2005-09-20 21:37                           ` Pavel Machek
     [not found]                             ` <200509210133.41710.rik@osrc.info>
2005-09-20 22:10                               ` Pavel Machek
2005-09-21  0:22                                 ` Hans Reiser
2005-09-21  0:04                         ` Theodore Ts'o
2005-09-21  0:13                           ` Hans Reiser
2005-09-21  0:27                           ` Ric Wheeler
2005-09-21  0:44                             ` Hans Reiser
2005-09-21  1:12                               ` Ric Wheeler
2005-09-21  3:05                                 ` Hans Reiser
2005-09-21  4:55                                   ` Gregory Maxwell
2005-09-21 11:21                                     ` Ric Wheeler
2005-09-21 17:36                                       ` Hans Reiser
2005-09-21 18:12                                         ` Ric Wheeler
2005-09-21 10:16                                   ` Vitaly Fertman
2005-09-21  1:01                           ` Gregory Maxwell
2005-09-21  1:15                             ` Ric Wheeler
2005-09-23  6:21                             ` David Greaves
2005-09-23 11:37                               ` Gregory Maxwell
2005-09-18 20:52                   ` Kyle Moffett
2005-09-19  0:56                     ` michael chang
2005-09-18 21:38                   ` Alan Cox
2005-09-19  5:07                     ` Hans Reiser
2005-09-19  9:01                       ` Christoph Hellwig
2005-09-19  9:16                         ` Christoph Hellwig
2005-09-19  9:21                         ` Andrew Morton
2005-09-19 10:43                       ` Alan Cox
2005-09-19 18:50                         ` Hans Reiser
2005-09-19 18:51                         ` Hans Reiser
2005-09-19 12:45                       ` Jens Axboe
2005-09-20  4:16                       ` Nick Piggin
2005-09-20  6:28                         ` Hans Reiser
2005-09-20  7:16                           ` Nick Piggin
2005-09-20  7:59                             ` Hans Reiser
2005-09-20  8:31                               ` elevators (was Re: I request inclusion of reiser4 in the mainline kernel) Nick Piggin
2005-09-20 17:18                                 ` Hans Reiser
2005-09-20 11:42                               ` I request inclusion of reiser4 in the mainline kernel Jens Axboe
2005-09-20 13:30                                 ` Lorenzo Allegrucci
2005-09-20 13:41                                   ` Jens Axboe
2005-09-20 13:55                                   ` Nikita Danilov
2005-09-20 17:46                                     ` Hans Reiser
2005-09-20 15:25                                   ` Randy.Dunlap
2005-09-20 17:21                                 ` Hans Reiser
2005-09-20 18:18                                   ` Jens Axboe
2005-09-20 15:42                           ` Horst von Brand
2005-09-20 17:46                             ` Hans Reiser
2005-09-20 18:25                               ` Jens Axboe
2005-09-20 18:27                               ` Nikita Danilov
2005-09-21 21:16                                 ` Hans Reiser
2005-09-21 21:37                                   ` Nikita Danilov
2005-09-21 22:07                                     ` Hans Reiser
2005-09-20 17:55                             ` Hans Reiser
2005-09-19 20:00               ` Bill Davidsen
2005-09-19 21:55                 ` Hans Reiser
2005-09-18 10:23         ` Christoph Hellwig
2005-09-18 18:10           ` David Masover
2005-09-19 20:41   ` Alexander Zarochentsev
2005-09-26 15:03   ` Vladimir V. Saveliev
2005-10-04 19:00     ` Christoph Hellwig
2005-10-04 21:10       ` Hans Reiser
2005-11-11 16:15   ` Vladimir V. Saveliev
2006-01-09  9:54   ` What's the state of Reiser4 inclusion " Giovanni A. Orlando
2006-01-09  9:15     ` Arjan van de Ven
2005-09-17 11:16 ` I request inclusion of reiser4 " Denis Vlasenko
2005-09-18  3:06   ` George Garvey
2005-09-18 10:02   ` Nikita Danilov
2005-09-19 19:36     ` Stephen Pollei
2005-09-19 21:10       ` Nikita Danilov
2005-09-19 21:47         ` Benjamin Herrenschmidt
2005-09-19 22:01         ` Hans Reiser
2005-09-19 23:15     ` Horst von Brand
2005-09-20  3:57       ` Stephen Pollei
2005-09-20  9:30         ` Nikita Danilov
2005-09-20 15:36           ` Horst von Brand
2005-09-20 17:43             ` Hans Reiser
2005-09-20 18:36               ` Horst von Brand
2005-09-20 22:08               ` Stephen Pollei [this message]
2005-09-21  0:21                 ` Hans Reiser
2005-09-21 10:47                   ` Nikita Danilov
2005-09-21 17:34                     ` Hans Reiser
2005-09-21 18:31                       ` Nikita Danilov
2005-09-21 18:39                         ` Hans Reiser
2005-09-20 17:53         ` Alexandre Oliva
2005-09-20 22:57           ` Stephen Pollei
2005-09-21  8:15             ` Alexandre Oliva
2005-09-18 22:12   ` Hans Reiser
2005-09-18 11:06 ` Christoph Hellwig
2005-09-19  9:18   ` Vladimir V. Saveliev
2005-09-19  9:24     ` Christoph Hellwig
     [not found]       ` <43302CF7.2010901@namesys.com>
     [not found]         ` <20050920154711.GA6698@infradead.org>
2005-09-30 12:10           ` Vladimir V. Saveliev
2005-09-30 17:27             ` Nikita Danilov
2005-10-04 18:12             ` Christoph Hellwig
2005-09-19  9:40     ` Alexey Dobriyan
2005-09-20 22:40 James Lamanna
2005-09-20 23:46 ` Vadim Lobanov
2005-09-20 23:51   ` James Lamanna

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=feed8cdd050920150866e7925d@mail.gmail.com \
    --to=stephen.pollei@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nikita@clusterfs.com \
    --cc=reiser@namesys.com \
    --cc=reiserfs-list@namesys.com \
    --cc=vda@ilport.com.ua \
    --cc=vonbrand@inf.utfsm.cl \
    /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;
as well as URLs for NNTP newsgroup(s).