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. >>