Re: [PATCH ieee80211-wifi 0/3] hostap/ieee80211: Re-send split and re-diffed patches

2005-08-14 Thread Jeff Garzik
Jouni Malinen wrote: These patches are the not yet applied patches 8 and 9 from the previous set. Old patch 8 was split into two and rediffed (ipw2200.c had changed). The new patch 1 is for ieee80211 branch of netdev-2.6 and patch 2 for ieee80211-wifi. These two need to be applied at the same tim

[PATCH ieee80211-wifi 2/3] hostap: Capability field is called ESS, not BSS

2005-08-14 Thread Jouni Malinen
Remove backwards compatibility define for WLAN_CAPABILITY_ESS now that net/ieee80211.h defines this. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap_common.h === --- netdev-2.6.or

[PATCH ieee80211-wifi 3/3] hostap: Replace WLAN_FC_ defines with ieee80211 ones

2005-08-14 Thread Jouni Malinen
Replace remaining WLAN_FC_* defines with the ones used in ieee80211 header file. This completes the move from hostap version of frame control field processing to ieee80211 version. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap_common.h =

[PATCH ieee80211-wifi 1/3] ieee80211: Capability field is called ESS, not BSS

2005-08-14 Thread Jouni Malinen
IEEE 802.11 has a capability field flag called ESS, but ieee80211 had renamed this to BSS for some reason. hostap has been using WLAN_CAPABILITY_ESS and since that matches with the standard, lets use it as the name for this define. Add WLAN_CAPABILITY_BSS as a backwards compatibility name for the s

[PATCH ieee80211-wifi 0/3] hostap/ieee80211: Re-send split and re-diffed patches

2005-08-14 Thread Jouni Malinen
These patches are the not yet applied patches 8 and 9 from the previous set. Old patch 8 was split into two and rediffed (ipw2200.c had changed). The new patch 1 is for ieee80211 branch of netdev-2.6 and patch 2 for ieee80211-wifi. These two need to be applied at the same time. Patch 3 is otherwise

Re: [PATCH ieee80211-wifi 0/9] hostap/ieee80211: hostap update and small changes to ieee80211

2005-08-14 Thread Jeff Garzik
Jouni Malinen wrote: This set of patches merges a bug fix for the Host AP driver (patch 1), does some cleanup for different hardware models (patches 2-3), continues conversion to generic ieee80211 code (patches 4-6), and tests how we can handle changes that have a dependency to modify ieee80211 a

Re: [NETLINK 8/10]: Support dynamic number of multicast groups per netlink family

2005-08-14 Thread David S. Miller
From: Patrick McHardy <[EMAIL PROTECTED]> Date: Mon, 15 Aug 2005 02:33:30 +0200 > [NETLINK]: Support dynamic number of multicast groups per netlink family > > Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> I applied patches 1 -> 7, but I had to stop after that. This patch here will break ne

[PATCH ieee80211-wifi 6/9] hostap: Use ieee80211 WLAN_FC_GET_{TYPE,STYPE}

2005-08-14 Thread Jouni Malinen
Replace temporary HOSTAP_FC_GET_{TYPE,STYPE} macros with the ieee80211 version of WLAN_FC_GET_{TYPE,STYPE}. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap.c === --- netdev-2.6.or

[PATCH ieee80211-wifi 8/9] ieee80211: Capability field is called ESS, not BSS

2005-08-14 Thread Jouni Malinen
IEEE 802.11 has a capability field flag called ESS, but ieee80211 had renamed this to BSS for some reason. hostap has been using WLAN_CAPABILITY_ESS and since that matches with the standard, lets use it as the name for this define. Add WLAN_CAPABILITY_BSS as a backwards compatibility name for the s

[PATCH ieee80211-wifi 5/9] hostap: Replace hostap_ieee80211_hdr with ieee80211_hdr

