Messages in this thread | | | Date | Thu, 26 Apr 2001 11:28:46 +0200 | From | Pavel Machek <> | Subject | Re: [patch] linux likes to kill bad inodes |
| |
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. > >> > > >> > Solution is not to write bad inodes back to disk. > >> > > >> > >> Wouldn't we rather make it so bad inodes don't get marked dirty at all? > > > > I guess this is cheaper: we can mark inode dirty at 1000 points, but > > you only write it at one point. > > Whoops, I worded that poorly. To me, it seems like a bug to dirty a bad > inode. If this patch works, it is because somewhere, somebody did > something with a bad inode, and thought the operation worked (otherwise, > why dirty it?).
Would it make sense to put the check into write_inode_ with BUG() and return, then?
> So yes, even if we dirty them in a 1000 different places, we need to find > the one place that believes it can do something worthwhile to a bad > inode.
Could not it be something as simple as atime update?
Pavel -- The best software in life is free (not shareware)! Pavel GCM d? s-: !g p?:+ au- a--@ w+ v- C++@ UL+++ L++ N++ E++ W--- M- Y- R+ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |