[GIT PULL] nvme patches for 4.11

Jens Axboe axboe at kernel.dk
Wed Feb 22 14:18:23 PST 2017


On 02/22/2017 03:16 PM, Jens Axboe wrote:
> On 02/22/2017 03:10 PM, Daniel Roschka wrote:
>> On Wednesday, 22. February 2017, 20:41:11 CET you wrote:
>>> A few issues with this:
>>>
>>> - It throws a merge conflict. No big deal, that was easy enough
>>>   to fix up.
>>>
>>> - What is this based on? Looks like some version of my for-next,
>>>   but ends up duplicating 4 patches from me. I don't remember
>>>   if I rebased my for-next this time around, but I often do. It's
>>>   not a stable branch like the for-4.xx/topic branches.
>>>
>>> - Out of 22 patches, one commit reverts another commit. That should
>>>   just have been squashed.
>>>
>>> I ended up simply fixing all of that up, and generating the patches
>>> and applying. Otherwise the whole history would have been a mess.
>>
>> Urgs, I don't know what went wrong with the pull request from Sagi, but at 
>> least the change in there I authored is fundamently different from what I 
>> submitted!
>>
>> Looking at git://git.infradead.org/nvme.git nvme-4.11, instead of my original 
>> patch adding a device ID for another NVMe controller for MacBooks, this branch 
>> replaces the original entry! That'd be a regression for all MacBook8,1 out 
>> there!
>>
>> See http://git.kernel.dk/cgit/linux-block/commit/?h=for-linus&id=4f6cc1c29b0b5d3a964d5ad0cbe7c48e6c44686f
>> vs. http://lists.infradead.org/pipermail/linux-nvme/2017-February/008323.html
>>
>> How do we get that fixed?
> 
> Beats me how that happened, Sagi? Sagi is the one that applied it. This
> is the commit in the nvme tree:
> 
> http://git.infradead.org/nvme.git/commitdiff/918896679f5ca6ee3462aadaaee53049635769c9?hp=1792eac722b0ab651e9eb543114e929dadb1f613
> 
> I will fix it up in my tree...

http://git.kernel.dk/cgit/linux-block/commit/?h=for-linus&id=124298bd03acebd9c9da29a794718aca31bec1f7

-- 
Jens Axboe




More information about the Linux-nvme mailing list