2005-08-14 Thread Jouni Malinen
Replace hostap-specific struct hostap_ieee80211_hdr with struct ieee80211_hdr from net/ieee80211.h. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap_80211.h === --- netdev-2.6.orig

[PATCH ieee80211-wifi 4/9] hostap: Remove extra defines

2005-08-14 Thread Jouni Malinen
Remove unused defines that are already available from generic kernel header files. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap_wlan.h === --- netdev-2.6.orig/drivers/net/wirel

[PATCH ieee80211-wifi 1/9] hostap: Fix skb->cb use for TX meta data

2005-08-14 Thread Jouni Malinen
Old AP mode code for power saving was using skb->cb for couple of flags before the more generic 802.11 TX code started using the same buffer for struct hostap_skb_tx_data. This resulted in the power save code corrupting the magic value in beginning of the buffer and TX code dropping the power saved

[PATCH ieee80211-wifi 9/9] hostap: Replace WLAN_FC_ defines with ieee80211 ones

2005-08-14 Thread Jouni Malinen
Replace remaining WLAN_FC_* defines with the ones used in ieee80211 header file. This completes the move from hostap version of frame control field processing to ieee80211 version. Signed-off-by: Jouni Malinen <[EMAIL PROTECTED]> Index: netdev-2.6/drivers/net/wireless/hostap/hostap_common.h =

[PATCH ieee80211-wifi 2/9] hostap: Remove experimental PCI bus master/DMA code

