Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1452405

Re: [RFC] usb: host: u132-hcd: Remove deprecated create_singlethread_workqueue

Show all headers | View raw

Hello, Alan.

On Wed, Jul 27, 2016 at 04:45:19PM -0400, Alan Stern wrote:
> > Hmm... That doesn't really make them dependable during memory reclaim.
> 
> True.  But it does mean that they can't cause a deadlock by waiting
> indefinitely for some other memory to be paged out to the very device
> they are on the access pathway for.
> 
> > What happens when those allocations fail?
> 
> The same thing that happens when any allocation fails -- the original
> I/O request fails with -ENOMEM or the equivalent.  In the case of 
> usb-storage, this is likely to trigger error recovery, which will need 
> to allocate memory of its own...  A bad situation to get into.

All that would do is deferring the deadlock, right?  I'm not sure it
makes a lot of sense to protect an IO path against memory pressure
half-way.  It either can be depended during memory reclaim or it
can't.  The use of GFP_NOIO / ATOMIC is probably increases the chance
of IO errors under moderate memory pressure too when there are
dependable memory backing devices (and there better be) which can push
things forward if called upon.

Can MM people please chime in?  The question is about USB stoage
devices and memory reclaim.  USB doesn't guarantee forward progress
under memory pressure but tries a best-effort attempt with GFP_NOIO
and ATOMIC.  Is this the right thing to do?

Thanks.

-- 
tejun

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Bhaktipriya Shridhar <bhaktipriya96@gmail.com> - 2016-07-27 11:30 +0200
  Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Oliver Neukum <oneukum@suse.com> - 2016-07-27 11:40 +0200
    Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Tejun Heo <tj@kernel.org> - 2016-07-27 20:10 +0200
      Re: [RFC] usb: host: u132-hcd: Remove deprecated create_singlethread_workqueue Alan Stern <stern@rowland.harvard.edu> - 2016-07-27 21:00 +0200
        Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Tejun Heo <tj@kernel.org> - 2016-07-27 21:30 +0200
          Re: [RFC] usb: host: u132-hcd: Remove deprecated create_singlethread_workqueue Alan Stern <stern@rowland.harvard.edu> - 2016-07-27 22:50 +0200
            Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Tejun Heo <tj@kernel.org> - 2016-07-29 15:20 +0200
              Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-01 16:00 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Tejun Heo <tj@kernel.org> - 2016-08-01 16:30 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated create_singlethread_workqueue Alan Stern <stern@rowland.harvard.edu> - 2016-08-01 21:10 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-01 21:40 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Oliver Neukum <oneukum@suse.com> - 2016-08-02 10:20 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-02 10:40 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Oliver Neukum <oneukum@suse.com> - 2016-08-02 12:10 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-02 14:00 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Oliver Neukum <oneukum@suse.com> - 2016-08-02 15:40 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Tejun Heo <tj@kernel.org> - 2016-08-02 21:20 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-02 23:40 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Michal Hocko <mhocko@kernel.org> - 2016-08-02 16:00 +0200
                Re: [RFC] usb: host: u132-hcd: Remove deprecated  create_singlethread_workqueue Oliver Neukum <oneukum@suse.com> - 2016-08-02 19:20 +0200

csiph-web