Hi Florian
thank you, I have applied your patch

Alfredo

On Apr 5, 2013, at 6:42 PM, Florian Wohlfart <[email protected]> wrote:

> On 03.04.2013 14:42, Alfredo Cardigliano wrote:
>> Hi Florian
>> I have fixed the compilation errors on both pf_ring and the ixgbe-dna driver 
>> (we will patch the new version asap). 
>> I did not test them yet, please update from svn and let me know.
>> 
> 
> Hi Alfredo,
> 
> thank you for your quick reaction! Your patch works, now I can compile
> PF_RING on kernel 3.7 without errors.
> 
> Your ixgbe patch also solves the issue of the kernel version number
> check in the makefile. When trying to compile the latest ixgbe-dna
> driver from svn, I got another error:
> 
> /root/PF_RING/drivers/DNA/ixgbe-3.10.16-DNA/src/ixgbe_main.c: In
> function ‘ixgbe_io_error_detected’:
> /root/PF_RING/drivers/DNA/ixgbe-3.10.16-DNA/src/ixgbe_main.c:9343:22:
> error: ‘struct pci_dev’ has no member named ‘pcie_type’
> 
> I compared the function "ixgbe_io_error_detected" with the same function
> in the latest vanilla ixgbe-3.14.5 driver. I noticed, that die line
> causing the error was changed in the latest driver. So I replaced line
> 9343 from:
> 
>   while (bdev && (bdev->pcie_type != PCI_EXP_TYPE_ROOT_PORT))
> 
> to
> 
>   while (bdev && (pci_pcie_type(bdev) != PCI_EXP_TYPE_ROOT_PORT))
> 
> 
> I don't know the implications of this change, but afterwards I could
> eventually compile the ixgbe-3.10.16-DNA without an error. I hope this
> helps!
> 
> Thank you for your efforts!
> Florian
> 
> 
>> Thank you
>> Alfredo
>> 
>> On Apr 2, 2013, at 7:14 PM, Florian Wohlfart <[email protected]> wrote:
>> 
>>> Hello ntop-developers,
>>> 
>>> When I tried to compile the latest trunk version of PF_RING for kernel
>>> 3.7, I got the error that "VM_RESERVED" is not defined. It seems like
>>> this flag was removed in kernel 3.7 and should be replaced by
>>> "(VM_DONTEXPAND | VM_DONTDUMP)". (taken from
>>> https://www.virtualbox.org/pipermail/vbox-dev/2012-October/005392.html)
>>> 
>>> Inserting the following lines into "PF_RING/kernel/pf_ring.c" worked for me:
>>> 
>>> #ifndef VM_RESERVED
>>> # define VM_RESERVED (VM_DONTEXPAND | VM_DONTDUMP)
>>> #endif
>>> 
>>> 
>>> Furthermore, the DNA driver ixgbe-3.10.16-DNA (which is included in the
>>> PF_RING sources) did not compile against kernel 3.7. The latest Intel
>>> driver (ixgbe-3.14.5) compiles, so the best solution would be to apply
>>> the DNA patches to the latest Intel driver. Are there any plans to
>>> provide a newer version of the patched ixgbe driver?
>>> 
>>> Florian
>>> 
>>> -- 
>>> Florian Wohlfart, M.Sc.
>>> Technische Universität München
>>> Department of Computer Science
>>> Network Architectures and Services
>>> _______________________________________________
>>> Ntop-dev mailing list
>>> [email protected]
>>> http://listgateway.unipi.it/mailman/listinfo/ntop-dev
>> 
>> _______________________________________________
>> Ntop-dev mailing list
>> [email protected]
>> http://listgateway.unipi.it/mailman/listinfo/ntop-dev
>> 
> 
> 
> -- 
> Florian Wohlfart, M.Sc.
> Technische Universität München
> Department of Computer Science
> Network Architectures and Services
> Phone: +49 89 289 18019, Room: 03.05.043

_______________________________________________
Ntop-dev mailing list
[email protected]
http://listgateway.unipi.it/mailman/listinfo/ntop-dev

Reply via email to