Re: [RFC, PATCH]: Pass link level header from/to PPP interface

2008-02-14 Thread Urs Thuermann
David Miller [EMAIL PROTECTED] writes: Can tcpdump and libpcap already handle PPP properly? Yes, with the (admittedly non-serious) limitation, that both, tcpdump and tcpdump -e don't show the PPP header. I don't remember how tcpdump behaved on BSD in this respect (so many years ago), but I

Re: [RFC, PATCH]: Pass link level header from/to PPP interface

2008-02-13 Thread Urs Thuermann
Hello Dave, But if libpcap and tcpdump can already identify PPP packets then, besides consistency, what does this buy us other than potential breakage? My reason for the patch is mostly cleanliness and to make the kernel match what is described in packet(7). If you open a PF_PACKET/SOCK_RAW

[PATCH] Fix comment for skb_pull_rcsum

2008-02-10 Thread Urs Thuermann
Fix comment for skb_pull_rcsum Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Index: net-2.6/net/core/skbuff.c === --- net-2.6.orig/net/core/skbuff.c 2008-02-06 22:17:58.0 +0100 +++ net-2.6/net/core/skbuff.c 2008

[RFC, PATCH]: Pass link level header from/to PPP interface

2008-02-10 Thread Urs Thuermann
, this is what SOCK_DGRAM is for. Currently SOCK_RAW and SOCK_DGRAM are the same although the packet(7) man page states that with SOCK_RAW packets are passed without any changes. This makes having SOCK_RAW besides SOCK_DGRAM useless for PPP. So what is your opinion about this change? Signed-off-by: Urs

[patch 0/3] CAN: Some updates for 2.6.25

2008-02-06 Thread urs . thuermann
The following patches are intended for 2.6.25. -- -- To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[patch 1/3] CAN: Clean up module auto loading

2008-02-06 Thread urs . thuermann
Remove local char array to construct module name. Don't call request_module() when CONFIG_KMOD is not set. Signed-off-by: Urs Thuermann [EMAIL PROTECTED] Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] --- net/can/af_can.c | 18 ++ 1 file changed, 6 insertions(+), 12

[patch 2/3] CAN: Move proto_{,un}register() out of spin-locked region

2008-02-06 Thread urs . thuermann
The implementation of proto_register() has changed so that it can now sleep. The call to proto_register() must be moved out of the spin-locked region. Signed-off-by: Urs Thuermann [EMAIL PROTECTED] Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] --- net/can/af_can.c | 27

[patch 3/3] CAN: Minor clean-ups

2008-02-06 Thread urs . thuermann
Remove unneeded variable. Rename local variable error to err like in all other places. Some white-space changes. Signed-off-by: Urs Thuermann [EMAIL PROTECTED] Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] --- net/can/raw.c | 24 +--- 1 file changed, 9 insertions(+), 15

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-11-16 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 0/7] CAN: New PF_CAN protocol family for 2.6.25, update

2007-11-16 Thread Urs Thuermann
. * Cleanup of timer code. This patch series applies against net-2.6.25 and is derived from Subversion revision r560 of http://svn.berlios.de/svnroot/repos/socketcan. It can be found in the directory http://svn.berlios.de/svnroot/repos/socketcan/trunk/patch-series/version. Urs Thuermann Oliver

[PATCH 7/7] CAN: Add documentation

2007-11-16 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 629 ++ 2 files

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-11-16 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 13 net/can/Makefile|3 net/can/bcm.c | 1561

[PATCH 2/7] CAN: Add PF_CAN core module

2007-11-16 Thread Urs Thuermann
This patch adds the CAN core functionality but no protocols or drivers. No protocol implementations are included here. They come as separate patches. Protocol numbers are already in include/linux/can.h. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL

[PATCH 6/7] CAN: Add maintainer entries

2007-11-16 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.25

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-11-16 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if.h |4

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-11-16 Thread Urs Thuermann
David Miller [EMAIL PROTECTED] writes: I really frown upon these local debugging macros people tend to want to submit with their changes. It really craps up the tree, even though it might be useful to you. We have now removed the debug code completely. The code is quite stable and we won't

[PATCH 3/7] CAN: Add raw protocol

