drivers/net/ethernet/hisilicon/hns/hnae.c:442:1-3: WARNING: PTR_ERR_OR_ZERO can
be used
Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR
Generated by: scripts/coccinelle/api/ptr_ret.cocci
CC: Arnd Bergmann
Signed-off-by: Fengguang Wu
---
hnae.c |5 +
1 file changed, 1 ins
tree: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
master
head: 7b4b000951f09cfe3604a6fdf5469894b4e75adb
commit: 876133d3161dcb743983d1f9e5bf247aea2fb0f1 [241/245] net: hisilicon: add
OF dependency
coccinelle warnings: (new ones prefixed by >>)
>> drivers/net/ethernet
On Mon, 2015-10-19 at 21:16 +0200, Bjørn Mork wrote:
> Oliver Neukum writes:
>
> > usbnet drives no devices of its own. It makes more sense to
> > select it whenever a driver for actual hardware that needs
> > it is chosen rather than offer it as an option of its own.
> >
> > Signed-off-by: Olive
On 17 October 2015 at 13:14, Florian Westphal wrote:
> @@ -606,19 +599,22 @@ struct sk_buff *nf_ct_frag6_gather(struct net *net,
> struct sk_buff *skb, u32 use
> spin_unlock_bh(&fq->q.lock);
> pr_debug("Can't insert skb to queue\n");
> inet_frag_put
On 17 October 2015 at 13:14, Florian Westphal wrote:
> [ CC netdev since patch #2 isn't nf-specific. Dave, if you want
> I can resubmit that one after the next nf-pull request; let me know if
> you would prefer that ].
>
> Openvswitch seems broken wrt. to defragmentation, it doesn't call
> nf
Hi Julia,
On Sat, Oct 17, 2015 at 11:32:19AM +0200, Julia Lawall wrote:
> The exit label performs device_unlock(&dev->dev);, which will fail when dev
> is NULL, and nfc_put_device(dev);, which is not useful when dev is NULL, so
> just exit the function immediately.
>
> Problem found using scripts
Hi Vincent,
On Wed, Oct 07, 2015 at 11:33:19AM +0200, Vincent Stehlé wrote:
> The function nfc_genl_llc_sdreq() can dereference the dev pointer while
> it is NULL on its error path. Create a new error handling label to avoid
> that.
>
> This fixes the following coccinelle error:
>
> ./net/nfc/
On Mon, Oct 19, 2015 at 8:40 PM, Eric Dumazet wrote:
> From: Eric Dumazet
>
> Tom Herbert added SIT support to GRO with commit
> 19424e052fb4 ("sit: Add gro callbacks to sit_offload"),
> later reverted by Herbert Xu.
>
> The problem came because Tom patch was building GRO
> packets without proper
From: Eric Dumazet
Tom Herbert added SIT support to GRO with commit
19424e052fb4 ("sit: Add gro callbacks to sit_offload"),
later reverted by Herbert Xu.
The problem came because Tom patch was building GRO
packets without proper meta data : If packets were locally
delivered, we would not care.
From: Eric Dumazet
While testing my SIT/GRO patch using netfilter TEE module and a dummy
device, I found some features were missing :
TSO IPv6, UFO, and encapsulated traffic.
ethtool -k dummy0 now gives :
...
tcp-segmentation-offload: on
tx-tcp-segmentation: on
tx-tcp-ecn-segmen
于 2015/10/20 10:14, Alexei Starovoitov 写道:
> On 10/19/15 3:37 AM, Kaixu Xia wrote:
>> +/* flags for PERF_EVENT_ARRAY maps*/
>> +enum {
>> +BPF_EVENT_CTL_BIT_CUR = 0,
>> +BPF_EVENT_CTL_BIT_ALL = 1,
>> +__NR_BPF_EVENT_CTL_BITS,
>> +};
>> +
>> +#defineBPF_CTL_BIT_FLAG_MASK \
>> +((
The presence of this attribute does not modify the ct_state for the
current packet, only future packets. Make this more clear in the header
definition.
Signed-off-by: Joe Stringer
---
v3: First post.
---
include/uapi/linux/openvswitch.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
di
Currently, 0-bits are generated in ct_state where the bit position is
undefined, and matches are accepted on these bit-positions. If userspace
requests to match the 0-value for this bit then it may expect only a
subset of traffic to match this value, whereas currently all packets
will have this bit
If userspace provides a ct action with no nested mark or label, then the
storage for these fields is zeroed. Later when actions are requested,
such zeroed fields are serialized even though userspace didn't
originally specify them. Fix the behaviour by ensuring that no action is
serialized in this c
New, related connections are marked as such as part of ovs_ct_lookup(),
but they are not marked as "new" if the commit flag is used. Make this
consistent by setting the "new" flag whenever !nf_ct_is_confirmed(ct).
Reported-by: Jarno Rajahalme
Signed-off-by: Joe Stringer
---
v2: Acked.
v3: Took a
On 10/19/15 3:37 AM, Kaixu Xia wrote:
+/* flags for PERF_EVENT_ARRAY maps*/
+enum {
+ BPF_EVENT_CTL_BIT_CUR = 0,
+ BPF_EVENT_CTL_BIT_ALL = 1,
+ __NR_BPF_EVENT_CTL_BITS,
+};
+
+#defineBPF_CTL_BIT_FLAG_MASK \
+ ((1ULL << __NR_BPF_EVENT_CTL_BITS) - 1)
+#define
On Fri, 2015-10-16 at 08:23 -0700, Eric Dumazet wrote:
> What about the following more complete patch ?
>
> We properly set skb->encapsulation and inner network header as some NIC
> drivers depend on it. Our GSO should also work properly I think.
>
> Wolfgang, could you please test it ? (this is
Hi,
We recently found a bug in our code which resulted in rule 0 in the
routing policy database being deleted, this is a bad thing and the man
page documentation for "ip rule" indicates
Rule 0 is special. It cannot be deleted or overridden.
which seems entirely reasonable. The code in net/core
On Tue, 2015-10-20 at 02:12 +0100, Alan Burlison wrote:
> Another problem is that if I call close() on a Linux socket that's in
> accept() the accept call just sits there until there's an incoming
> connection, which succeeds even though the socket is supposed to be
> closed, but then an immedi
On 20/10/2015 00:33, Eric Dumazet wrote:
It looks it is a long standing problem, right ?
Yep, seems so.
inet_shutdown() has this very specific comment from beginning of git
tree :
switch (sk->sk_state) {
...
/* Remaining two branches are temporary solution for missing
On 10/19/15 4:02 PM, Hannes Frederic Sowa wrote:
I bet commercial software will make use of this ebpf framework, too. And
the kernel always helped me and gave me a way to see what is going on,
debug which part of my operating system universe interacts with which
other part. Merely dropping file d
From: Eric Dumazet
iptables -I INPUT ... -j TEE --gateway 10.1.2.3
because --oif was not specified
tee_tg_check() sets ->priv pointer to NULL in this case.
Fixes: bbde9fc1824a ("netfilter: factor out packet duplication for IPv4/IPv6")
Signed-off-by: Eric Dumazet
---
net/netfilter/xt_TEE.c |
On Mon, Oct 19, 2015 at 5:18 PM, Christopher Hall
wrote:
> On Thu, 15 Oct 2015 01:15:57 -0700, Thomas Gleixner
> wrote:
>>>
>>> >
>>> > > +#define SHADOW_HISTORY_DEPTH 7
>>> >
>>> > And that number is 7 because?
>>>
>>> Due to power of 2 it will be 8 instead. As above the useful history is
>>> 8-
On 10/19/15 3:17 PM, Daniel Borkmann wrote:
On 10/19/2015 10:48 PM, Alexei Starovoitov wrote:
On 10/19/15 1:03 PM, Hannes Frederic Sowa wrote:
I doubt it will stay a lightweight feature as it should not be in the
responsibility of user space to provide those debug facilities.
It feels we're
On 10/19/15 at 04:13pm, Joe Stringer wrote:
> I think that the proper solution instead of this patch is to set NEW
> if !nf_ct_is_confirmed(ct). This is more accurately the meaning for
> 'NEW' that we are actually trying to expose. As long as this is done
> before confirming the connection during a
Thomas,
On Thu, 15 Oct 2015 01:15:57 -0700, Thomas Gleixner
wrote:
>
> > +#define SHADOW_HISTORY_DEPTH 7
>
> And that number is 7 because?
Due to power of 2 it will be 8 instead. As above the useful history is
8-2*1
ms (1 ms is the minimum jiffy length). Array size 4 would not be enough
On Mon, 2015-10-19 at 09:59 -0700, Stephen Hemminger wrote:
> This looks like corner case, but worth forwarding.
>
> Begin forwarded message:
>
> Date: Mon, 19 Oct 2015 13:21:33 +
> From: "bugzilla-dae...@bugzilla.kernel.org"
>
> To: "shemmin...@linux-foundation.org"
> Subject: [Bug 106241
On 19 October 2015 at 02:03, Thomas Graf wrote:
> On 10/19/15 at 12:07am, Joe Stringer wrote:
>> > I'm probably missing something obvious. Why is the reply direction
>> > not considered NEW? Wouldn't this consider an ICMPv6 as related+new
>> > depending on simply the direction?
>>
>> My thoughts w
On Sun, 2015-10-18 at 22:19 -0700, Stephen Hemminger wrote:
> On Sat, 17 Oct 2015 00:21:44 -0400
> Michael Chan wrote:
>
> > +static bool bnxt_vf_pciid(int idx)
> > +{
> > + if (idx == BCM57304_VF || idx == BCM57404_VF)
> > + return true;
> > + return false;
> > +}
> > +
>
> I pre
From: Kiran Patil
This patch introduces a spinlock which is to be used for synchronizing
access to VSI's MAC filter list.
This patch also synchronizes execution of other codepaths which are
accessing VSI's MAC filter list with execution of
service_task:sync_vsi_filters.
In function i40e_add_vsi
From: Jesse Brandeburg
The dynamic algorithm, while now working, doesn't have good
performance in 40G mode.
One part of this patch addresses the high CPU utilization of some small
streaming workloads that the driver should reduce CPU in.
It also changes the minimum ITR that the dynamic algorith
From: Mitch Williams
When the VF driver is unable to communicate with the PF, it just gives
up and never tries again. Aside from the obvious character flaw that
this shows, it's also a lousy user experience.
When PF communications fail, wait five seconds, and try again. And
again. Don't give up,
From: Anjali Singhai
Acquire NVM, before issuing an AQ read nvm command for X722.
We need to acquire the NVM before issuing an AQ read to the NVM
otherwise we will get EBUSY from the FW. Also release when done.
This fixes the two X722 issues with respect to eeprom checksum verify
and reading NVM
From: Jesse Brandeburg
This change moves a multi-line register setting into a function
which simplifies reading the flow of the enable function.
This also fixes a bug where the enable function was enabling
the interrupt twice while trying to update the two interrupt
throttle rate thresholds for
From: Jesse Brandeburg
The driver was using a value expressed in 2us increments
for the divisor to figure out our bytes/usec values.
Fix the usecs variable to contain a value in microseconds.
Change-ID: I5c20493103c295d6f201947bb908add7040b7c41
Signed-off-by: Jesse Brandeburg
Tested-by: Andrew
From: Jesse Brandeburg
The adaptive ITR (interrupt throttle rate) algorithm was adjusting
the hardware's interrupt rate too frequently. This caused a lot
of variation in the interrupt rate for fairly constant workloads.
Change the code to have a counter and adjust only once every N
number of in
From: Alexander Duyck
The i40e_intr and i40e/i40evf_msix_clean_rings functions run from hard
interrupt context or with interrupts already disabled in netpoll.
They can use napi_schedule_irqoff() instead of napi_schedule()
Signed-off-by: Alexander Duyck
Tested-by: Andrew Bowers
Signed-off-by:
From: Carolyn Wyborny
This patch fixes a problem where our Tools MAC Loopback test, after
driver unbind would fail. This was because the hw was configured
for multiqueue and unbind operation did not clear this configuration.
The problem is fixed by resetting this configuration in i40e_remove.
C
From: Jean Sacren
The second argument name in the kernel-doc argument list for
i40e_features_check() was slightly off. Fix it for the kernel doc.
Signed-off-by: Jean Sacren
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
1 file cha
From: Jesse Brandeburg
The defines from the RSS enabling call were mistakenly
missed in the patches to the i40e which should have been
to i40evf as well.
This is a follow up to (commit ed921559886dd40528) "fix
32 bit build warnings".
Signed-off-by: Jesse Brandeburg
Tested-by: Andrew Bowers
Si
From: Anjali Singhai Jain
A mask value of 0x1FF was accidentally replaced with a bit mask
causing flow director sideband to be broken.
Change-ID: Id3387f67dd1b567b41692b570b383c58671e1eae
Signed-off-by: Anjali Singhai Jain
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/
From: Jean Sacren
'err' would be overwritten immediately, so we should declare it only
rather than initialize it to zero.
Signed-off-by: Jean Sacren
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
1 file changed, 1 insertion(+), 1
From: Carolyn Wyborny
This patch fixes a problem where the NVMUpdate Tool, when using the PHY
NVM feature, gets bad data from the PHY because of contention on the
MDIO interface from get PHY capability calls from the driver during
regular operations. The problem is fixed by adding a check if med
From: Catherine Sullivan
There is an error coming back from get_phy_capabilities that does not
seem to have any functional implications. We will continue looking into
why this error message is occurring, but in the meantime, we will move it
to debug to avoid confusion.
Change-ID: I9091754bf62c06
From: Catherine Sullivan
Bump.
Change-ID: Id0a7ecaa491f88ce94c9eba4901e592a56044ee0
Signed-off-by: Catherine Sullivan
Tested-by: Andrew Bowers
Signed-off-by: Jeff Kirsher
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
drivers/net/ethernet/intel/i40evf/i40evf_main.c | 2 +-
2 fil
From: Catherine Sullivan
We were previously checking if autoneg was allowed to change before
checking if autoneg was changing. We need to do this in the other order
or else we will erroneously return EINVAL when autoneg is not changing.
Change-ID: Iff9f7d1c9bddc1ad1e5d227d4f42754f90155410
Signed
This series contains updates to i40e and i40evf only.
Kiran adds a spinlock around code accessing VSI MAC filter list to
ensure that we are synchronizing access to the filter list, otherwise
we can end up with multiple accesses at the same time which can cause
the VSI MAC filter list to get in an
On 10/19/2015 10:48 PM, Alexei Starovoitov wrote:
On 10/19/15 1:03 PM, Hannes Frederic Sowa wrote:
I doubt it will stay a lightweight feature as it should not be in the
responsibility of user space to provide those debug facilities.
It feels we're talking past each other.
I want to solve 'per
On Mon, Oct 19, 2015 at 1:33 PM, Florian Westphal wrote:
> Ani Sinha wrote:
>> On Sun, Oct 18, 2015 at 2:40 PM, Florian Westphal wrote:
>> > Ani Sinha wrote:
>> >> Indeed. So it seems to me that we have run into one another such case.
>> >> In patch c6825c0976fa7893692, I see we have added an a
On Sun, 2015-10-18 at 22:21 -0700, Stephen Hemminger wrote:
> On Sat, 17 Oct 2015 00:21:44 -0400
> Michael Chan wrote:
>
> > +static inline u32 bnxt_tx_avail(struct bnxt *bp, struct bnxt_tx_ring_info
> > *txr)
> > +{
> > + /* Tell compiler to fetch tx indices from memory. */
> > + barrier()
A two-part patchset that fixes some "unaligned access" warnings
that showed up my sparc test machines with ipsec set up.
Sowmini Varadhan (2):
crypto/x509: Fix unaligned access in x509_get_sig_params()
Fix unaligned access in xfrm_notify_sa() for DELSA
crypto/asymmetric_keys/x509_public_key
On sparc, deleting established SAs (e.g., by restarting ipsec
at the peer) results in unaligned access messages via
xfrm_del_sa -> km_state_notify -> xfrm_send_state_notify().
Use an aligned pointer to xfrm_usersa_info for this case.
Signed-off-by: Sowmini Varadhan
---
net/xfrm/xfrm_user.c |
x509_get_sig_params() has the same code pattern as the one in
pkcs7_verify() that is fixed by commit 62f57d05e287 ("crypto: pkcs7 - Fix
unaligned access in pkcs7_verify()") so apply a similar fix here: make
sure that desc is pointing at an algined value past the digest_size,
and take alignment valu
On Mon, 2015-10-19 at 18:51 -0200, Renato Westphal wrote:
> Commit e520af48c7e5a introduced the following bug when setting the
> TCP_REPAIR sockoption:
>
>
> Since tcp_xmit_probe_skb() can be called from process context, use
> NET_INC_STATS() instead of NET_INC_STATS_BH().
>
> Signed-off-by: Re
On Sun, Sep 13, 2015 at 4:52 AM, Pablo Neira Ayuso wrote:
> On Sat, Sep 12, 2015 at 06:01:27PM -0700, Akemi Yagi wrote:
>> Thanks to all for this effort. The patch is finally in the mainline
>> kernel 4.3-rc1 released today. Now I wish backporting to the 4.2
>> branch goes smoothly.
>
> It's on it
Commit e520af48c7e5a introduced the following bug when setting the
TCP_REPAIR sockoption:
[ 2860.657036] BUG: using __this_cpu_add() in preemptible [] code:
daemon/12164
[ 2860.657045] caller is __this_cpu_preempt_check+0x13/0x20
[ 2860.657049] CPU: 1 PID: 12164 Comm: daemon Not tainted 4
On 10/19/15 1:03 PM, Hannes Frederic Sowa wrote:
I doubt it will stay a lightweight feature as it should not be in the
responsibility of user space to provide those debug facilities.
It feels we're talking past each other.
I want to solve 'persistent map' problem.
debugging of maps/progs, hier
On Fri, Oct 16, 2015 at 5:19 AM, Jamal Hadi Salim wrote:
>
> Like i said earlier, the idea of keeping track of backlog is useful.
> I think the commit log threw me off.
> "It is odd to see qlen!=0 but backlog==0"
> If you change that commit log then:
Sure, I think DaveM probably already discards
Ani Sinha wrote:
> On Sun, Oct 18, 2015 at 2:40 PM, Florian Westphal wrote:
> > Ani Sinha wrote:
> >> Indeed. So it seems to me that we have run into one another such case.
> >> In patch c6825c0976fa7893692, I see we have added an additional check
> >> (along with comparing tuple and zone) to v
On Sun, Oct 18, 2015 at 2:40 PM, Florian Westphal wrote:
> Ani Sinha wrote:
>> Indeed. So it seems to me that we have run into one another such case.
>> In patch c6825c0976fa7893692, I see we have added an additional check (along
>> with comparing tuple and zone) to verify that if the conntrack
tree: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
master
head: 7b4b000951f09cfe3604a6fdf5469894b4e75adb
commit: 876133d3161dcb743983d1f9e5bf247aea2fb0f1 [241/245] net: hisilicon: add
OF dependency
config: blackfin-allmodconfig (attached as .config)
reproduce:
wg
Hi Alexei,
On Mon, Oct 19, 2015, at 21:34, Alexei Starovoitov wrote:
> On 10/19/15 11:46 AM, Hannes Frederic Sowa wrote:
> > Hi,
> >
> > On Mon, Oct 19, 2015, at 20:15, Alexei Starovoitov wrote:
> >> On 10/19/15 10:37 AM, Daniel Borkmann wrote:
> >>> An eBPF program or map loading/destruction is *
On 10/19/15 11:46 AM, Hannes Frederic Sowa wrote:
Hi,
On Mon, Oct 19, 2015, at 20:15, Alexei Starovoitov wrote:
On 10/19/15 10:37 AM, Daniel Borkmann wrote:
An eBPF program or map loading/destruction is *not* by any means to be
considered fast-path. We currently hold a global mutex during load
Oliver Neukum writes:
> usbnet drives no devices of its own. It makes more sense to
> select it whenever a driver for actual hardware that needs
> it is chosen rather than offer it as an option of its own.
>
> Signed-off-by: Oliver Neukum
> ---
> drivers/net/usb/Kconfig | 75
> +++-
On Mon, Oct 19, 2015 at 03:10:46PM +0800, yalin wang wrote:
> This patch change map_lookup_elem() and map_update_elem() function
> to use u64 temp variable if the key_size or value_size is less than
> u64, we don't need use kmalloc() for these small variables.
>
> Signed-off-by: yalin wang
> ---
Hi,
On Mon, Oct 19, 2015, at 20:15, Alexei Starovoitov wrote:
> On 10/19/15 10:37 AM, Daniel Borkmann wrote:
> > An eBPF program or map loading/destruction is *not* by any means to be
> > considered fast-path. We currently hold a global mutex during loading.
> > So, how can that be considered fast
On Sat, Oct 17, 2015 at 6:12 PM, Thomas F Herbert
wrote:
> Add support for 802.1ad including the ability to push and pop double
> tagged vlans. Add support for 802.1ad to netlink parsing and flow
> conversion. Uses double nested encap attributes to represent double
> tagged vlan. Inner TPID encode
On Sat, Oct 17, 2015 at 6:12 PM, Thomas F Herbert
wrote:
> openvswitch: Add support for 8021.AD
>
> Change the description of the VLAN tpid field.
>
> Signed-off-by: Thomas F Herbert
Acked-by: Pravin B Shelar
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a m
On Sat, Oct 17, 2015 at 6:12 PM, Thomas F Herbert
wrote:
> Signed-off-by: Thomas F Herbert
Acked-by: Pravin B Shelar
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordom
From: Nikolay Borisov
Commit 00590fdd5be0 introduced RCU locking in list type and in
doing so introduced a memory allocation in list_set_add, which
is done in an atomic context, due to the fact that ipset rcu
list modifications are serialised with a spin lock. The reason
why we can't use a mutex
net/built-in.o: In function `nf_dup_ipv4': (.text+0xed24d): undefined reference
to `nf_conntrack_untracked'
net/built-in.o: In function `nf_dup_ipv4': (.text+0xed267): undefined reference
to `nf_conntrack_untracked'
net/built-in.o: In function `nf_dup_ipv6': (.text+0x158aef): undefined
reference
Hi David,
The following patchset contains four Netfilter fixes for net, they are:
1) Fix Kconfig dependencies of new nf_dup_ipv4 and nf_dup_ipv6.
2) Remove bogus test nh_scope in IPv4 rpfilter match that is breaking
--accept-local, from Xin Long.
3) Wait for RCU grace period after dropping t
From: lucien
--accept-local option works for res.type == RTN_LOCAL, which should be
from the local table, but there, the fib_info's nh->nh_scope =
RT_SCOPE_NOWHERE ( > RT_SCOPE_HOST). in fib_create_info().
if (cfg->fc_scope == RT_SCOPE_HOST) {
struct fib_nh *nh = fi->fib
From: Florian Westphal
We need to sync packet rx again after flushing the queue entries.
Otherwise, the following race could happen:
cpu1: nf_unregister_hook(H) called, H unliked from lists, calls
synchronize_net() to wait for packet rx completion.
Problem is that while no new nf_queue_entry st
On 10/19/15 10:37 AM, Daniel Borkmann wrote:
An eBPF program or map loading/destruction is *not* by any means to be
considered fast-path. We currently hold a global mutex during loading.
So, how can that be considered fast-path? Similarly, socket creation/
destruction is also not fast-path, etc.
On Mon, Oct 19, 2015 at 5:37 AM, Elad Raz wrote:
> Configure ageing time to the HW for newly bridged device
>
> CC: Scott Feldman
> CC: Jiri Pirko
> Signed-off-by: Elad Raz
Acked-by: Scott Feldman
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to
On Mon, Oct 19, 2015 at 9:35 AM, Jiri Pirko wrote:
> Sun, Oct 05, 2014 at 02:45:01AM CEST, mahe...@google.com wrote:
>>Earlier change to use usable slave array for TLB mode had an additional
>>performance advantage. So extending the same logic to all other modes
>>that use xmit-hash for slave sele
Mon, Oct 19, 2015 at 08:01:24PM CEST, mahe...@google.com wrote:
>On Mon, Oct 19, 2015 at 9:35 AM, Jiri Pirko wrote:
>> Sun, Oct 05, 2014 at 02:45:01AM CEST, mahe...@google.com wrote:
>>>Earlier change to use usable slave array for TLB mode had an additional
>>>performance advantage. So extending t
On Mon, Oct 19, 2015 at 8:31 AM, James Morse wrote:
> "openvswitch: Remove vport stats" removed the per-vport statistics, in
> order to use the netdev's statistics fields.
> "openvswitch: Fix ovs_vport_get_stats()" fixed the export of these stats
> to user-space, by using the provided netdev_ops t
On 10/19/2015 06:22 PM, Alexei Starovoitov wrote:
On 10/19/15 7:23 AM, Daniel Borkmann wrote:
The mknod is not the holder but rather the kobject which should be
represented in sysfs will be. So you can still get the map major:minor
by looking up the /dev file in the correspdonding sysfs director
Hi David L,
On 10/14/2015 2:15 PM, Santosh Shilimkar wrote:
From: Santosh Shilimkar
To further improve the RDS connection scalabilty on massive systems
where number of sockets grows into tens of thousands of sockets, there
is a need of larger bind hashtable. Pre-allocated 8K or 16K table is
n
This looks like corner case, but worth forwarding.
Begin forwarded message:
Date: Mon, 19 Oct 2015 13:21:33 +
From: "bugzilla-dae...@bugzilla.kernel.org"
To: "shemmin...@linux-foundation.org"
Subject: [Bug 106241] New: shutdown(3)/close(3) behaviour is incorrect for
sockets in accept(3)
> From: Rasmus Villemoes [mailto:li...@rasmusvillemoes.dk]
> Sent: Saturday, October 17, 2015 1:58 PM
> Subject: [PATCH] intel: i40e: fix confused code
>
> This code is pretty confused. The variable name 'bytes_not_copied'
> clearly indicates that the programmer knew the semantics of
> copy_{to,fr
On 10/18/2015 6:56 PM, David Miller wrote:
From: Santosh Shilimkar
Date: Wed, 14 Oct 2015 14:15:31 -0700
From: Santosh Shilimkar
To further improve the RDS connection scalabilty on massive systems
where number of sockets grows into tens of thousands of sockets, there
is a need of larger bin
On Mon, 2015-10-19 at 15:59 +, Insu Yun wrote:
> Freeing sk_buff genereated by skb_recv_datagram is always by
> skb_free_datagram, not kfree_skb.
>
> Signed-off-by: Insu Yun
> ---
> net/nfc/llcp_sock.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/nfc/llcp_soc
This patch fix spelling typos found in Documentation/
DocBook/80211.xml.
It is because this file was generated from comments in source,
so I had to fix comments in source files.
Signed-off-by: Masanari Iida
---
include/net/cfg80211.h | 2 +-
include/net/mac80211.h | 6 +++---
net/mac80211/sta_
Sun, Oct 05, 2014 at 02:45:01AM CEST, mahe...@google.com wrote:
>Earlier change to use usable slave array for TLB mode had an additional
>performance advantage. So extending the same logic to all other modes
>that use xmit-hash for slave selection (viz 802.3AD, and XOR modes).
>Also consolidating t
On 10/19/15 7:23 AM, Daniel Borkmann wrote:
The mknod is not the holder but rather the kobject which should be
represented in sysfs will be. So you can still get the map major:minor
by looking up the /dev file in the correspdonding sysfs directory or I
think we should provide a 'unbind' file, whi
if_nlmsg_size() overestimates the minimum allocation size of netlink
dump request (when called from rtnl_calcit()) or the size of the
message (when called from rtnl_getlink()). This is because
ext_filter_mask is not supported by rtnl_link_get_af_size() and
rtnl_link_get_size().
The over-estimation
On Mon, 19 Oct 2015 17:02:23 +0100
Paul McQuade wrote:
> diff --git a/drivers/net/wireless/b43/phy_lp.c
> b/drivers/net/wireless/b43/phy_lp.c
> index 058a9f2..086f0ba 100644
> --- a/drivers/net/wireless/b43/phy_lp.c
> +++ b/drivers/net/wireless/b43/phy_lp.c
> @@ -2502,7 +2502,7 @@ static int lpp
On Mon, 19 Oct 2015 17:02:22 +0100
Paul McQuade wrote:
> Fixed Pointer Coding Style
>
> Signed-off-by: Paul McQuade
> ---
> drivers/net/wireless/b43/main.c | 6 +++---
> drivers/net/wireless/b43/main.h | 2 +-
> drivers/net/wireless/b43/xmit.h | 2 +-
> 3 files changed, 5 insertions(+), 5 dele
Paul McQuade (3):
net: wireless: rsi: Coding Spaces
net: wireless: b43: Coding Style
net: wireless: b43: Statics are init to 0
drivers/net/wireless/b43/main.c | 8
drivers/net/wireless/b43/main.h | 2 +-
drivers/net/wireless/b43/phy_lp.c | 2 +-
dr
No Need to state Statics are a 0 or NULL
Signed-off-by: Paul McQuade
---
drivers/net/wireless/b43/main.c | 2 +-
drivers/net/wireless/b43/phy_lp.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 040c
Removed space before that ','
Signed-off-by: Paul McQuade
---
drivers/net/wireless/rsi/rsi_91x_sdio_ops.c | 2 +-
drivers/net/wireless/rsi/rsi_mgmt.h | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/rsi/rsi_91x_sdio_ops.c
b/drivers/net/wirele
Fixed Pointer Coding Style
Signed-off-by: Paul McQuade
---
drivers/net/wireless/b43/main.c | 6 +++---
drivers/net/wireless/b43/main.h | 2 +-
drivers/net/wireless/b43/xmit.h | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wire
Freeing sk_buff genereated by skb_recv_datagram is always by
skb_free_datagram, not kfree_skb.
Signed-off-by: Insu Yun
---
net/nfc/llcp_sock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/nfc/llcp_sock.c b/net/nfc/llcp_sock.c
index b7de0da..15e681f 100644
--- a/net/nfc
Clean up array_rd32 so that it uses igb_rd32 the same as rd32, per the
suggestion of Alexander Duyck, and use io_addr in more places, so that
we don't have the need to call E1000_REMOVED (which simply looks for a
null hw_addr) nearly as much.
CC: Mark Rustad
CC: Jeff Kirsher
CC: Alexander Duyck
In commit d999297c3dbbe ("tipc: reduce locking scope during packet reception")
we altered the packet retransmission function. Since then, when
restransmitting packets, we create a clone of the original buffer
using __pskb_copy(skb, MIN_H_SIZE), where MIN_H_SIZE is the size of
the area we want to ha
CONFIG_NET_VENDOR_CAVIUM is only used to hide/show config options and to
include subdirectories in the build, so it doesn't make sense to make it
tristate.
Signed-off-by: Andreas Schwab
---
drivers/net/ethernet/cavium/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dr
On Mon, Oct 19, 2015 at 11:06:49AM +0200, Jiri Benc wrote:
> On Sun, 18 Oct 2015 08:11:58 -0700, Alexei Starovoitov wrote:
> > it looks dangerous.
> > Does it mean that 'for (4B) { create new dev; free old dev; }
> > will keep incrementing that max index and dos it eventually?
>
> This is not chan
1 - 100 of 141 matches
Mail list logo
