Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread Linus Torvalds
On Mon, 30 Apr 2001, Alan Cox wrote: > > Any reason a bad inode can't have its i_sb changed to a bad_inode_fs ? That would be my personal preference too, this was just the quick hack version. Changing superblocks might have other consequences (like getting the superblock inode lists right

Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread David S. Miller
Alan Cox writes: > Any reason a bad inode can't have its i_sb changed to a bad_inode_fs ? I believe this is what Linus suggested. Later, David S. Miller [EMAIL PROTECTED] - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED]

Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread Alan Cox
> { > - if (inode->i_sb && inode->i_sb->s_op && inode->i_sb->s_op->write_inode) > + if (inode->i_sb && inode->i_sb->s_op && inode->i_sb->s_op->write_inode && >!is_bad_inode(inode)) > inode->i_sb->s_op->write_inode(inode, sync); > } Any reason a bad inode can't have its

Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread Alan Cox
{ - if (inode-i_sb inode-i_sb-s_op inode-i_sb-s_op-write_inode) + if (inode-i_sb inode-i_sb-s_op inode-i_sb-s_op-write_inode !is_bad_inode(inode)) inode-i_sb-s_op-write_inode(inode, sync); } Any reason a bad inode can't have its i_sb changed to a bad_inode_fs ?

Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread David S. Miller
Alan Cox writes: Any reason a bad inode can't have its i_sb changed to a bad_inode_fs ? I believe this is what Linus suggested. Later, David S. Miller [EMAIL PROTECTED] - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More

Re: [patch] linux likes to kill bad inodes

2001-04-30 Thread Linus Torvalds
On Mon, 30 Apr 2001, Alan Cox wrote: Any reason a bad inode can't have its i_sb changed to a bad_inode_fs ? That would be my personal preference too, this was just the quick hack version. Changing superblocks might have other consequences (like getting the superblock inode lists right etc).

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Andreas Dilger
I previously wrote: > I will post a patch separately which handles a couple of cases where > *_delete_inode() does not call clear_inode() in all cases. OK, here it is. The ext2_delete_inode() change isn't exactly a bug fix, but rather a "performance" change. No need to hold BLK to check status

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Linus Torvalds
On Fri, 27 Apr 2001, Andreas Dilger wrote: > > However, since make_bad_inode() only changes the file methods and not > the superblock Please just make "make_bad_inode()" just do inode->i_sb = bad_super_block; and do everything else too. It's not acceptable to make low-level

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Andreas Dilger
Pavel writes: > [I'd really like to get patch it; killing user's data without good > reason seems evil to me, and this did quite a lot of damage to my > $HOME.] > > Pavel > PS: Only filesystem at use at time of problem was ext2, and

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Chris Mason
On Friday, April 27, 2001 12:28:54 AM +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Okay, so what about following patch, followed by attempt to debug it? > [I'd really like to get patch it; killing user's data without good > reason seems evil to me, and this did quite a lot of damage to my >

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Pavel Machek
Hi! > >> > I had a temporary disk failure (played with acpi too much). What > >> > happened was that disk was not able to do anything for five minutes > >> > or so. When disk recovered, linux happily overwrote all inodes it > >> > could not read while disk was down with zeros -> massive disk >

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Pavel Machek
Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it could not read while disk was down with zeros - massive disk corruption.

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Chris Mason
On Friday, April 27, 2001 12:28:54 AM +0200 Pavel Machek [EMAIL PROTECTED] wrote: Okay, so what about following patch, followed by attempt to debug it? [I'd really like to get patch it; killing user's data without good reason seems evil to me, and this did quite a lot of damage to my

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Andreas Dilger
Pavel writes: [I'd really like to get patch it; killing user's data without good reason seems evil to me, and this did quite a lot of damage to my $HOME.] Pavel PS: Only filesystem at use at time of problem was ext2, and it was

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Linus Torvalds
On Fri, 27 Apr 2001, Andreas Dilger wrote: However, since make_bad_inode() only changes the file methods and not the superblock Please just make make_bad_inode() just do inode-i_sb = bad_super_block; and do everything else too. It's not acceptable to make low-level filesystems

Re: [patch] linux likes to kill bad inodes

2001-04-27 Thread Andreas Dilger
I previously wrote: I will post a patch separately which handles a couple of cases where *_delete_inode() does not call clear_inode() in all cases. OK, here it is. The ext2_delete_inode() change isn't exactly a bug fix, but rather a performance change. No need to hold BLK to check status or

Re: [patch] linux likes to kill bad inodes

2001-04-26 Thread Jan Kara
> > > On Wednesday, April 25, 2001 10:01:20 PM +0200 Pavel Machek <[EMAIL PROTECTED]> > wrote: > > > Hi! > > > >> > Hi! > >> > > >> > I had a temporary disk failure (played with acpi too much). What > >> > happened was that disk was not able to do anything for five minutes > >> > or so. When

Re: [patch] linux likes to kill bad inodes

2001-04-26 Thread Pavel Machek
Hi! > >> > Hi! > >> > > >> > I had a temporary disk failure (played with acpi too much). What > >> > happened was that disk was not able to do anything for five minutes > >> > or so. When disk recovered, linux happily overwrote all inodes it > >> > could not read while disk was down with zeros

Re: [patch] linux likes to kill bad inodes

2001-04-26 Thread Pavel Machek
Hi! Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it could not read while disk was down with zeros - massive disk

Re: [patch] linux likes to kill bad inodes

2001-04-26 Thread Jan Kara
On Wednesday, April 25, 2001 10:01:20 PM +0200 Pavel Machek [EMAIL PROTECTED] wrote: Hi! Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Chris Mason
On Wednesday, April 25, 2001 10:01:20 PM +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > >> > Hi! >> > >> > I had a temporary disk failure (played with acpi too much). What >> > happened was that disk was not able to do anything for five minutes >> > or so. When disk recovered, linux

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Pavel Machek
Hi! > > Hi! > > > > I had a temporary disk failure (played with acpi too much). What > > happened was that disk was not able to do anything for five minutes > > or so. When disk recovered, linux happily overwrote all inodes it > > could not read while disk was down with zeros -> massive disk >

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Chris Mason
On Sunday, April 22, 2001 02:10:42 PM +0200 Pavel Machek <[EMAIL PROTECTED]> wrote: > Hi! > > I had a temporary disk failure (played with acpi too much). What > happened was that disk was not able to do anything for five minutes > or so. When disk recovered, linux happily overwrote all inodes

[patch] linux likes to kill bad inodes

2001-04-25 Thread Pavel Machek
Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it could not read while disk was down with zeros -> massive disk corruption. Solution is not to

[patch] linux likes to kill bad inodes

2001-04-25 Thread Pavel Machek
Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it could not read while disk was down with zeros - massive disk corruption. Solution is not to

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Chris Mason
On Sunday, April 22, 2001 02:10:42 PM +0200 Pavel Machek [EMAIL PROTECTED] wrote: Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Pavel Machek
Hi! Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote all inodes it could not read while disk was down with zeros - massive disk corruption.

Re: [patch] linux likes to kill bad inodes

2001-04-25 Thread Chris Mason
On Wednesday, April 25, 2001 10:01:20 PM +0200 Pavel Machek [EMAIL PROTECTED] wrote: Hi! Hi! I had a temporary disk failure (played with acpi too much). What happened was that disk was not able to do anything for five minutes or so. When disk recovered, linux happily overwrote