From: "Ed L. Cashin" <ecashin@coraid.com>
To: linux-kernel@vger.kernel.org
Cc: Greg KH <greg@kroah.com>,
boddingt@optusnet.com.au, Andrew Morton <akpm@osdl.org>,
bugme-daemon@bugzilla.kernel.org,
Christoph Hellwig <hch@infradead.org>
Subject: bio pages with zero page reference count
Date: Mon, 18 Dec 2006 17:21:09 -0500 [thread overview]
Message-ID: <20061218222109.GA23156@coraid.com> (raw)
In-Reply-To: <20061218175300.GM23156@coraid.com>
(This email is a followup to "Re: [PATCH 2.6.19.1] fix aoe without
scatter-gather [Bug 7662]".)
On Mon, Dec 18, 2006 at 12:53:00PM -0500, Ed L. Cashin wrote:
...
> This patch eliminates the offset data on cards that don't support
> scatter-gather or have had scatter-gather turned off. There remains
> an unrelated issue that I'll address in a separate email.
After fixing the problem with the skb headers, we noticed that there
were still problems when scatter gather wasn't in use. XFS was giving
us bios that had pages with a reference count of zero.
The aoe driver sets up the skb with the frags pointing to the pages,
and when scatter gather isn't supported and __pskb_pull_tail gets
involved, put_page is called after the data is copied from the pages.
That causes problems because of the zero page reference count.
It seems like it would always be incorrect for one part of the kernel
to give pages with a zero reference count to another part of the
kernel, so this seems like a bug in XFS.
Christoph Hellwig, though, points out,
> It's a kmalloced page. The same can happen with ext3 aswell, but
> only when doing log recovery. The last time this came up (vs
> iscsi) the conclusion was that the driver needs to handle this
> case.
In attempting to find the conversation he was referencing, I only
found this:
Subject: tcp_sendpage and page allocation lifetime vs. iscsi
Date: 2005-04-25 17:02:59 GMT
http://article.gmane.org/gmane.linux.kernel/298377
If anyone has a better reference, I'd like to see it.
--
Ed L Cashin <ecashin@coraid.com>
next prev parent reply other threads:[~2006-12-18 22:21 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20061209234305.c65b4e14.akpm@osdl.org>
2006-12-18 17:53 ` [PATCH 2.6.19.1] fix aoe without scatter-gather [Bug 7662] Ed L. Cashin
2006-12-18 22:21 ` Ed L. Cashin [this message]
2006-12-18 22:53 ` bio pages with zero page reference count Christoph Hellwig
2007-01-19 16:21 ` Ed L. Cashin
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=20061218222109.GA23156@coraid.com \
--to=ecashin@coraid.com \
--cc=akpm@osdl.org \
--cc=boddingt@optusnet.com.au \
--cc=bugme-daemon@bugzilla.kernel.org \
--cc=greg@kroah.com \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=support@coraid.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