public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Peter Samuelson <peter@cadcamlab.org>
To: Roberto Ragusa <robertoragusa@technologist.com>
Cc: linux-kernel@vger.kernel.org, kressb@fsc-usa.com
Subject: Re: kernel panic in SoftwareRAID autodetection
Date: Tue, 5 Dec 2000 18:36:57 -0600	[thread overview]
Message-ID: <20001205183657.J6567@cadcamlab.org> (raw)
In-Reply-To: <14893.25967.936504.881427@notabene.cse.unsw.edu.au> <yam8375.1358.149393648@a4000>
In-Reply-To: <yam8375.1358.149393648@a4000>; from robertoragusa@technologist.com on Wed, Dec 06, 2000 at 01:08:27AM +0200


[Roberto Ragusa]
> BTW, here is a little patch regarding a silly problem I found
> about RAID partitions naming (/proc/partitions).
> No more "md8" "md9" "md:" "md;" ... but "md8" "md9" "md10" "md11" ...
> Well, this patch should work up to "md99".

This stuff *really* should be split out into the drivers.  Brian Kress
had a patch against test11 for this.  Brian?  You want to fold in this
fix?

> +		if (unit<10) {
> +			sprintf(buf, "%s%c", maj, '0' + unit);
> +			return buf;
> +		}
> +		else {
> +			sprintf(buf, "%s%c%c", maj, '0' + unit / 10, '0' + unit % 10);
> +			return buf;
> +		}

Errrm, that's ugly. (:  Use %d, unless I'm missing something:

			sprintf(buf, "%s%d", maj, unit);

Peter
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

  reply	other threads:[~2000-12-06  1:08 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-12-01 14:50 kernel panic in SoftwareRAID autodetection Roberto Ragusa
2000-12-01 19:47 ` Neil Brown
2000-12-04 20:21   ` Roberto Ragusa
2000-12-05 22:00     ` Neil Brown
2000-12-05 23:08       ` Roberto Ragusa
2000-12-06  0:36         ` Peter Samuelson [this message]
2000-12-06 13:16           ` Brian Kress
2000-12-06 16:40             ` Peter Samuelson
2000-12-06 19:03               ` Brian Kress
2000-12-06 20:54             ` Neil Brown
2000-12-06 21:32               ` Brian Kress
2000-12-07  1:55                 ` Neil Brown

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=20001205183657.J6567@cadcamlab.org \
    --to=peter@cadcamlab.org \
    --cc=kressb@fsc-usa.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robertoragusa@technologist.com \
    /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