Mon, Jan 21, 2019 at 12:07:14PM CET, era...@mellanox.com wrote:
>
>
>On 1/20/2019 8:45 PM, David Miller wrote:
>> From: Jiri Pirko <j...@resnulli.us>
>> Date: Sun, 20 Jan 2019 12:08:50 +0100
>> 
>>> I haven't have time to review this due to travel. I think it was mistake
>>> to merge this as the buffer api is wrong in my opinion. I would vote for
>>> revert if possible.
>> 
>> Let's spend a couple days trying to fix things up first.
>
>will do.
>I reviewed the comments and saw that the major comment is around the 
>"Devlink buffers", which Jiri asked to replace with "Devlink msg".
>
>The issue is that there is no easy way to modify one into the other 
>without breaking things, so I am planning to provide a series on top 
>which does:
>1. Add devlink msg API
>2. Move devlink reporter to use new API
>3. Move existing reporter to use the new API
>4. Get rid of the old API

Will be harder to review then revert and redo. Also harder for you to do
the patches. Small nightmare :/

>
>thanks,
>Eran
>
>> 
>> I cannot make contributors hostage to other people's travel
>> schedule, no way.
>> 
>> If it seems hopeless by Wednesday, I will revert.
>
>Should be published by then, hopefully sooner.
>
>> 
>> Thanks.
>> 

Reply via email to