Re: [PATCH v3] Grow: Split Grow_reshape into helper function

2022年8月25日 00:33，Jes Sorensen [off-list ref] 写道：

On 8/24/22 12:09, Coly Li wrote:

quoted

quoted

2022年8月24日 23:56，Jes Sorensen [off-list ref] 写道：

quoted

Hi Jes,

Please check the version I post to you in series “mdadm-CI for-jes/20220728: patches for merge” (Message-Id: [ref]), the patch in this series is rebased and confirmed with Mateusz, it could be applied to upstream mdadm.

quoted

I applied this one, but none of the versions applied cleanly. I had to
play formail games to pull it out of your stack, as I am not going to
apply a set of 23 commits in one batch without going through them.

These days I was in partner’s office and planed to repost the rebased version soon. If you don’t do the rebase yet, please wait for me to post a v4 version on behavior of Mateusz tomorrow.

No worries, I already pulled some of it in, but you can check my repo
and see whats there.

quoted

quoted

It's really awesome to have your help reviewing patches, much
appreciated, but I would prefer to keep them in the original batches so
I can pull them from patchwork, rather than trying to deal with the
giant stack.

How about we improve the process like this,
1) I will continue to review and response the patches from the original emails, so patch work may track them as they were.
2) For all the reviewed patches are not handled by your after a period, let’s set it as 2 weeks for now, I will post a email with all the patches with their message-IDs to you as a remind.

This sounds good to me! I think it's also fine to have a branch with
everything applied for testing, it's just less easy for me to pull from.

Yes there is already. Every time when I post the reviewed patches to you, there is a branch for all the patches. For example currently the branches in mdadm-CI tree,
  remotes/origin/20220315-testing
  remotes/origin/20220406-testing
  remotes/origin/20220719-testing
  remotes/origin/for-jes/20220402
  remotes/origin/for-jes/20220620
  remotes/origin/for-jes/20220728
  remotes/origin/master

All the branches in for-jes/ are the patches I posted to you as email thread. You may take any one for your convenience. And the latest branch will be rebased against the latest mdadm tree before I post the remind email to you.

BTW, the mdadm-CI tree can be found from git://git.kernel.org/pub/scm/linux/kernel/git/colyli/mdadm.git

Thanks.

Coly Li