On Thu, 2015-01-29 at 14:51 +0200, Or Gerlitz wrote: > On 1/27/2015 7:02 PM, Doug Ledford wrote: > > [...] > > I haven't heard an argument from you yet that I believe beats the points > > I've made above. So I believe a solution that does not revert back to > > having two separate code paths to be maintained is preferable to your patch. > > Doug, > > It's not going to work this way. > > We should 1st and most take a decision what gonna happen with the > 3.19driver
Yes. And that's what my above comment was about, the 3.19 driver. My opinion that I stated then, and I'll reiterate now, is that we should revert the original 8 or pull in my full patchset. > and only then restart/resume the multi way/arguments discussion, and we > have > very little time for 3.20-rc1too, BTW. > > So lets close the 3.19 saga, again, either revert your eight patches I would support that option. > or > apply Erez's patch. I disagree with this option. You've chosen one regression to highlight, but ignored other regressions, and Erez's patch by itself does not address those other regressions. You can't go picking and choosing which regressions to highlight and ignore if your rationale for justification is that we don't allow regressions in releases. Certainly, failure to unload the module or reboot due to a hand in ib_sa_unregister_client would be considered a regression too, yes? But that isn't addressed by Erez's patchset. I'm tired of arguing about this Or. You will not change my mind. Erez's patch is a bandaid that only solves one particular issue while ignoring others, it doesn't have half the testing my patchset has, it doesn't address half the issues mine does, and with it in place there are still glaring problems left for the end user to suffer through. As much as it pains me to admit it, my original patchset had issues that were pretty severe, and a bandaid *does not do the job*. It takes some honest to god stitching to fix that up. If my much more complete fixup is "too much, too late", so be it. We revert the original 8 patches. But a bandaid on a katana slice doesn't cut it. > This way or another for 3.20 we can do things right by whatever method > we agree on. Agreed. -- Doug Ledford <[email protected]> GPG KeyID: 0E572FDD
signature.asc
Description: This is a digitally signed message part
