public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Trond Myklebust <trond.myklebust@fys.uio.no>
To: Kenny Simpson <theonetruekenny@yahoo.com>
Cc: Andrew Morton <akpm@osdl.org>, linux-kernel@vger.kernel.org
Subject: Re: nfs unhappiness with memory pressure
Date: Mon, 05 Dec 2005 17:39:27 -0500	[thread overview]
Message-ID: <1133822367.8003.5.camel@lade.trondhjem.org> (raw)
In-Reply-To: <20051205210442.17357.qmail@web34106.mail.mud.yahoo.com>

[-- Attachment #1: Type: text/plain, Size: 1780 bytes --]

Gah... This is the problem:


[  125.263017] rpciod/2      D 00000004     0  2772     15          2773
2771 (L-TLB)
[  125.271338] f70ccaa8 f70cca98 00000004 00000004 ffffffce f70cca74
c0117800 00000002 
[  125.279560]        c221f560 c2330030 05f561fa 0000000e 00000202
c221ffe0 f70ccabc 00000202 
[  125.288692]        c221ffe0 f70ccabc c221f560 00000002 00000143
2b8c50f7 0000001d c2330030 
[  125.297821] Call Trace:
[  125.300636]  [<c04086e1>] schedule_timeout+0x54/0xa5
[  125.305931]  [<c040866e>] io_schedule_timeout+0x29/0x33
[  125.311495]  [<c02880c4>] blk_congestion_wait+0x70/0x85
[  125.317058]  [<c014136b>] throttle_vm_writeout+0x69/0x7d
[  125.322720]  [<c014714d>] shrink_zone+0xe0/0xfa
[  125.327560]  [<c01471d4>] shrink_caches+0x6d/0x6f
[  125.332581]  [<c01472a6>] try_to_free_pages+0xd0/0x1b5
[  125.338056]  [<c013fa4b>] __alloc_pages+0x135/0x2e8
[  125.343258]  [<c03b74ad>] tcp_sendmsg+0xaa0/0xb78
[  125.348281]  [<c03d4666>] inet_sendmsg+0x48/0x53
[  125.353212]  [<c0388716>] sock_sendmsg+0xb8/0xd3
[  125.358147]  [<c0388773>] kernel_sendmsg+0x42/0x4f
[  125.363259]  [<c038bc00>] sock_no_sendpage+0x5e/0x77
[  125.368556]  [<c03ee7af>] xs_tcp_send_request+0x2af/0x375
[  125.374306]  [<c03ed4c5>] xprt_transmit+0x4f/0x20b
....

If a process is blocking in an OOM situation in the socket layer, then
you are hosed.

Argh... Not sure entirely how to deal with that... We definitely don't
want the thing futzing around inside throttle_vm_writeout(), 'cos
writeout isn't going to happen while the socket blocks.

...OTOH, I'm not entirely sure we want to use GFP_ATOMIC and risk
bleeding the emergency pools dry: we also need those in order to receive
replies from the server.

Ah, well... Try GFP_ATOMIC first, and see if that helps.

Cheers,
 Trond


[-- Attachment #2: linux-2.6.15-fix_sock_allocation.dif --]
[-- Type: text/plain, Size: 1865 bytes --]

RPC: Do not block on skb allocation

 If we get something like the following,
 [  125.300636]  [<c04086e1>] schedule_timeout+0x54/0xa5
 [  125.305931]  [<c040866e>] io_schedule_timeout+0x29/0x33
 [  125.311495]  [<c02880c4>] blk_congestion_wait+0x70/0x85
 [  125.317058]  [<c014136b>] throttle_vm_writeout+0x69/0x7d
 [  125.322720]  [<c014714d>] shrink_zone+0xe0/0xfa
 [  125.327560]  [<c01471d4>] shrink_caches+0x6d/0x6f
 [  125.332581]  [<c01472a6>] try_to_free_pages+0xd0/0x1b5
 [  125.338056]  [<c013fa4b>] __alloc_pages+0x135/0x2e8
 [  125.343258]  [<c03b74ad>] tcp_sendmsg+0xaa0/0xb78
 [  125.348281]  [<c03d4666>] inet_sendmsg+0x48/0x53
 [  125.353212]  [<c0388716>] sock_sendmsg+0xb8/0xd3
 [  125.358147]  [<c0388773>] kernel_sendmsg+0x42/0x4f
 [  125.363259]  [<c038bc00>] sock_no_sendpage+0x5e/0x77
 [  125.368556]  [<c03ee7af>] xs_tcp_send_request+0x2af/0x375
 then the socket is blocked until memory is reclaimed, and no
 progress can ever be made.

 Try to access the emergency pools by using GFP_ATOMIC.

 Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
---

 net/sunrpc/xprtsock.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c
index 0a51fd4..77e8800 100644
--- a/net/sunrpc/xprtsock.c
+++ b/net/sunrpc/xprtsock.c
@@ -990,6 +990,7 @@ static void xs_udp_connect_worker(void *
 		sk->sk_data_ready = xs_udp_data_ready;
 		sk->sk_write_space = xs_udp_write_space;
 		sk->sk_no_check = UDP_CSUM_NORCV;
+		sk->sk_allocation = GFP_ATOMIC;
 
 		xprt_set_connected(xprt);
 
@@ -1074,6 +1075,7 @@ static void xs_tcp_connect_worker(void *
 		sk->sk_data_ready = xs_tcp_data_ready;
 		sk->sk_state_change = xs_tcp_state_change;
 		sk->sk_write_space = xs_tcp_write_space;
+		sk->sk_allocation = GFP_ATOMIC;
 
 		/* socket options */
 		sk->sk_userlocks |= SOCK_BINDPORT_LOCK;

  reply	other threads:[~2005-12-05 22:39 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20051115224645.27832.qmail@web34103.mail.mud.yahoo.com>
2005-11-15 23:47 ` mmap over nfs leads to excessive system load Kenny Simpson
2005-11-16  4:31   ` William Lee Irwin III
2005-11-16  6:05     ` Kenny Simpson
2005-11-16  7:45   ` Andrew Morton
2005-11-16 14:03     ` Trond Myklebust
2005-11-16 15:01       ` Kenny Simpson
2005-11-16 17:44         ` Trond Myklebust
2005-11-16 18:00           ` Andrew Morton
2005-11-16 18:34             ` Trond Myklebust
2005-11-16 18:38               ` Christoph Hellwig
2005-11-17 13:08                 ` Nikita Danilov
2005-11-16 19:09               ` Andrew Morton
2005-11-16 20:05                 ` Trond Myklebust
2005-11-16 20:56                   ` Kenny Simpson
2005-11-16 21:02                     ` Kenny Simpson
2005-11-16 21:09                     ` Trond Myklebust
2005-11-16 21:17                       ` Kenny Simpson
2005-11-16 21:41                       ` Kenny Simpson
2005-11-16 21:57                         ` Trond Myklebust
2005-11-16 22:04                           ` Kenny Simpson
2005-11-16 22:39                           ` Kenny Simpson
2005-11-16 23:06                             ` Trond Myklebust
2005-11-17 15:40                               ` Chuck Lever
2005-11-17 16:56                                 ` Kenny Simpson
2005-11-17 16:01                               ` Kenny Simpson
2005-11-17 21:04                                 ` Andrew Morton
2005-11-17 21:15                                   ` Kenny Simpson
2005-11-18 16:55                                   ` Kenny Simpson
2005-11-18 17:26                                   ` Kenny Simpson
2005-11-18 21:57                                   ` Kenny Simpson
2005-11-21 17:13                                   ` infinite loop? with mmap, nfs, pwrite, O_DIRECT Kenny Simpson
2005-11-21 17:49                                     ` Chuck Lever
2005-11-21 18:40                                       ` Kenny Simpson
2005-11-21 20:39                                         ` Andrew Morton
2005-11-21 21:39                                           ` Kenny Simpson
2005-11-21 22:42                                             ` Trond Myklebust
2005-11-21 23:14                                               ` Kenny Simpson
2005-11-21 23:34                                               ` Andrew Morton
2005-11-21 23:58                                                 ` Trond Myklebust
2005-11-22  0:09                                                   ` Andrew Morton
2005-11-22  0:18                                                     ` Trond Myklebust
2005-11-22  0:25                                                       ` Trond Myklebust
2005-11-22  0:28                                                       ` Andrew Morton
2005-11-22  0:49                                                         ` Trond Myklebust
2005-11-30 20:04                                                           ` nfs unhappiness with memory pressure Kenny Simpson
2005-11-30 21:42                                                             ` Keith Mannthey
2005-11-30 22:18                                                               ` Kenny Simpson
2005-12-01 14:49                                                               ` Kenny Simpson
2005-12-05 18:01                                                             ` Kenny Simpson
2005-12-05 19:44                                                               ` Kenny Simpson
2005-12-05 20:14                                                                 ` Kenny Simpson
2005-12-05 20:13                                                               ` Trond Myklebust
2005-12-05 20:33                                                                 ` Trond Myklebust
2005-12-05 20:52                                                                   ` Nick Piggin
2005-12-05 21:18                                                                     ` Trond Myklebust
2005-12-05 21:23                                                                       ` Trond Myklebust
2005-12-05 23:40                                                                         ` Nick Piggin
2005-12-06  0:48                                                                           ` Trond Myklebust
2005-12-05 21:51                                                                       ` Kenny Simpson
2005-12-05 21:04                                                                   ` Kenny Simpson
2005-12-05 22:39                                                                     ` Trond Myklebust [this message]
2005-12-06  3:36                                                                       ` Andrew Morton
2005-12-06  3:36                                                                       ` Andrew Morton
2005-12-06  4:40                                                                         ` Trond Myklebust
2005-12-06  5:42                                                                           ` Nick Piggin
2005-12-06 12:18                                                                             ` Trond Myklebust
2005-12-06 19:31                                                                         ` Kenny Simpson
2005-12-06 15:51                                                                       ` Kenny Simpson
2005-11-21 21:54                                           ` infinite loop? with mmap, nfs, pwrite, O_DIRECT Kenny Simpson
2005-11-21 20:12                                       ` Kenny Simpson
2005-11-17 17:02                               ` mmap over nfs leads to excessive system load Chuck Lever
2005-11-17 17:07                                 ` Trond Myklebust
2005-11-18 19:59                               ` Kenny Simpson
2005-11-16 21:31                   ` Andrew Morton
2005-11-16 21:49                     ` Trond Myklebust
2005-11-16 22:10                       ` Andrew Morton
2005-11-16 22:23                         ` Trond Myklebust
2005-11-16 22:38                           ` Trond Myklebust
2005-11-16 22:50                             ` Andrew Morton
2005-11-16 22:44                           ` Andrew Morton
2005-11-16 23:10                             ` Trond Myklebust
2005-11-17  0:06                             ` Trond Myklebust
2005-11-17  0:25                               ` Andrew Morton
2005-11-17  0:28                                 ` Trond Myklebust
2005-11-17  0:38                                   ` Andrew Morton
2005-11-17  0:47                                     ` Trond Myklebust
2005-11-16 18:48           ` Kenny Simpson
2005-11-16 19:06           ` Kenny Simpson
2005-12-06 13:40 nfs unhappiness with memory pressure Albert Herranz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1133822367.8003.5.camel@lade.trondhjem.org \
    --to=trond.myklebust@fys.uio.no \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=theonetruekenny@yahoo.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox