All of lore.kernel.org
 help / color / mirror / Atom feed
* [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions?
@ 2011-06-07 15:41 Goldwyn Rodrigues
  2011-06-07 17:30 ` Sunil Mushran
  2011-06-08  6:41 ` Joel Becker
  0 siblings, 2 replies; 3+ messages in thread
From: Goldwyn Rodrigues @ 2011-06-07 15:41 UTC (permalink / raw)
  To: ocfs2-devel

Hi,

There have been index tree corruptions, where the directory entries
are correct however the index directory is not. In this case, the
filesystem turns read-only where as all the information to complete
the operation is still available because the primary directory
structure is sane. The ocfs2 module could just disable indexed
directory for this directory and continue handling the operation like
the normal directories without indexes. Of course, it still writes the
warning message, to run fsck, to the kernel log  but avoid the RO
filesystem which halt all I/O writes.

Th ocfs2-tools fix is posted at:
http://oss.oracle.com/pipermail/ocfs2-tools-devel/2011-May/003812.html
However, it would need changes for indexed directory rebuilds.

Let me know what you think.

-- 
Goldwyn

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions?
  2011-06-07 15:41 [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions? Goldwyn Rodrigues
@ 2011-06-07 17:30 ` Sunil Mushran
  2011-06-08  6:41 ` Joel Becker
  1 sibling, 0 replies; 3+ messages in thread
From: Sunil Mushran @ 2011-06-07 17:30 UTC (permalink / raw)
  To: ocfs2-devel

On 06/07/2011 08:41 AM, Goldwyn Rodrigues wrote:
> There have been index tree corruptions, where the directory entries
> are correct however the index directory is not. In this case, the
> filesystem turns read-only where as all the information to complete
> the operation is still available because the primary directory
> structure is sane. The ocfs2 module could just disable indexed
> directory for this directory and continue handling the operation like
> the normal directories without indexes. Of course, it still writes the
> warning message, to run fsck, to the kernel log  but avoid the RO
> filesystem which halt all I/O writes.
>
> Th ocfs2-tools fix is posted at:
> http://oss.oracle.com/pipermail/ocfs2-tools-devel/2011-May/003812.html
> However, it would need changes for indexed directory rebuilds.
>
> Let me know what you think.

I think it looks good. Though I would prefer if Mark also acked this
as he knows dxdir best.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions?
  2011-06-07 15:41 [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions? Goldwyn Rodrigues
  2011-06-07 17:30 ` Sunil Mushran
@ 2011-06-08  6:41 ` Joel Becker
  1 sibling, 0 replies; 3+ messages in thread
From: Joel Becker @ 2011-06-08  6:41 UTC (permalink / raw)
  To: ocfs2-devel

On Tue, Jun 07, 2011 at 10:41:42AM -0500, Goldwyn Rodrigues wrote:
> There have been index tree corruptions, where the directory entries
> are correct however the index directory is not. In this case, the
> filesystem turns read-only where as all the information to complete
> the operation is still available because the primary directory
> structure is sane. The ocfs2 module could just disable indexed
> directory for this directory and continue handling the operation like
> the normal directories without indexes. Of course, it still writes the
> warning message, to run fsck, to the kernel log  but avoid the RO
> filesystem which halt all I/O writes.

Goldwyn,
	You're correct, this is a more graceful solution.  We've just
never gotten around to it.

Joel

-- 

"Behind every successful man there's a lot of unsuccessful years."
        - Bob Brown

			http://www.jlbec.org/
			jlbec at evilplan.org

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-06-08  6:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-07 15:41 [Ocfs2-devel] Do filesystem need to turn RO on dx corruptions? Goldwyn Rodrigues
2011-06-07 17:30 ` Sunil Mushran
2011-06-08  6:41 ` Joel Becker

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.