lkml.org 
[lkml]   [2013]   [Dec]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Writeback threads and freezable
On Mon, Dec 16, 2013 at 02:56:52PM +1100, Dave Chinner wrote:
> > What are you suggesting? Implementing separate warm and hot unplug
> > paths? That makes no sense whatsoever. Device hot unplug is just a
> > sub operation of general device unplug which should be able to succeed
> > whether the underlying device is failing IOs or not.
>
> I don't care. Trying to issue IO from an an IO error handling path
> where the device has just been removed is fundamentally broken.

What? Have you even read the original message? IO error handling
path isn't issuing the IO here. The hot unplug operation is
completely asynchronous to the IO path. What's dead locking is not
the filesystem and IO path but device driver layer and hot unplug
path. IOs are not stalled.

> Indeed, what I'm asking for is for a notification so that we can
> *shut the filesystem down* straight away, rather than have to wait
> for an IO error in a critical metadata structure to trigger the
> shutdown for us.

which has exactly zero to do with the issue at hand. Would you please
stop derailing the discussion? Again, IOs are not stalled. That's
not the issue here.

> > > It's simply not a valid thing to do - just how is the filesystem
> > > supposed to sync to a non-existent device?
> >
> > By issuing all IOs and then handling the failures appropriately.
>
> Which bit of "filesystem might deadlock trying to issue IO" didn't
> you understand?

smh, I give up.

--
tejun


\
 
 \ /
  Last update: 2013-12-16 14:21    [W:0.071 / U:0.948 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site