2007-11-16 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 11 net/can/Makefile|3 net/can/raw.c | 763

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-11-15 Thread Urs Thuermann
Stephen Hemminger [EMAIL PROTECTED] writes: +#ifdef CONFIG_CAN_DEBUG_CORE +extern void can_debug_skb(struct sk_buff *skb); +extern void can_debug_cframe(const char *msg, struct can_frame *cframe); +#define DBG(fmt, args...) (DBG_VAR 1 ? printk( \ +

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-11-15 Thread Urs Thuermann
Joe Perches [EMAIL PROTECTED] writes: On Thu, 2007-11-15 at 08:40 +0100, Oliver Hartkopp wrote: Stephen Hemminger wrote: +#ifdef CONFIG_CAN_DEBUG_CORE +extern void can_debug_skb(struct sk_buff *skb); +extern void can_debug_cframe(const char *msg, struct can_frame *cframe); +#define

[PATCH 6/7] CAN: Add maintainer entries

2007-11-14 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.25

[PATCH 7/7] CAN: Add documentation

2007-11-14 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 637 ++ 2 files

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-11-14 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 13 net/can/Makefile|3 net/can/bcm.c | 1763

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-11-14 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 3/7] CAN: Add raw protocol

2007-11-14 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 11 net/can/Makefile|3 net/can/raw.c | 811

[PATCH 0/7] CAN: New PF_CAN protocol family for 2.6.25

2007-11-14 Thread Urs Thuermann
in the directory http://svn.berlios.de/svnroot/repos/socketcan/trunk/patch-series/version. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp The changes in try #10 were: * Rename our static debug variables to {can,raw,bcm,vcan}_debug. * Use module_param_named() so

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-11-14 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if.h |4

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-11-14 Thread Urs Thuermann
, replaced return 0 by NETDEV_TX_OK. The new patch is below. urs This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann

[PATCH] [AF_PACKET] Fix minor code duplication

2007-11-08 Thread Urs Thuermann
Simplify some code by eliminating duplicate if-else clauses in packet_do_bind(). Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- net-2.6/net/packet/af_packet.c.orig 2007-11-05 13:07:28.0 +0100 +++ net-2.6/net/packet/af_packet.c 2007-11-08 12:14:25.0 +0100 @@ -886,20

Re: [PATCH] - trivial - Improve appletalk checksum calculation

2007-10-28 Thread Urs Thuermann
Herbert Xu [EMAIL PROTECTED] writes: But your code differs significantly from Stephen's version. However, if it is correct it does look like a good improvement. So please write a simple test program. It can't that bad since there are only 65536 values to test :) I think the code is simple

Re: net-2.6.24 rebased...

2007-10-09 Thread Urs Thuermann
Hallo Dave, 9.1MB, 739 changesets, keep those patches flowing! Last week I have sent another version of our patch series for PF_CAN. The changes after the last review feedback were only cosmetics. Do you have any plans with that code for this or the next release? Regards, urs - To unsubscribe

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-10-05 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 13 net/can/Makefile|3 net/can/bcm.c | 1763

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-10-05 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 0/7] CAN: Add new PF_CAN protocol family, try #10

2007-10-05 Thread Urs Thuermann
://svn.berlios.de/svnroot/repos/socketcan/trunk/patch-series/version. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

[PATCH 6/7] CAN: Add maintainer entries

2007-10-05 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.24

[PATCH 3/7] CAN: Add raw protocol

2007-10-05 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 11 net/can/Makefile|3 net/can/raw.c | 810

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-10-05 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if.h |4

[PATCH 7/7] CAN: Add documentation

