On 10/2/2017 12:23 AM, Leon Romanovsky wrote:
Upstream is not a development playground and you should submit your code once you think that it is ready. So we are assuming that VNIC is working and you are interested to debug your EM and such code doesn't belong to the kernel.
This patch aside, I do have to take some issue with this statement. While upstream is not a "playground" we should be submitting code early. I keep harping on iterative development, show your work. Granted that doesn't mean write broken code and toss it over the fence to kernel.org, but pieces that can be broken up and tested on their own are best.
If you still insist on 2.a, the solution should be in your company: add debugfs locally, write tests, find and fix bugs and submit them to upstream.
Now back to this patch series. debugfs vs NetLink, I don't think it really matters if the rationale for having the patch in the first place is wrong. In other words you would have still opposed this even if it were NetLink I assume? We are discussing internally but for now I think this series can be dropped.
There is no need to add "one-time" interface to clean the code.
That I would agree to. -Denny -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
