Re: [dpdk-dev] [PATCH] usertools: enhance device bind script module detection

[Thomas Monjalon <thomas@monjalon.net>]

28/10/2019 18:09, Thomas Monjalon:
> 28/10/2019 15:50, Pavan Nikhilesh Bhagavatula:
> > >21/09/2019 20:16, pbhagavatula@marvell.com:
> > >> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> > >>
> > >> Some kernel modules have '-' in their name when populated in
> > >> '/sys/bus/pci/drivers/' where as the kernel always populates
> > >> '/sys/modules/' with '_'.
> > >> Replace '-' in module name with '_' to avoid script complaining
> > >> that module not loaded.
> > >
> > >Sorry it does not look obvious to me.
> > >Please, could you describe what is wrong and what is the real cause?
> > >
> > 
> > dpdk-devbind.py can also be used to bind devices to kernel modules.
> > Some kernel modules are registered with '-' in their name and are populated 
> > with '-' in '/sys/bus/pci/drivers/' but kernel populates the name in /sys/modules/ as '_'
> 
> Sorry, still not clear.
> What do you mean by "registered"?
> 
> > This confuses the current script making it unable to bind to such modules. The patch 
> > addresses it by always replacing '-' with '_' when looking in /sys/modules/
> 
> Yes this sentence is clear (except it is /sys/module/).
> Please send a v2 with a clear explanation.


Sorry, forgot to insist:
You must start the description with what is wrong.
You can describe the bug you are experiencing.

Thanks