All of lore.kernel.org
 help / color / mirror / Atom feed
From: florin@iucha.net (Florin Iucha)
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Adrian Bunk <bunk@stusta.de>,
	OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/4] 2.6.21-rc7 NFS writes: fix a series of issues
Date: Tue, 17 Apr 2007 22:30:55 -0500	[thread overview]
Message-ID: <20070418033055.GA24044@iucha.net> (raw)
In-Reply-To: <1176865565.6796.16.camel@heimdal.trondhjem.org>

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

On Tue, Apr 17, 2007 at 11:06:05PM -0400, Trond Myklebust wrote:
> > > I've split the issues introduced by the 2.6.21-rcX write code up into 4
> > > subproblems.
> > > 
> > > The first patch is just a cleanup in order to ease review.
> > > 
> > > Patch number 2 ensures that we never release the PG_writeback flag until
> > > _after_ we've either discarded the unstable request altogether, or put it
> > > on the nfs_inode's commit or dirty lists.
> > > 
> > > Patch number 3 fixes the 'desynchronized value of nfs_i.ncommit' error. It
> > > uses the PG_NEED_COMMIT flag as an indicator for whether or not the request
> > > may be redirtied.
> > > 
> > > Patch number 4 protects the NFS '.set_page_dirty' address_space operation
> > > against races with nfs_inode_add_request.
> > 
> > For 2.6.21, yes?
> 
> Right. A couple of nasty regressions have been sighted. This series
> attempts to deal with them all.

The good news is that the Gnome session log-in progresses to the point
where both top and bottom bars are painted (gray) and the bottom bar
is populated with icons (2.6.21-rc7 vanilla stops after displaying the
splash).  The bad news is that it stops there.

Big-copy fails as well, after 2.5G transferred.

The process traces are at:

   http://iucha.net/nfs/21-rc7-nfs1/gnome-session
   http://iucha.net/nfs/21-rc7-nfs1/big-copy

Regards,
florin

-- 
Bruce Schneier expects the Spanish Inquisition.
      http://geekz.co.uk/schneierfacts/fact/163

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

  reply	other threads:[~2007-04-18  3:50 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1176736734.6761.45.camel@heimdal.trondhjem.org>
     [not found] ` <Pine.LNX.4.64.0704160904560.5473@woody.linux-foundation.org>
     [not found]   ` <1176740307.6761.56.camel@heimdal.trondhjem.org>
     [not found]     ` <1176741408.6761.62.camel@heimdal.trondhjem.org>
     [not found]       ` <20070416125905.GA2769@iucha.net>
     [not found]         ` <1176792399.3035.30.camel@twins>
     [not found]           ` <1176796503.3035.33.camel@twins>
2007-04-17 17:01             ` nfs: desynchronized value of nfs_i.ncommit OGAWA Hirofumi
2007-04-17 22:44               ` Trond Myklebust
2007-04-18  1:19               ` [PATCH 0/4] 2.6.21-rc7 NFS writes: fix a series of issues Trond Myklebust
2007-04-18  1:29                 ` [PATCH 1/4] NFS: clean up the unstable write code Trond Myklebust
2007-04-18  1:29                 ` [PATCH 2/4] NFS: Don't clear PG_writeback until after we've processed unstable writes Trond Myklebust
2007-04-18  1:29                 ` [PATCH 3/4] NFS: Fix the 'desynchronized value of nfs_i.ncommit' error Trond Myklebust
2007-04-18  1:29                 ` [PATCH 4/4] NFS: Fix race in nfs_set_page_dirty Trond Myklebust
2007-04-18  2:58                 ` [PATCH 0/4] 2.6.21-rc7 NFS writes: fix a series of issues Andrew Morton
2007-04-18  3:06                   ` Trond Myklebust
2007-04-18  3:30                     ` Florin Iucha [this message]
2007-04-18  3:54                       ` Trond Myklebust
2007-04-18  4:07                         ` Florin Iucha
2007-04-18  4:13                           ` Andrew Morton
2007-04-18  4:30                             ` Florin Iucha
2007-04-18  5:14                               ` Linus Torvalds
2007-04-18  5:26                                 ` Florin Iucha
2007-04-18  5:37                                 ` Andrew Morton
2007-04-18 12:38                                   ` Florin Iucha
2007-04-18 13:15                                     ` Trond Myklebust
2007-04-18 13:42                                       ` Florin Iucha
2007-04-18 14:11                                         ` Trond Myklebust
2007-04-18 14:17                                           ` Florin Iucha
2007-04-18 14:19                                             ` Trond Myklebust
2007-04-19  1:52                                           ` Florin Iucha
2007-04-19  2:45                                             ` Trond Myklebust
2007-04-19  4:38                                               ` Success! Was: " Florin Iucha
2007-04-19 15:12                                               ` Chuck Lever
2007-04-19 15:17                                                 ` Trond Myklebust
2007-04-19 15:50                                                   ` Florin Iucha
2007-04-19 16:09                                                     ` Trond Myklebust
2007-04-19 19:58                                                       ` Failure! " Florin Iucha
2007-04-19 21:30                                                         ` Trond Myklebust
2007-04-19 21:49                                                           ` Florin Iucha
2007-04-20 13:30                                                             ` Success! Was: " Florin Iucha
2007-04-20 13:37                                                               ` Trond Myklebust
2007-04-20 13:51                                                                 ` Florin Iucha
2007-04-18 14:14                                         ` Florin Iucha
2007-04-29 19:41                                   ` Rogier Wolff
2007-04-29 20:09                                     ` Peter Zijlstra
2007-04-18 11:38                           ` Trond Myklebust
2007-04-18  9:54                     ` OGAWA Hirofumi
2007-04-18  8:19                 ` Peter Zijlstra
2007-04-18 16:41                   ` Peter Zijlstra

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=20070418033055.GA24044@iucha.net \
    --to=florin@iucha.net \
    --cc=Trond.Myklebust@netapp.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=bunk@stusta.de \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    /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 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.