From: Boaz Harrosh <bharrosh@panasas.com>
To: <tao.peng@emc.com>
Cc: <bergwolf@gmail.com>, <bhalevy@tonian.com>,
<Trond.Myklebust@netapp.com>, <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH-RESEND 4/4] pnfsblock: do not ask for layout in pg_init
Date: Wed, 7 Dec 2011 16:08:58 +0200 [thread overview]
Message-ID: <4EDF737A.2080400@panasas.com> (raw)
In-Reply-To: <F19688880B763E40B28B2B462677FBF805E5043F8B@MX09A.corp.emc.com>
On 12/02/2011 06:59 AM, tao.peng@emc.com wrote:
>>
> I'm sorry but I don't get it... How do you count the first contiguous
> chunk in .pg_init? All you know at .pg_init is an empty io desc and
> an nfs page. Are you going to scan the radix tree and find the next
> page that write_cache_pages is going to flush out? I don't really
> think anyone will agree on it... Please look into the second part of
> your solution and then you'll see why I am worried that it may never
> get done.>
> Best Regards,
> Tao
>
Rrrr, one thing I cannot do is make some one understand.
I guess I'll have to do it my self. It's so simple I'm go'ne cry.
I have more pressed things on my plate then fix broken blocks servers
but you are forcing my hands. The last code you sent is not acceptable
because it is totally not usable by objects and files (segments).
Give me a week. Not that it should take longer then an hour but
because I have burning matters to attend to first.
Boaz
>> Boaz
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2011-12-07 14:09 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-03 4:56 [PATCH-RESEND 4/4] pnfsblock: do not ask for layout in pg_init Peng Tao
2011-11-29 17:48 ` Jim Rees
2011-11-30 5:43 ` tao.peng
2011-11-30 12:57 ` Benny Halevy
2011-11-30 13:17 ` Peng Tao
2011-12-01 1:18 ` Boaz Harrosh
2011-12-01 5:05 ` tao.peng
2011-12-01 9:57 ` Benny Halevy
2011-12-01 17:33 ` Boaz Harrosh
2011-12-01 18:00 ` Boaz Harrosh
2011-12-02 4:59 ` tao.peng
2011-12-07 14:08 ` Boaz Harrosh [this message]
2011-12-08 3:32 ` tao.peng
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=4EDF737A.2080400@panasas.com \
--to=bharrosh@panasas.com \
--cc=Trond.Myklebust@netapp.com \
--cc=bergwolf@gmail.com \
--cc=bhalevy@tonian.com \
--cc=linux-nfs@vger.kernel.org \
--cc=tao.peng@emc.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;
as well as URLs for NNTP newsgroup(s).