Return-Path: Received: from daytona.panasas.com ([67.152.220.89]:14857 "EHLO daytona.panasas.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756556Ab1EWSug (ORCPT ); Mon, 23 May 2011 14:50:36 -0400 Message-ID: <4DDAAC64.6050807@panasas.com> Date: Mon, 23 May 2011 21:50:12 +0300 From: Boaz Harrosh To: Benny Halevy CC: Trond Myklebust , NFS list Subject: Re: [PATCHSET v6 0/26] pnfs for 2.6.40 References: <4DDA8C3D.5080706@panasas.com> In-Reply-To: <4DDA8C3D.5080706@panasas.com> Content-Type: text/plain; charset=UTF-8 Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On 05/23/2011 07:33 PM, Benny Halevy wrote: Benny Hi I have a problem that the default wsize is very small 64K and I get small IOs. I found that the governing member right now is NFS_SERVER()->wsize I did the below hack on My current code, but you took that away from me. diff --git a/fs/nfs/objlayout/objlayout.c b/fs/nfs/objlayout/objlayout.c index ec40408..f7b09e1 100644 --- a/fs/nfs/objlayout/objlayout.c +++ b/fs/nfs/objlayout/objlayout.c + server->wsize = ((PAGE_SIZE - sizeof(struct bio)) / sizeof(struct bio_vec)) + * PAGE_SIZE * 2; - dprintk("%s: Return data=%p\n", __func__, data); + dprintk("%s: Return data=%p wsize=0x%x\n", __func__, data, server->wsize); return 0; } What do you want that we do to replace this. The default 64K is to small. I don't mind that for pnfs it will be ~0 and the pg_test() will test for maxc_size as well. But then we'll also need the current size or the start_index Boaz