2005-08-14 Thread Jouni Malinen
PCI version of Prism2.5/3 has undocumented DMA support for TX/RX data, but this seems to have some hardware bugs that prevent it from being used properly for TX. RX side could possibly be made to work reliably. Even though DMA support would be very useful for saving host CPU (from about 40% to 5-1

[PATCH ieee80211-wifi 0/9] hostap/ieee80211: hostap update and small changes to ieee80211

2005-08-14 Thread Jouni Malinen
This set of patches merges a bug fix for the Host AP driver (patch 1), does some cleanup for different hardware models (patches 2-3), continues conversion to generic ieee80211 code (patches 4-6), and tests how we can handle changes that have a dependency to modify ieee80211 and hostap at the same t

[PATCH ieee80211-wifi 7/9] ieee80211: Fix frame control pver mask

2005-08-14 Thread Jouni Malinen
IEEE 802.11 frame control has two bits reserved for protocol version. IEEE80211_FCTL_VERS was not used anywhere, but I would assume it was supposed to be a mask for the protocol field and as such, it should be 0x0003, not 0x0002. This matches with WLAN_FC_PVER definition in hostap. Signed-off-by:

[PATCH ieee80211-wifi 3/9] hostap: Use void *hw_priv instead of #ifdef in local data

2005-08-14 Thread Jouni Malinen
Replace hardware model specific #ifdef's in struct local_info with void *hw_priv that is pointing to cs/pci/plx specific data structure. This removes unneeded #ifdef's and as such, is a step towards making it possible to share objects for hostap_hw.c and hostap_download.c with cs/pci/plx drivers wi

Re: skb->pkt_type

2005-08-14 Thread David S. Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Mon, 15 Aug 2005 10:42:44 +1000 > Well I don't think I understand the new skb->user solution yet :) > > For a start, skb->users will prevent the skb->data area from being > freed as well as the skb itself. We'll also need to audit the TCP > code paths t

Re: skb->pkt_type

2005-08-14 Thread Herbert Xu
On Sun, Aug 14, 2005 at 05:16:58PM -0700, David S. Miller wrote: > > I think the atomic operation will still be cheaper than these > two function calls. Even though the second one will be "cheap" > we could be completely done with just one call and not even pay > the reduced price of the second a

[NETLINK 10/10]: Add "groups" argument to netlink_kernel_create

2005-08-14 Thread Patrick McHardy
[NETLINK]: Add "groups" argument to netlink_kernel_create Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 54c15156b46007c036c80f3df7e5202348b47ff6 tree fc1980ec428fc1d95dca654ab7add443c738f655 parent 8880eca333ef74bc4662f9b1a8b43b4e97135876 author Patrick McHardy <[EMAIL PROTECTED]

[NETLINK 9/10]: Add set/getsockopt options to support more than 32 groups

2005-08-14 Thread Patrick McHardy
[NETLINK]: Add set/getsockopt options to support more than 32 groups NETLINK_ADD_MEMBERSHIP/NETLINK_DROP_MEMBERSHIP are used to join/leave groups, NETLINK_PKTINFO is used to enable nl_pktinfo control messages for received packets to get the extended destination group number. Signed-off-by: Patri

[NETLINK 8/10]: Support dynamic number of multicast groups per netlink family

2005-08-14 Thread Patrick McHardy
[NETLINK]: Support dynamic number of multicast groups per netlink family Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 68bcf4640f376652194969a64317e13c35371bfc tree a131b8cd33df58e4679f69b0a50852d4b8ba3120 parent 159b54716bcf5aea4c3141a64072613db7934e11 author Patrick McHardy <[E

[NETLINK 7/10]: Return -EPROTONOSUPPORT in netlink_create() if no kernel socket is registered

2005-08-14 Thread Patrick McHardy
[NETLINK]: Return -EPROTONOSUPPORT in netlink_create() if no kernel socket is registered This is necessary for dynamic number of netlink groups to make sure we know the number of possible groups before bind() is called. With this change pure userspace communication using unused netlink protocols

[NETLINK 6/10]: Convert netlink users to use group numbers instead of bitmasks

2005-08-14 Thread Patrick McHardy
[NETLINK]: Convert netlink users to use group numbers instead of bitmasks Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit b96b2df4048e0b5fbfc9979d5a5fbcb200211b43 tree 16bd808aaa1a4d1abe7b66715dada313423abdcb parent ea41e53e0de17bc1079aab5f5342485532473f19 author Patrick McHardy <[

[NETLINK 5/10]: Use group numbers instead of bitmasks internally

2005-08-14 Thread Patrick McHardy
[NETLINK]: Use group numbers instead of bitmasks internally Using the group number allows increasing the number of groups without beeing limited by the size of the bitmask. It introduces one limitation for netlink users: messages can't be broadcasted to multiple groups anymore, however this featu

[NETLINK 4/10]: Fix module refcounting problems

2005-08-14 Thread Patrick McHardy
[NETLINK]: Fix module refcounting problems Use-after-free: the struct proto_ops containing the module pointer is freed when a socket with pid=0 is released, which besides for kernel sockets is true for all unbound sockets. Module refcount leak: when the kernel socket is closed before all user so

[NETLINK 3/10]: Remove unused groups member from struct netlink_skb_parms

2005-08-14 Thread Patrick McHardy
[NETLINK]: Remove unused groups member from struct netlink_skb_parms Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit cac8b0e31deb55b3839460d8d64a48b3d24ea3d9 tree 706a5b6ecec93a86713e9baf7f36043f363686ab parent 653290b7c92c670ededb396042792b17390942ae author Patrick McHardy <[EMAIL

[NETLINK 2/10]: Fix missing dst_groups initializations in netlink_broadcast users

2005-08-14 Thread Patrick McHardy
[NETLINK]: Fix missing dst_groups initializations in netlink_broadcast users netlink_broadcast users must initialize NETLINK_CB(skb).dst_groups to the destination group mask for netlink_recvmsg. Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 653290b7c92c670ededb396042792b17390942

[NETLINK 1/10]: w1_int.c: fix default netlink group

2005-08-14 Thread Patrick McHardy
[NETLINK]: w1_int.c: fix default netlink group w1 does not need to multicast its state to several groups at once, and upcoming netlink changes will not allow bitmask for groups anyway. Signed-off-by: Evgeniy Polyakov <[EMAIL PROTECTED]> Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- com

[NETLINK 0/10]: Netlink fixes/dynamic groups v2

2005-08-14 Thread Patrick McHardy
I discovered a couple of problems in the last patchset, so here is an updated one. Changes since last posting: - resolve conflicts with socket monitoring changes - include selinux in changes, has been missed before - fix failed BUG_TRAP caused by one of the patches - include w1 bugfix to dst_grou

Re: skb->stamp conversion missing from latest net-2.6.14

2005-08-14 Thread David S. Miller
From: Patrick McHardy <[EMAIL PROTECTED]> Date: Sun, 14 Aug 2005 23:23:32 +0200 > [NET]: Store skb->timestamp as offset to a base timestamp > > Reduces skb size by 8 bytes. > > Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> Updated patch applied, thanks Patrick. - To unsubscribe from this l

Re: skb->pkt_type

2005-08-14 Thread David S. Miller
From: Herbert Xu <[EMAIL PROTECTED]> Date: Mon, 15 Aug 2005 09:52:51 +1000 > 1) When you allocate for fast clones you do two kmem_cache_alloc's > in a row. You store the second pointer in the shared skb data area. > > The fast path within kmem_cache_alloc is good enough that the second > one sho

Re: skb->pkt_type

2005-08-14 Thread David S. Miller
From: Thomas Graf <[EMAIL PROTECTED]> Date: Sun, 14 Aug 2005 02:31:30 +0200 > The disadvantage is that we cannot do "clone resycling" easly > anymore, that is we allow a fast-clone'able skb to be cloned > up again if the clone portion was already freed. I do not see much value in that capability

Re: interface shutdown does not work (e1000)

2005-08-14 Thread Bernd Eckenfels
On Sun, Aug 14, 2005 at 05:02:37PM -0700, David S. Miller wrote: > Bringing an interface down does not disassosciate the IP addresses > confiured to that interface from the system. Yes, but I am not sure if that is the case here, we need more input from the reporter. It looked more like the down i

Re: interface shutdown does not work (e1000)

2005-08-14 Thread David S. Miller
From: Bernd Eckenfels <[EMAIL PROTECTED]> Date: Sun, 14 Aug 2005 19:51:39 +0200 > I have this error report on ifconfig (also a problem with ip link) that a > down interface receives packets. I remeber this was discussed before but > cannot find a result of the outcome. Is this a driver problem? B

Re: [DCCP]: Fix compiler warnings

2005-08-14 Thread Arnaldo Carvalho de Melo
Em Sun, Aug 14, 2005 at 07:11:46PM +0200, Patrick McHardy escreveu: > gcc-4.0.1 (new debian default) emits a couple of warnings in > the dccp code, this patch silences them. > > [DCCP]: Fix compiler warnings Thanks Patrick, I'm applying this to the dccp-2.6.14.git tree. - Arnaldo - To unsubscri

Re: skb->pkt_type

2005-08-14 Thread Herbert Xu
On Sat, Aug 13, 2005 at 02:10:55PM -0700, David S. Miller wrote: > Yes, you need an atomic in the fast path, but it's only done > for fast clones and their parent SKBs, and such an atomic is > _MUCH_ cheaper than a kmem_cache_alloc especially since it > touches more cache local state (the atomic co

[NETLINK 10/*]: w1_int.c: fix default netlink group

2005-08-14 Thread Patrick McHardy
Evgeniy's patch fixed to apply on top of my netlink changes. [NETLINK]: w1_int.c: fix default netlink group w1 does not need to multicast its state to several groups at once, and upcoming netlink changes will not allow bitmask for groups anyway. Signed-off-by: Evgeniy Polyakov <[EMAIL PROTECTED]

[NETLINK 9/*]: Fixed failed BUG_TRAP

2005-08-14 Thread Patrick McHardy
Fix minor fallout from patch 6/8. [NETLINK]: Fixed failed BUG_TRAP I forgot to set groups to NULL after freeing it: KERNEL: assertion (!nlk_sk(sk)->groups) failed at net/netlink/af_netlink.c (146) Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 1018286a3c3542c9926e287811a24bf85e1

Re: [RFC NETLINK 0/8]: Support dynamic number of groups

2005-08-14 Thread Patrick McHardy
Evgeniy Polyakov wrote: > You've done a good job, thanks Patrick. > > One small comment after diagonal patchset review - > last patch will fail to apply on top of w1 changes :) > since it already has a group changes to 1. The w1 changes don't seem to be in mainline yet. I'm going to fix up your

Re: skb->stamp conversion missing from latest net-2.6.14

2005-08-14 Thread Patrick McHardy
Ben Greear wrote: > Patrick McHardy wrote: >> >> +static inline void skb_get_timestamp(struct sk_buff *skb, struct >> timeval *stamp) >> +{ >> +stamp->tv_sec = skb->tstamp.off_sec; >> +stamp->tv_usec = skb->tstamp.off_usec; >> +if (skb->tstamp.off_sec) { >> +stamp->tv_sec +=

Re: [DCCP]: Fix compiler warnings

2005-08-14 Thread Patrick McHardy
Mitchell Blank Jr wrote: > Patrick McHardy wrote: > >>-static inline const int before48(const u64 seq1, const u64 seq2) >>+static inline int before48(const u64 seq1, const u64 seq2) > > Perhaps "__attribute__ ((const))" is what was meant here? Doesn't matter for inline functions, the compiler k

Re: [DCCP]: Fix compiler warnings

2005-08-14 Thread Mitchell Blank Jr
Patrick McHardy wrote: > -static inline const int before48(const u64 seq1, const u64 seq2) > +static inline int before48(const u64 seq1, const u64 seq2) Perhaps "__attribute__ ((const))" is what was meant here? -Mitch - To unsubscribe from this list: send the line "unsubscribe netdev" in the bod

[PATCH ppp-2.4.3] add SC_MUST_COMP flag

2005-08-14 Thread Matt Domsch
(rediff with license on include/linux/if_ppp.h untouched, and I had named the new flag SC_MUST_COMP not SC_MUST_COMPRESS) PPP 2.4.3 patch creates the new flag SC_MUST_COMP. This is set on CCP UP with MPPE enabled, and never cleared. The kernel uses this flag to drop packets which aren't compress

Re: [PATCH ppp-2.4.3] add SC_MUST_COMPRESS flag

2005-08-14 Thread Matt Domsch
On Sun, Aug 14, 2005 at 04:31:47PM +0200, Patrick McHardy wrote: > Patrick McHardy wrote: > > Matt Domsch wrote: > > > >> include/linux/if_ppp.h | 62 > >> + > > > > > > - * 2. Redistributions in binary form must reproduce the above copyright > >

interface shutdown does not work (e1000)

2005-08-14 Thread Bernd Eckenfels
Hello, I have this error report on ifconfig (also a problem with ip link) that a down interface receives packets. I remeber this was discussed before but cannot find a result of the outcome. Is this a driver problem? http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=322654 Bernd -- (OO) --

Re: skb->stamp conversion missing from latest net-2.6.14

2005-08-14 Thread Ben Greear
Patrick McHardy wrote: David S. Miller wrote: I just double-checked, and I didn't lose it during the recent rebasing, as the tree I started with didn't have the skb->stamp patch either. Are you sure I did apply it at some point? Yes, I know it was in your tree before the first rebasing. E

[DCCP]: Fix compiler warnings

2005-08-14 Thread Patrick McHardy
gcc-4.0.1 (new debian default) emits a couple of warnings in the dccp code, this patch silences them. [DCCP]: Fix compiler warnings may be a false warning if there always is something on ccid3hcrx_hist: net/dccp/ccids/ccid3.c: In function 'ccid3_hc_rx_packet_recv': net/dccp/ccids/ccid3.c:1634:

Re: [RFC NETLINK 0/8]: Support dynamic number of groups

2005-08-14 Thread Evgeniy Polyakov
On Sat, Aug 13, 2005 at 02:35:53AM +0200, Patrick McHardy ([EMAIL PROTECTED]) wrote: > Hi, > > besides a small bugfix, this patchset adds support for dynamic number > of groups to netlink. To support an arbitary number of groups a couple > of changes had to me made, I'll explain them below. The p

Re: skb->stamp conversion missing from latest net-2.6.14

2005-08-14 Thread Patrick McHardy
David S. Miller wrote: I just double-checked, and I didn't lose it during the recent rebasing, as the tree I started with didn't have the skb->stamp patch either. Are you sure I did apply it at some point? Yes, I know it was in your tree before the first rebasing. Either way, please resend i

Re: [PATCH 1/2] LSM-IPSec Networking Hooks -- mods based on Herbert's comments

2005-08-14 Thread Trent Jaeger
Thanks for the comments. I'll be mostly on vacation (moving) this week, so I will be getting back to you after Aug 22. Regards, Trent. Trent Jaeger IBM T.J. Watson Research Center 19 Skyline Drive, Hawthorne, NY 10532 (914) 784-7225, F

Re: [PATCH ppp-2.4.3] add SC_MUST_COMPRESS flag

2005-08-14 Thread Patrick McHardy
Patrick McHardy wrote: > Matt Domsch wrote: > >> include/linux/if_ppp.h | 62 >> + > > > - * 2. Redistributions in binary form must reproduce the above copyright > - *notice, this list of conditions and the following disclaimer in > - *th

Re: [PATCH ppp-2.4.3] add SC_MUST_COMPRESS flag

2005-08-14 Thread Patrick McHardy
Matt Domsch wrote: > include/linux/if_ppp.h | 62 > + - * 2. Redistributions in binary form must reproduce the above copyright - *notice, this list of conditions and the following disclaimer in - *the documentation and/or other materials p

[PATCH ppp-2.4.3] add SC_MUST_COMPRESS flag

2005-08-14 Thread Matt Domsch
PPP 2.4.3 patch creates the new flag SC_MUST_COMPRESS. This is set on CCP UP with MPPE enabled, and never cleared. The kernel uses this flag to drop packets which aren't compressed/decompressed properly when MPPE is enabled. In the existing code, on SIGUSR2, CCP is renegotiated, and with MPPE en

[PATCH 2.6.13-rc] ppp_mppe: add PPP MPPE encryption module

2005-08-14 Thread Matt Domsch
This patch implements the Microsoft Point-to-Point Tunnelling Protocol kernel module, with minor changes to ppp_generic to have it drop packets if they are not compressed/decompressed properly. This adds a new ppp flag SC_MUST_COMPRESS which will be set by pppd upon CCP UP with MPPE enabled. dr

Re: [Fwd: [patch 02/15] ppp_mppe: add PPP MPPE encryption module]

2005-08-14 Thread Matt Domsch
On Fri, Jul 08, 2005 at 09:42:21PM +1000, Paul Mackerras wrote: > Some comments on the MPPE kernel patch (sorry it's taken me so long): Likewise, my apologies for the delay in implementing your suggestions. I do really appreciate your feedback. > > +static inline struct sk_buff * > > +pad_comp

Re: [NETLINK 6/8]: Support dynamic number of multicast groups per netlink family

2005-08-14 Thread Patrick McHardy
Thomas Graf wrote: > * Patrick McHardy <[EMAIL PROTECTED]> 2005-08-13 02:36 > >>[NETLINK]: Support dynamic number of multicast groups per netlink family >> >>Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> >> >>- if ((err = __netlink_create(sock, protocol) < 0)) >>+ nlk->groups = kmallo