From: Allison Henderson <achender@linux.vnet.ibm.com>
To: Andrea Arcangeli <aarcange@redhat.com>
Cc: Yongqiang Yang <xiaoqiangnk@gmail.com>,
linux-ext4@vger.kernel.org, Theodore Tso <tytso@mit.edu>,
Jan Kara <jack@suse.cz>
Subject: Re: [PATCH 2/2] ext4: ext4_discard_partial_page_buffers_no_lock() wrong parameters
Date: Mon, 12 Dec 2011 13:07:47 -0700 [thread overview]
Message-ID: <4EE65F13.3030800@linux.vnet.ibm.com> (raw)
In-Reply-To: <20111212165517.GE16411@redhat.com>
On 12/12/2011 09:55 AM, Andrea Arcangeli wrote:
> On Mon, Dec 12, 2011 at 11:17:54AM +0800, Yongqiang Yang wrote:
>> Hi Andrea,
>>
>> The code you are testing are removed in recent patches, the patches
>> have not been merged.
>
> I checked out the git ext4 tree and the two patches applies clean to
> it. So I'm unsure why you say it was removed. origin/dev origin/master
> are equal too.
>
>>
>> Please try following patches:
>> [PATCH 1/2] ext4: let mpage_submit_io works well when blocksize< pagesize
>> [PATCH 2/2] ext4: let ext4_discard_partial_buffers handle pages
>> without buffers correctly
>>
>> and
>>
>> [PATCH 1/2] ext4: remove a wrong BUG_ON in ext4_ext_convert_to_initialized
>> [PATCH 2/2] ext4: let ext4_bio_write_page handle EOF correctly
>
> Is there a git tree with these patches? I'm not subscribed to the list.
>
> Thanks,
> Andrea
Hi Andrea,
I think what Yongqiang means is that the code you are modifying is being
removed in the patches listed above and replaced with a different
solution. The patches have not yet been merged so you will not see them
in a git tree yet, so you will need to apply them yourself. If you are
not subscribed to the list, you can still find the patches here:
http://www.spinics.net/lists/linux-ext4/msg29109.html
http://www.spinics.net/lists/linux-ext4/msg29110.html
http://www.spinics.net/lists/linux-ext4/msg29375.html
http://www.spinics.net/lists/linux-ext4/msg29376.html
We want to make sure the solution works for everyone, so please apply
these patches and let us know if it corrects the issues you are seeing.
Thx!
Allison Henderson
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ext4" 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-12 20:08 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-12 2:27 ext4_da_write_end() and copied == 0 Andrea Arcangeli
2011-12-12 2:27 ` [PATCH 1/2] ext4: avoid hangs in ext4_da_should_update_i_disksize() Andrea Arcangeli
2011-12-12 3:28 ` Yongqiang Yang
2011-12-12 16:57 ` Andrea Arcangeli
2011-12-13 1:05 ` Yongqiang Yang
2011-12-12 20:27 ` Jan Kara
2011-12-14 2:44 ` Ted Ts'o
2011-12-12 2:27 ` [PATCH 2/2] ext4: ext4_discard_partial_page_buffers_no_lock() wrong parameters Andrea Arcangeli
2011-12-12 3:17 ` Yongqiang Yang
2011-12-12 16:55 ` Andrea Arcangeli
2011-12-12 20:07 ` Allison Henderson [this message]
2011-12-13 0:40 ` Andrea Arcangeli
2011-12-13 1:16 ` Yongqiang Yang
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=4EE65F13.3030800@linux.vnet.ibm.com \
--to=achender@linux.vnet.ibm.com \
--cc=aarcange@redhat.com \
--cc=jack@suse.cz \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=xiaoqiangnk@gmail.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 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.