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 t

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_

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

2008-02-10 Thread Urs Thuermann
all, 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? Signe

[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/sk

[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,

[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_

[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

[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 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

[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

[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 &

[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/Makefil

[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

[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/. Urs Thuermann Oliver Hartkopp

[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 +

[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

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

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 *cfra

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 5/7] CAN: Add virtual CAN netdevice driver

2007-11-14 Thread Urs Thuermann
onst char banner[] = + KERN_INFO "vcan: Virtual CAN interface driver\n"; + +MODULE_DESCRIPTION("virtual CAN interface"); +MODULE_LICENSE("Dual BSD/GPL"); +MODULE_AUTHOR("Urs Thuermann <[EMAIL PROTECTED]>"); + +#ifdef CONFIG_CAN_D

[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

[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 +

[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

[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/Makefil

[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

[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/. 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 that the

[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

[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 @@ -

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

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

[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/Makefil

[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 +

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

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

[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

[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

[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

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 i

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 { ca

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? Namespac

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 > > #def

[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

[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 +

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

2007-10-02 Thread Urs Thuermann
etcan/trunk/patch-series/. 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://vger.kernel.org/majordomo-info.html

[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

[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/Makefil

[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

[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

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, i

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

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

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 ha

[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 +

[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_ar

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

2007-09-25 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

[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/Makefil

[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

[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

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

2007-09-25 Thread Urs Thuermann
and is derived from Subversion revision r493 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/. Thanks very much for your work! Best regards, Urs Thuermann Oliver Hartkopp -- - To unsubscribe from this

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 can_

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 af

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 whe

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, _

[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

[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

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

2007-09-20 Thread Urs Thuermann
This patch series applies against net-2.6.24 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/. Thanks very much for your work! Best regards, Urs Thuermann

[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/Makefil

[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 +

[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

[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_ar

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)) >

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 P

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-dev

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 > > + broadca

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 th

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 e

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

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.ht

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 http://vger.kernel.org/majordomo

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

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: http://svn.berlios.de/svnroo

[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 +

[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

[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

[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_ar

[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/Makefil

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

2007-09-17 Thread Urs Thuermann
t for the allocation of a couple of numbers 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 "unsubs

[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

[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 +

[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

[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

[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_ar

[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(+) In

[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/Makefil

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

2007-08-03 Thread Urs Thuermann
or 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 PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

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

2007-07-09 Thread Urs Thuermann
? Regards, 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 <[EMAIL PROTECTED]> --- drivers

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

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 pre

[patch 7/7] CAN: Add documentation

2007-06-21 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 +

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

2007-06-21 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

  1   2   >