Thread (65 messages) 65 messages, 4 authors, 2017-04-07

Re: [PATCH v7 1/7] net/ark: stub PMD for Atomic Rules Arkville

From: Ferruh Yigit <hidden>
Date: 2017-03-31 14:51:28

On 3/29/2017 10:32 PM, Ed Czeck wrote:
Enable Arkville on supported configurations
Add overview documentation
Minimum driver support for valid compile
Arkville PMD is not supported on ARM or PowerPC at this time

v6:
* Address review comments
* Unify messaging, logging and debug macros to ark_logs.h

v5:
* Address comments from Ferruh Yigit [off-list ref]
* Added documentation on driver args
* Makefile fixes
* Safe argument processing
* vdev args to dev args

v4:
* Address issues report from review
* Add internal comments on driver arg
* provide a bare-bones dev init to avoid compiler warnings

v3:
* Split large patch into several smaller ones

Signed-off-by: Ed Czeck <redacted>
Signed-off-by: John Miller <redacted>
Hi Ed,

Can you please update release notes [1] to announce the ARK PMD?


Apart from that, overall PMD looks good to me.

Still I suggest pushing the PMD integration to RC2 phase, to give more
chance for reviews.

If PMD gets acked before that time, we can integrate earlier. If there
is no objection till RC2, it gets merged in RC2.

This means less time for testing after integration, is merging on RC2
will leave you enough time for testing?

Thanks,
ferruh


[1]
doc/guides/rel_notes/release_17_05.rst
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help