Re: [PATCH 0/4] lightnvm: pblk: add support for chunk metadata on erase

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/03/2018 11:16 AM, Javier Gonzalez wrote:
On 31 Aug 2018, at 15.57, Matias Bjørling <mb@xxxxxxxxxxx> wrote:

On 08/31/2018 03:34 PM, Javier González wrote:
Matias,
This patchset implements support for retrieving chunk metadata when
submitting a reset/erase command. Patches 0 and 1 are small fixes that
can be directly merged into your patch:
      lightnvm: move bad block and chunk state logic to core
Also, note that these do not apply on top of your for-4.20/core due them
depending on patches that I sent before that you have not picked up yet.
You can see them though in for-4.20/pblk. I'll rebase as patches in the
list appear in your tree.

Thanks. It is really confusing when you guys maintains an implicit order and posts the patches separately. I will appreciate that patches that are related are posted together, such that I don't have to manually track what comes before another. That makes it less of a pain for me to keep track of and we can keep the reviews together.

This is the patches that I have in the pipeline (from before the e-mails from today):

  - This serie - Pending review
  - Serie: pblk: support variable OOB size - Waiting on review from Igor
  - lightnvm: pblk: recover open lines on 2.0 devices. Which doesn't apply due to the fixes to the pad distance patch.


Yes, I know and I apologize - we should have a better flow. What do you
say that for windows like this, where we have a number of patches that
have dependencies that we post them in meaningful patchsets and point to
a branch where they are ordered, like in a PR? Then we can rebase and
propagate changes properly?

I am with the patchset posted, that should have the order. I just wanted to mention it. One thing that would be good, if you do have patches you have upstream, feel free to push them in smaller increments, so we can pull them in as we go. Only a nitpick, it is obviously up to you guys how you want to do it :)


For this window, I'll rebase the rest of the patches in for-4.20/pblk on
top of your for-4.20/core, then we can propagate changes as they come.

Javier





[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux