[dpdk-dev] [PATCH v4] app/pdump: add pudmp exits with primary support.

Varghese, Vipin vipin.varghese at intel.com
Tue Apr 30 12:37:47 CEST 2019


snipped
> > Thanks for the patch work with rte_eal_alaram. But I am not able to
> > find
> >
> > 1. the documentation update.
> > 2. cover letter.
> 
> Why would a single patch need a cover letter? I don't think it's needed in this
> case. The commit message is enough.

In my opinion, the cover letter is to be added as it is new feature and explains the reasoning behind the new change. Please let me know if there change in the same?

snipped


More information about the dev mailing list