All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rusty Russell <rusty@rustcorp.com.au>
To: Brad Hards <bhards@bigpond.net.au>
To: Linus Torvalds <torvalds@transmeta.com>,
	Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [patch] i386 "General Options" - begone [take 2]
Date: Tue, 04 Jun 2002 08:42:47 +1000	[thread overview]
Message-ID: <E17F0XH-0002ic-00@wagner.rustcorp.com.au> (raw)
In-Reply-To: Your message of "Mon, 03 Jun 2002 13:18:09 +1000." <200206031318.09634.bhards@bigpond.net.au>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1987 bytes --]

In message <200206031318.09634.bhards@bigpond.net.au> you write:
> While moving software suspend, I also took the chance to tweak the Config.help 
> entry.

For trivial at least, please split the patches.  It makes it easy for
me and/or Linus to accept only one.

Also, please mention clearly if you obsolete a previous trivial patch...

> diff -Naur -X dontdiff linux-2.5.20-clean/arch/i386/Config.help linux-2.5.20-
config-munging/arch/i386/Config.help
> --- linux-2.5.20-clean/arch/i386/Config.help	Thu May 30 04:42:46 2002
> +++ linux-2.5.20-config-munging/arch/i386/Config.help	Mon Jun  3 12:39:48 200
2
> @@ -641,7 +641,8 @@
>    off or put into a power conserving "sleep" mode if they are not
>    being used.  There are two competing standards for doing this: APM
>    and ACPI.  If you want to use either one, say Y here and then also
> -  to the requisite support below.
> +  to the requisite support below. This option is also required for
> +  "software suspend", see below.
>  
>    Power Management is most important for battery powered laptop
>    computers; if you have a laptop, check out the Linux Laptop home

Like code, descriptions develop scar tissue when you do the "minimally
invasive" change.  Consider this classic trap-for-skimmers from the
glibc "snprintf" man page, and learn:

Return value
       These  functions  return  the number of characters printed
       (not including the trailing `\0' used  to  end  output  to
       strings).   snprintf  and vsnprintf do not write more than
       size bytes (including the trailing '\0'), and return -1 if
       the  output  was truncated due to this limit.  (Thus until
       glibc 2.0.6. Since glibc 2.1 these  functions  follow  the
       C99  standard and return the number of characters (exclud­
       ing the trailing '\0') which would have  been  written  to
       the final string if enough space had been available.)

Rusty.
--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

  reply	other threads:[~2002-06-03 22:39 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-03  1:56 Linux 2.5.20 Linus Torvalds
2002-06-03  3:06 ` 2.5.20 -- suspend.c still breaks the build (originally reported for 2.5.18) Miles Lane
2002-06-04 14:06   ` Pavel Machek
2002-06-03  3:18 ` [patch] i386 "General Options" - begone [take 2] Brad Hards
2002-06-03 22:42   ` Rusty Russell [this message]
2002-06-04 14:09   ` Pavel Machek
2002-06-04 22:05     ` Brad Hards
2002-06-02  5:16       ` Pavel Machek
2002-06-03 13:34 ` Oops Linux 2.5.20 Martin Dalecki
2002-06-03 14:06 ` [PATCH] 2.5.20 airo wireless - "I can't get no, compilation..." Martin Dalecki
2002-06-03 14:09 ` ]PATCH] 2.5.20 IDE 83 Martin Dalecki
2002-06-04 12:07   ` Jens Axboe
2002-06-04 15:30 ` [PATCH] 2.5.20 IDE 84 Martin Dalecki
2002-06-05 13:03 ` [PATCH] 2.5.20 IDE 85 Martin Dalecki
2002-06-05 14:17   ` Jens Axboe
2002-06-05 14:00     ` Martin Dalecki
2002-06-05 15:48       ` Jens Axboe
2002-06-05 15:52         ` Jens Axboe
2002-06-05 15:10           ` Martin Dalecki
2002-06-05 16:14             ` Jens Axboe
2002-06-05 15:26               ` Martin Dalecki
2002-06-05 18:02               ` Jeff Garzik
2002-06-06  7:17           ` Martin Dalecki
2002-06-05 16:36 ` [PATCH] Cleanup i386 <linux/init.h> abuses Tom Rini
2002-06-07 11:01   ` Pavel Machek
2002-06-07 19:19     ` Thunder from the hill
2002-06-07 19:26     ` Tom Rini
2002-06-05 23:22 ` [PATCH] Add <linux/kdev_t.h> to <linux/bio.h> Tom Rini
2002-06-05 23:34   ` Russell King
2002-06-05 23:42     ` Tom Rini
2002-06-06 18:33 ` [PATCH] Move vmalloc wrappers out of include/linux/vmalloc.h Tom Rini
2002-06-06 19:44 ` [PATCH] Remove <linux/mm.h> from <linux/vmalloc.h> Tom Rini
2002-06-06 21:02   ` [PATCH] More work on removing " Tom Rini
2002-06-06 21:21 ` [PATCH] Include <linux/gfp.h> directly instead of via <linux/mm.h> Tom Rini
2002-06-06 21:24 ` [PATCH] Remove numerous includes from <linux/mm.h> Tom Rini
  -- strict thread matches above, loose matches on Subject: below --
2002-06-04 20:59 [patch] i386 "General Options" - begone [take 2] Grover, Andrew
2002-06-04 21:20 ` Dave Jones
2002-06-05  1:34 ` Brad Hards
2002-06-05 10:29   ` Dave Jones
2002-06-05 23:11     ` Brad Hards
2002-06-04 21:58 Grover, Andrew
2002-06-04 22:09 ` Dave Jones
2002-06-04 23:16 ` Alan Cox
2002-06-07  5:38   ` fchabaud
2002-06-04 23:09 Grover, Andrew
2002-06-04 23:25 ` Dave Jones
2002-06-05  1:00 ` Alan Cox
2002-06-04 23:31 Grover, Andrew

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=E17F0XH-0002ic-00@wagner.rustcorp.com.au \
    --to=rusty@rustcorp.com.au \
    --cc=bhards@bigpond.net.au \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.