On Thu, Nov 23, 2023 at 09:40:57AM +0530, Jerin Jacob wrote: > On Tue, Nov 21, 2023 at 11:47 PM Bruce Richardson > <bruce.richard...@intel.com> wrote: > > > > Document explicitly the scheduling types supported by this driver, both > > via info_get() function, and via table in the documentation. > > > > Signed-off-by: Bruce Richardson <bruce.richard...@intel.com> > > --- > > > > Maintainers, please check this patch carefully, as I'm not sure the > > correct way to document this. > > > > According to the docs for this driver, it supports parallel only via > > ordered. Therefore, I've actually made the docs inconsistent from the > > flags claimed in the API. I've documented that PARALLEL is supported in > > the info_get() flags, so code that checks for that will run, but I've > > omitted it from the table in the docs, since it is not directly > > supported. Is this a good compromise, or an accurate reflection of the > > driver? > > --- > > doc/guides/eventdevs/features/opdl.ini | 2 ++ > > drivers/event/opdl/opdl_evdev.c | 3 +++ > > 2 files changed, 5 insertions(+) > > > > diff --git a/doc/guides/eventdevs/features/opdl.ini > > b/doc/guides/eventdevs/features/opdl.ini > > index 5cc35d3c77..7adccc98de 100644 > > --- a/doc/guides/eventdevs/features/opdl.ini > > +++ b/doc/guides/eventdevs/features/opdl.ini > > @@ -4,6 +4,8 @@ > > ; Refer to default.ini for the full list of available PMD features. > > ; > > [Scheduling Features] > > +atomic_scheduling = Y > > +ordered_scheduling = Y > > Missed parallel >
Deliberate omission for now. See note above. Basically, parallel is supported through ordered, so I added the flag below to stop apps from breaking, but I wasn't sure about advertising it in the docs. Will add it if you feel its best to keep them consistent. /Bruce
