-----Original Message----- > Date: Mon, 17 Sep 2018 21:17:34 +0200 > From: Mattias Rönnblom <mattias.ronnb...@ericsson.com> > To: Jerin Jacob <jerin.ja...@caviumnetworks.com> > CC: bruce.richard...@intel.com, dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v3 10/10] event/dsw: include DSW event > device documentation > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 > Thunderbird/52.9.1 > > External Email > > On 2018-09-17 14:29, Jerin Jacob wrote: > > > > diff --git a/doc/guides/eventdevs/dsw.rst b/doc/guides/eventdevs/dsw.rst > > > new file mode 100644 > > > index 000000000..de41ae9d3 > > > --- /dev/null > > > +++ b/doc/guides/eventdevs/dsw.rst > > > @@ -0,0 +1,97 @@ > > > +.. SPDX-License-Identifier: BSD-3-Clause > > > + Copyright(c) 2017 Intel Corporation. > > > > Is adding Intel copyright by intention? > > > > dsw.rst is a derived work of sw.rst, so I figured I should leave the > Intel copyright. But maybe that's not needed?
Looks so. > > > > + Copyright(c) 2018 Ericsson AB > > > + > > > +Distributed Software Eventdev Poll Mode Driver > > > +============================================== > > > + > > > +The distributed software eventdev is a parallel implementation of the > > > > s/parallel/driver?? > > > > I'm not sure I follow. What do you mean? > > "Parallel" here is as in executing on multiple CPU cores. I read "Parallel" as "alternative" on that context. So I think you can elaborate a bit, like ending sentence "where scheduler is executing on multiple CPU cores" or something similar what you like to add more clarity. > > I could always add "driver". OK. > > Regards, > Mattias
