Thread (14 messages) 14 messages, 6 authors, 2016-10-19

Re: [PATCH] drivers: make driver names consistent

From: Thomas Monjalon <hidden>
Date: 2016-10-18 16:46:41

2016-10-18 22:18, Yuanhan Liu:
On Tue, Oct 18, 2016 at 03:42:54PM +0200, Thomas Monjalon wrote:
quoted
2016-10-18 21:06, Yuanhan Liu:
quoted
On Tue, Oct 18, 2016 at 02:50:16PM +0200, Jan Blunck wrote:
quoted
quoted
From my understanding this is a massive API breakage. This forces all
existing users of the virtual PMDs to change with zero benefit. Even
if that isn't enough it also makes it impossible to switch between
releases by recompiling.

Can we please revert these changes and work on some aliasing support
for the PMDs to fix it long term?
+1. Aliasing is also something I would suggest before making such renames.
It is a brutal change, yes.
It was announced in 16.07 release notes though.
Yes, but it still took me a while (by running git bisect) to figure out
what went wrong: I wasn't aware of such note, that I was thinking maybe
something is broken.

Later I also got quite few same complains. It may also took them a while
to know what's happened.

Anyway, my point is, for this kind of change, we should have added the
alias support firstly.
Yes.
If that's been done, then the announcement is not
needed at all?
The announcement would be needed to remove the aliases, later.

quoted
We can try to make this change more progressive by keeping old names
as aliases for some time.
Is there a volunteer to work on vdev names aliases,
with the target of integrating them in RC2 or RC3?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help