2007-10-05 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 637 ++ 2 files

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-10-04 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: +struct sockaddr_can { + sa_family_t can_family; + int can_ifindex; + union { + struct { canid_t rx_id, tx_id; } tp16; + struct { canid_t rx_id, tx_id; } tp20; + struct { canid_t rx_id,

Re: [PATCH 3/7] CAN: Add raw protocol

2007-10-04 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: +static inline struct raw_sock *raw_sk(const struct sock *sk) +{ + return (struct raw_sock *)sk; +} What if I want to do some kernel module that uses INET raw sockets (include/net/icmp.h) and CAN raw sockets? Namespace collision,

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-10-04 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: +#ifdef CONFIG_CAN_DEBUG_DEVICES +static int debug; +module_param(debug, int, S_IRUGO); +#endif Can debug be a boolean? Like its counterpart on DCCP: debug used to a bit mask, like it still is in core.h. You can see this in the test

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-10-02 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if.h |4

[PATCH 3/7] CAN: Add raw protocol

2007-10-02 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 11 net/can/Makefile|3 net/can/raw.c | 821

[PATCH 0/7] CAN: Add new PF_CAN protocol family, try #9

2007-10-02 Thread Urs Thuermann
regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 7/7] CAN: Add documentation

2007-10-02 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 637 ++ 2 files

[PATCH 6/7] CAN: Add maintainer entries

2007-10-02 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.24

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-10-02 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-10-02 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 13 net/can/Makefile|3 net/can/bcm.c | 1774

Re: [PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-10-02 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: --- net-2.6.24.orig/include/linux/if_arp.h 2007-10-02 12:10:51.0 +0200 +++ net-2.6.24/include/linux/if_arp.h 2007-10-02 12:11:01.0 +0200 @@ -52,6 +52,7 @@ #define ARPHRD_ROSE270 #define ARPHRD_X25

[PATCH 6/7] CAN: Add maintainer entries

2007-09-25 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.24

[PATCH 0/7] CAN: Add new PF_CAN protocol family, try #8

2007-09-25 Thread Urs Thuermann
/svnroot/repos/socketcan. It can be found in the directory http://svn.berlios.de/svnroot/repos/socketcan/trunk/patch-series/version. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this list: send the line unsubscribe netdev in the body

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-25 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 3/7] CAN: Add raw protocol

2007-09-25 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 11 net/can/Makefile|3 net/can/raw.c | 822

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-09-25 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1

[PATCH 7/7] CAN: Add documentation

2007-09-25 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 634 ++ 2 files

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-25 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: + skb_queue_purge(sk-sk_receive_queue); + if (sk-sk_protinfo) + kfree(sk-sk_protinfo); +} Is it really needed to do this sk_protinfo check? Thanks for finding this. This is from 2.6.12 times or so. We have other CAN

Re: [PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-09-25 Thread Urs Thuermann
Arnaldo Carvalho de Melo [EMAIL PROTECTED] writes: + skb = alloc_skb(CFSIZ, + in_interrupt() ? GFP_ATOMIC : GFP_KERNEL); We have gfp_any() for this: Ah, ok. That looks better. Applied. urs - To unsubscribe from this list: send the line unsubscribe netdev in the body

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-25 Thread Urs Thuermann
YOSHIFUJI Hideaki / È£ÑÀ [EMAIL PROTECTED] writes: I'm not a lawyer, but the following lines: | + * Alternatively, provided that this notice is retained in full, this ~~ | + * software may be distributed under the terms of

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-25 Thread Urs Thuermann
Stephen Hemminger [EMAIL PROTECTED] writes: Then please make all exported symbols marked EXPORT_SYMBOL_GPL to make sure that the other CAN protocol can not reuse your infrastructure. We don't want to force other CAN protocol implementations to be GPL also. AFAIR from discussions on LKML, it

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-24 Thread Urs Thuermann
Joe Perches [EMAIL PROTECTED] writes: I'd prefer something like this, which removes the unnecessary kmalloc/kfree pairs or the equivalent conversions to functions. I have changed this to a static buffer. Since this is only in #ifdef CONFIG_CAN_DEBUG_CORE, it shouldn't hurt. #define

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-21 Thread Urs Thuermann
Joe Perches [EMAIL PROTECTED] writes: On Thu, 2007-09-20 at 20:43 +0200, Urs Thuermann wrote: +#define DBG(...) (debug 1 ? \ + (printk(KERN_DEBUG can-%s %s: , \ + IDENT, __func__), printk(args)) : 0) +#define DBG_FRAME

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-21 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: You drop the module reference again when leaving this function. So sock-ops might contain a stale pointer if the module is unloaded after this. You need to either keep the module reference while the socket is alive or remove stale references when

Re: [PATCH 3/7] CAN: Add raw protocol

2007-09-21 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: Urs Thuermann wrote: +config CAN_RAW_USER + bool Allow non-root users to access Raw CAN Protocol sockets If you plan to remove this option, it should happen before merging since it affects userspace visible behaviour. We have discussed

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-20 Thread Urs Thuermann
Hi Patrick, I have done allmost all changes to the code as you suggested. The changes to use the return value of can_rx_register() also fixed a minor flax with failing bind() and setsockopt() on raw sockets. But there are two things left I would like to ask/understand: Patrick McHardy [EMAIL

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-20 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: No, you need to add your own locking to prevent this, something list this: registration/unregistration: take lock change proto_tab[] release lock lookup: take lock cp = proto_tab[] if (cp !try_module_get(cp-owner)) cp = NULL

[PATCH 3/7] CAN: Add raw protocol

2007-09-20 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 26 + net/can/Makefile|3 net/can/raw.c | 828

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-09-20 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1

[PATCH 7/7] CAN: Add documentation

2007-09-20 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 635 ++ 2 files

[PATCH 0/7] CAN: Add new PF_CAN protocol family, try #7

2007-09-20 Thread Urs Thuermann
and is derived from Subversion revision r484 of http://svn.berlios.de/svnroot/repos/socketcan. It can be found in the directory http://svn.berlios.de/svnroot/repos/socketcan/trunk/patch-series/version. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp P.S. Greetings from some

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-20 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-09-20 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 28 net/can/Makefile|3 net/can/bcm.c | 1784

[PATCH 6/7] CAN: Add maintainer entries

2007-09-20 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.24

Re: [PATCH 7/7] CAN: Add documentation

2007-09-18 Thread Urs Thuermann
Bill Fink [EMAIL PROTECTED] writes: One more typo. decive - device above. Thank you. It's fixed now. urs - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: e1000 driver and samba

2007-09-18 Thread Urs Thuermann
Bill Fink [EMAIL PROTECTED] writes: It may also be a useful test to disable hardware TSO support via ethtool -K ethX tso off. All suggestions here on the list, i.e. checking for flow control, duplex, cable problems, etc. don't explain (at least to me) why LF sees file corruption. How can a

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-18 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: Looks pretty good, please see below for a few comments (mostly minor nitpicking, a few things that look like real bugs). Nothing that couldn't be fixed after merging though. Thank you for your review. I'll go through it and your other mail this

Re: [PATCH 2/7] CAN: Add PF_CAN core module

2007-09-18 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: +++ net-2.6.24/include/linux/can.h 2007-09-17 10:27:09.0 +0200 Is this file used only from within the kernel? If so you could use the nicer-to-look-at u8/u16/u32 types instead of the double underscored ones. No, this file contains the

Re: [PATCH 3/7] CAN: Add raw protocol

2007-09-18 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: +config CAN_RAW_USER + bool Allow non-root users to access Raw CAN Protocol sockets + depends on CAN_RAW + default N + ---help--- + The Controller Area Network is a local field bus transmitting only + broadcast messages

Re: [PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-18 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: +static int loopback; /* loopback testing. Default: 0 (Off) */ +module_param(loopback, int, S_IRUGO); +MODULE_PARM_DESC(loopback, Loop back frames (for testing). Default: 0 (Off)); I would still prefer to have this on a per-device level

[PATCH 6/7] CAN: Add maintainer entries

2007-09-17 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6.24

[PATCH 0/7] CAN: Add new PF_CAN protocol family, try #6

2007-09-17 Thread Urs Thuermann
for protocol, arp hw type, and a line discipline. Please review this patch series for integration into your tree. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

[PATCH 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-09-17 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1

[PATCH 5/7] CAN: Add virtual CAN netdevice driver

2007-09-17 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[PATCH 3/7] CAN: Add raw protocol

2007-09-17 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 26 + net/can/Makefile|3 net/can/raw.c | 767

[PATCH 7/7] CAN: Add documentation

2007-09-17 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 635 ++ 2 files

[PATCH 4/7] CAN: Add broadcast manager (bcm) protocol

2007-09-17 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 28 net/can/Makefile|3 net/can/bcm.c | 1762

Re: [PATCH 7/7] CAN: Add documentation

2007-09-17 Thread Urs Thuermann
Hi Randy, Thanks for all of this informative documentation. I have some typo/spello corrections below. Thank you very much. I have applied all your suggestions to our SVN repository. To reduce traffic on the list, I only repeat the pointer to the repository:

Re: [PATCH 7/7] CAN: Add documentation

2007-09-17 Thread Urs Thuermann
Thomas Gleixner [EMAIL PROTECTED] writes: Please do, having the patch in mail makes it easier to review and to comment. OK, here it is: This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL

Re: [PATCH 7/7] CAN: Add documentation

2007-09-17 Thread Urs Thuermann
Hi Randy, Just a few more minor changes... Thanks again. Again, thank you. I have applied these two typo fixes, too. urs - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

[patch 0/7] CAN: Add new PF_CAN protocol family, try #5

2007-08-03 Thread Urs Thuermann
much for your work! Best regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[patch 6/7] CAN: Add maintainer entries

2007-08-03 Thread Urs Thuermann
This patch adds entries in the CREDITS and MAINTAINERS file for CAN. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- CREDITS | 16 MAINTAINERS |9 + 2 files changed, 25 insertions(+) Index: net-2.6/CREDITS

[patch 5/7] CAN: Add virtual CAN netdevice driver

2007-08-03 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[patch 3/7] CAN: Add raw protocol

2007-08-03 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 26 + net/can/Makefile|3 net/can/raw.c | 757

[patch 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-08-03 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1

[patch 7/7] CAN: Add documentation

2007-08-03 Thread Urs Thuermann
This patch adds documentation for the PF_CAN protocol family. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- Documentation/networking/00-INDEX |2 Documentation/networking/can.txt | 635 ++ 2 files

[patch 4/7] CAN: Add broadcast manager (bcm) protocol

2007-08-03 Thread Urs Thuermann
This patch adds the CAN broadcast manager (bcm) protocol. Signed-off-by: Oliver Hartkopp [EMAIL PROTECTED] Signed-off-by: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/bcm.h | 65 + net/can/Kconfig | 28 net/can/Makefile|3 net/can/bcm.c | 1755

Re: [patch 5/7] CAN: Add virtual CAN netdevice driver

2007-07-09 Thread Urs Thuermann
) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers/net/can/Kconfig | 25 drivers/net/can

Re: [patch 5/7] CAN: Add virtual CAN netdevice driver

2007-07-04 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: It should create as many devices as necessary to operate (similar to the loopback device) by default. Optional interfaces that are used for addressing reasons should be manually added by the user as needed. And it should not use module parameters for

Re: [patch 5/7] CAN: Add virtual CAN netdevice driver

2007-06-22 Thread Urs Thuermann
Patrick McHardy [EMAIL PROTECTED] writes: Is there a reason why you're still doing the allocate n devices on init thing instead of using the rtnl_link API? Sorry, it's simply a matter of time. We have been extremely busy with other projects and two presentations (mgmt, customers, and press)

[patch 1/7] CAN: Allocate protocol numbers for PF_CAN

2007-06-21 Thread Urs Thuermann
This patch adds a protocol/address family number, ARP hardware type, ethernet packet type, and a line discipline number for the SocketCAN implementation. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- include/linux/if_arp.h |1

[patch 5/7] CAN: Add virtual CAN netdevice driver

2007-06-21 Thread Urs Thuermann
This patch adds the virtual CAN bus (vcan) network driver. The vcan device is just a loopback device for CAN frames, no real CAN hardware is involved. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- drivers/net/Makefile |1 drivers

[patch 3/7] CAN: Add raw protocol

2007-06-21 Thread Urs Thuermann
This patch adds the CAN raw protocol. Signed-Off-By: Oliver Hartkopp [EMAIL PROTECTED] Signed-Off-By: Urs Thuermann [EMAIL PROTECTED] --- include/linux/can/raw.h | 31 + net/can/Kconfig | 26 + net/can/Makefile|3 net/can/raw.c | 751

  1   2   >