Re: [dm-devel] [PATCH] device-mapper: Bad Block relocation target

On Thu, 2007-02-22 at 19:18 -0500, Daniel Drake wrote:
> Piet Delaney wrote:
> > On Sat, 2007-02-10 at 01:01 +0000, Daniel Drake wrote:
> > 
> > Nice patch Daniel, wondering about using it in 2.6.12. 
> 
> It won't compile there. You can probably find a 2.6.12-compatible 
> version in an EVMS tarball though.

I think I found it at:

      http://dev.gentoo.org/~dsd/genpatches/patches-2.6.12-10.htm

looks like it should be in the extras patch I just downloaded:

    4305_dm-bbr.patch
    EVMS 2.5.2
    Bad block relocation support for LiveCD users

What's the LiveCD got to do with it?

So if I install it I'd be want to be sure it works. How do ya test it?

> 
> It's also not written by me, copyright messages indicate IBM wrote it.
> 
> I will hopefully look at the last few feedback items and send in a newer 
> version soon for the next round of feedback.

While break-pointing around the ext3 code looking for the cause of
a hang I was thinking that many times they leave in the white space 
as if the brackets were there.
	.					.
	.					.
	.					.
	if (panic) {				if (panic) 
		printk("die\n");			printk("die\n");
	}
	return;                                 return;
}					}

Just leaving in the blank line seems to provide much of the
parse-ability of your current style to me and very likely 
keeps it in the kernel coding style.

-piet

> 
> Daniel
-- 
Piet Delaney                                    Phone: (408) 200-5256
Blue Lane Technologies                          Fax:   (408) 200-5299
10450 Bubb Rd.
Cupertino, Ca. 95014                            Email: [email protected]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

_______________________________________________
Evms-devel mailing list
[email protected]
To subscribe/unsubscribe, please visit:
https://lists.sourceforge.net/lists/listinfo/evms-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.0 (GNU/Linux)

iD8DBQBF3jiHJICwm/rv3hoRAqM7AJ9BcNf3KBGRwktyveiaTPhc4X97hQCfYVB5
AXO+4YU2s4JIO/8Yuk3N+S8=
=dWW7
-----END PGP SIGNATURE-----