On 10/24/2017 4:56 PM, Ferruh Yigit wrote: > On 10/24/2017 3:35 AM, Gaetan Rivet wrote: >> This flag was deprecated in the previous release, and this is its removal. > > This change was in wait list, but it would be much more better if not received > just before rc2, but in early stages in the release, at worst before rc1. > > Although patch looks good to me, getting ethdev layer update in rc2 is > creating > some stress. > >> We are going from an explicit support of device detach, to an implicit one, >> with specific checks in place for PMDs not supporting the feature. > > Removing this flag implies all ethdev devices are detachable, right. > That is why you need to update error value handling in pci hotplug code, and > return error in nfp remove path, since there is no way to say not detachable. > >> >> Attempting to Cc all relevant maintainers: >> >> Cc: Ajit Khaparde <[email protected]> >> Cc: Alejandro Lucero <[email protected]> >> Cc: Beilei Xing <[email protected]> >> Cc: Bruce Richardson <[email protected]> >> Cc: Cristian Dumitrescu <[email protected]> >> Cc: Declan Doherty <[email protected]> >> Cc: Ferruh Yigit <[email protected]> >> Cc: Jing Chen <[email protected]> >> Cc: John Miller <[email protected]> >> Cc: John W. Linville <[email protected]> >> Cc: Konstantin Ananyev <[email protected]> >> Cc: Matt Peters <[email protected]> >> Cc: Nelio Laranjeiro <[email protected]> >> Cc: Pascal Mazon <[email protected]> >> Cc: Shrikrishna Khare <[email protected]> >> Cc: Srisivasubramanian Srinivasan <[email protected]> >> Cc: Tetsuya Mukawa <[email protected]> >> Cc: Wenzhuo Lu <[email protected]> >> Cc: Yongseok Koh <[email protected]> >> >> Sorry for any undue ML traffic going your way if you are not interested >> in this series. >> >> I'd like to highlight Alejandro Lucero for the NFP PMD, as the patch is >> somewhat more involved for this PMD and I don't have the hardware available >> to validate the change. > > Let's give one more day to Alejandro to comment on NFP update. > > And others to comment on overall, if there is no objection include this into > rc2.
btw, not directly related to this patch, but related to the hotplug, testpmd crashes after detaching a port and start forwarding, anyone have bandwidth to look at this? > >> >> v2: >> >> + Split ether patch: >> First remove the use of the flag, then remove the flag >> >> + Restore RTE_ETH_DEV_BONDED_SLAVE check, >> previously hastily removed along the DETACHABLE one. >> >> + Added release note, removed deprecation notice >> >> Gaetan Rivet (6): >> ethdev: do not rely on detachable flag in detach >> ethdev: remove detachable device flag >> pci: propagate PMD removal error value for unplug >> doc: follow detachable flag disappearance >> doc: update detachable flag deprecation notice >> doc: note ether API change for detachable flag > > <...> >
