On 1/2/16, 1:14 PM, "Stephen Hemminger" <stephen at networkplumber.org> wrote:

>On Sat, 2 Jan 2016 19:52:16 +0100
>Jan Viktorin <viktorin at rehivetech.com> wrote:
>
>> On Sat, 2 Jan 2016 18:35:08 +0000
>> "Wiles, Keith" <keith.wiles at intel.com> wrote:
>> 
>> > >Yes, DPDK needs to work in embedded environments with device tree.
>> > >Would it be possible reimplement device tree parsing in user space?
>> > >Ideally with a shared code from kernel??  
>> > 
>> > Stephen, Do you mean we have to add kernel code to support DPDK on SoC 
>> > Platforms? If that is the case I would like to not require code be added 
>> > to the kernel to support DPDK.
>> 
>> We will need a kernel module. But this is not necessarily related to the
>> device-tree parsing.
>> 
>> > >
>> > >On a pratical level, the new SoC support must be optional
>> > >(via DPDK config infrastructure), since most architectures won't be using 
>> > >it.
>> > >In most cases, it is better from usability if everything is runtime based,
>> > >but with SoC this is a platform/architecture configuration.  
>> > 
>> > I am not sure I agree with the optional support, as it could be stated 
>> > that PCI support is optional on SoC platforms. It would be best to not 
>> > treat SoC support as special compared to PCI support. Other then extra 
>> > footprint it does not seem reasonable to require SoC support to be 
>> > ifdef?ed in the code. Plus adding more ifdefs is not a good testing 
>> > solution.
>> 
>> This is a matter of preserving ABI. Turning PCI-support to be optional
>> seems to be a difficult step at the moment.
>> 
>> > 
>> > Can we detect somehow we are on a system with SoC support or even a system 
>> > that supports PCI for that matter?
>> 
>> IMO, we can detect two things: "PCI is present on the system" and
>> "Device tree is accessible in /proc/device-tree". Is this acceptable?
>> 
>
>I am just as concerned with building and having useless code.
>For now most environments can just use PCI, and having to carry around
>dead code seems wasteful and potential for some security abuse as well.

Hi Stephen,

With including every archive with the include whole archive option means we 
already have a huge amount of dead code in a DPDK image :-( Adding a bit more 
is not going to make a difference IMO. Also a system without PCI could be a 
security abuse too.

I think we just figure out how not to call the PCI or SoC code at runtime if 
not supported and compile it in always.
>


Regards,
Keith




Reply via email to