Hi Herbert,
On Wed, 17 Jun 2020 00:38:07 +1000 Herbert Xu
wrote:
>
> On Tue, Jun 16, 2020 at 04:38:49AM +0100, Al Viro wrote:
> >
> > Folded and pushed
>
> Thanks Al. Here's another one that I just got, could you add this
> one too?
>
> diff --git a/drivers/mtd/nand/raw/cadence-nand-control
Currently, updating binding table (add service binding to
name table/withdraw a service binding) is being sent over replicast.
However, if we are scaling up clusters to > 100 nodes/containers this
method is less affection because of looping through nodes in a cluster one
by one.
It is worth to use
Alexei Starovoitov wrote:
> On Tue, Jun 16, 2020 at 1:30 AM Lorenz Bauer wrote:
> >
> > On Tue, 16 Jun 2020 at 04:55, Alexei Starovoitov
> > wrote:
> > >
> > > On Mon, Jun 15, 2020 at 7:43 AM Lorenz Bauer wrote:
> > > >
> > > > On Fri, 12 Jun 2020 at 23:36, Alexei Starovoitov
> > > > wrote:
> >
This series of patches adds support to read/write different binary images
of serial flash present in Chelsio terminator.
Vishal Kulkarni (5):
cxgb4: update set_flash to flash different images
cxgb4: add support to flash PHY image
cxgb4: add support to flash boot image
cxgb4: add support to
This patch adds support to dump flash memory via
ethtool --get-dump
Signed-off-by: Vishal Kulkarni
---
drivers/net/ethernet/chelsio/cxgb4/cudbg_if.h | 3 +-
.../net/ethernet/chelsio/cxgb4/cudbg_lib.c| 38 +++
.../net/ethernet/chelsio/cxgb4/cudbg_lib.h| 4 +-
.../net/eth
Update set_flash to flash boot cfg image to flash region
Signed-off-by: Vishal Kulkarni
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4.h| 9 ++
.../ethernet/chelsio/cxgb4/cxgb4_ethtool.c| 30 +++
drivers/net/ethernet/chelsio/cxgb4/t4_hw.c| 90 +++
3 files changed,
Update set_flash to flash boot image to flash region
Signed-off-by: Vishal Kulkarni
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4.h| 48 +
.../ethernet/chelsio/cxgb4/cxgb4_ethtool.c| 56 ++
drivers/net/ethernet/chelsio/cxgb4/t4_hw.c| 187 ++
drivers/net/ether
Update set_flash to flash PHY image to flash region
Signed-off-by: Vishal Kulkarni
---
drivers/net/ethernet/chelsio/cxgb4/cxgb4.h| 1 +
.../ethernet/chelsio/cxgb4/cxgb4_ethtool.c| 39 +++
2 files changed, 40 insertions(+)
diff --git a/drivers/net/ethernet/chelsio/cxgb4/
Chelsio adapter contains different flash regions and each
region is used by different binary files. This patch adds
support to flash images like PHY firmware, boot and boot config
using ethtool -f N.
The N value mapping is as follows.
N = 0 : Parse image and decide which region to flash
N = 1 : Fi
Toke Høiland-Jørgensen wrote:
> Syzkaller discovered that creating a hash of type devmap_hash with a large
> number of entries can hit the memory allocator limit for allocating
> contiguous memory regions. There's really no reason to use kmalloc_array()
> directly in the devmap code, so just switch
Jesper Dangaard Brouer wrote:
> On Tue, 16 Jun 2020 18:35:18 +0800
> Hangbin Liu wrote:
>
> > In commit 34cc0b338a61 we only handled the frame_sz in
> > convert_to_xdp_frame().
> > This patch will also handle frame_sz in xdp_convert_zc_to_xdp_frame().
> >
> > Fixes: 34cc0b338a61 ("xdp: Xdp_fram
On Wed, Jun 10, 2020 at 09:14:34AM -0700, Eric Biggers wrote:
> This series consolidates and modernizes the lists of crypto algorithms
> that are selected by the IPsec kconfig options, and adds CRYPTO_SEQIV
> since it no longer gets selected automatically by other things.
>
> See previous discussi
Roses are red
Violets are blue
We're taking bug fixes
And new features too
Add Marvell 88E1340 support
Signed-off-by: Maxim Kochetkov
---
drivers/net/phy/marvell.c | 23 +++
include/linux/marvell_phy.h | 1 +
2 files changed, 24 insertions(+)
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index 7fc8e10c5f33..4cc4e25fed2d 1006
Add Marvell 88E1548 support
Signed-off-by: Maxim Kochetkov
---
drivers/net/phy/marvell.c | 24
include/linux/marvell_phy.h | 1 +
2 files changed, 25 insertions(+)
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c
index 4cc4e25fed2d..f0d4ca87e4bc 100
Yes. This is 88E1340S. My mistake.
16.06.2020 17:54, Andrew Lunn wrote:
On Tue, Jun 16, 2020 at 10:01:11AM +0300, Maxim Kochetkov wrote:
Add Marvell 88E1340 support
Hi Maxim
Are you sure this is an 88E1340, not a 88E1340S?
Marvells DSDT SDK has:
MAD_88E1340S = 0x1C,/* 88E1340S */
From: Martin
Code to handle multiproto configuration is missing.
Signed-off-by: Martin
---
Changes in v2:
- Initialization of conf structure is removed as that change is included
in another patch.
drivers/net/bareudp.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers
Since the classic device list now supports flags, convert the wakeable
list into a flag on the existing device list.
Signed-off-by: Abhishek Pandit-Subedi
Reviewed-by: Alain Michaud
---
include/net/bluetooth/hci_core.h | 11 ++-
net/bluetooth/hci_core.c | 1 -
net/bluetooth/hc
In order to more easily add device flags to classic devices, create
a new type of bdaddr_list that supports setting flags.
Signed-off-by: Abhishek Pandit-Subedi
Reviewed-by: Alain Michaud
---
include/net/bluetooth/hci_core.h | 18 --
net/bluetooth/hci_core.c | 58 ++
Replace the wakeable boolean with flags in hci_conn_params and all users
of this boolean. This will be used by the get/set device flags mgmt op.
Signed-off-by: Abhishek Pandit-Subedi
Reviewed-by: Alain Michaud
---
include/net/bluetooth/hci_core.h | 2 +-
net/bluetooth/hci_request.c | 3 ++
Add the get device flags and set device flags mgmt ops and the device
flags changed event. Their behavior is described in detail in
mgmt-api.txt in bluez.
Sample btmon trace when a HID device is added (trimmed to 75 chars):
@ MGMT Command: Unknown (0x0050) plen 11{0x0001} [hci0] 18:06:14.
Hi linux-bluetooth,
This series adds support for configuring the Remote Wakeup flag on
devices by implementing Get Device Flags, Set Device Flags and Device
Flags Changed.
This was tested with some userspace changes to update the Remote Wakeup
flag (these changes will be upstreamed as Bluez pat
rt_set_nexthop in __mkroute_output will check the nh->nh_scope value to
determine whether to use the gw or not.
if (nh->nh_gw && nh->nh_scope == RT_SCOPE_LINK) {
rt->rt_gateway = nh->nh_gw;
rt->rt_uses_gateway = 1;
}
On 6/17/2020 4:38 AM, Pablo Neira Ayuso wrote:
> On Tue, Jun 16, 2020 at 05:47:17PM +0200, Simon Horman wrote:
>> On Tue, Jun 16, 2020 at 11:18:16PM +0800, wenxu wrote:
>>> 在 2020/6/16 22:34, Simon Horman 写道:
On Tue, Jun 16, 2020 at 10:20:46PM +0800, wenxu wrote:
> 在 2020/6/16 18:51, Sim
On 2020/6/11 下午7:34, Michael S. Tsirkin wrote:
static void vhost_vq_free_iovecs(struct vhost_virtqueue *vq)
{
kfree(vq->descs);
@@ -394,6 +400,9 @@ static long vhost_dev_alloc_iovecs(struct vhost_dev *dev)
for (i = 0; i < dev->nvqs; ++i) {
vq = dev->vqs[i];
On 6/16/20 8:07 PM, guodeqing wrote:
> ie.,
> $ ifconfig eth0 6.6.6.6 netmask 255.255.255.0
>
> $ ip rule add from 6.6.6.6 table
>
> $ ip route add 9.9.9.9 via 6.6.6.6
>
> $ ping -I 6.6.6.6 9.9.9.9
> PING 9.9.9.9 (9.9.9.9) from 6.6.6.6 : 56(84) bytes of data.
>
> 3 packets transmitted, 0 r
On 6/16/2020 11:47 PM, Simon Horman wrote:
> On Tue, Jun 16, 2020 at 11:18:16PM +0800, wenxu wrote:
>> 在 2020/6/16 22:34, Simon Horman 写道:
>>> On Tue, Jun 16, 2020 at 10:20:46PM +0800, wenxu wrote:
在 2020/6/16 18:51, Simon Horman 写道:
> On Tue, Jun 16, 2020 at 11:19:38AM +0800, we...@uclo
On 6/17/2020 4:13 AM, Pablo Neira Ayuso wrote:
> On Tue, Jun 16, 2020 at 11:19:38AM +0800, we...@ucloud.cn wrote:
>> From: wenxu
>>
>> In the function __flow_block_indr_cleanup, The match stataments
>> this->cb_priv == cb_priv is always false, the flow_block_cb->cb_priv
>> is totally different d
On 6/17/2020 4:30 AM, Pablo Neira Ayuso wrote:
> On Tue, Jun 16, 2020 at 10:17:50PM +0200, Pablo Neira Ayuso wrote:
>> On Tue, Jun 16, 2020 at 11:19:39AM +0800, we...@ucloud.cn wrote:
>>> From: wenxu
>>>
>>> When a indr device add in offload success. The block->nooffloaddevcnt
>>> should be 0. A
On 6/17/2020 4:17 AM, Pablo Neira Ayuso wrote:
> On Tue, Jun 16, 2020 at 11:19:39AM +0800, we...@ucloud.cn wrote:
>> From: wenxu
>>
>> When a indr device add in offload success. The block->nooffloaddevcnt
>> should be 0. After the representor go away. When the dir device go away
>> the flow_bloc
ie.,
$ ifconfig eth0 6.6.6.6 netmask 255.255.255.0
$ ip rule add from 6.6.6.6 table
$ ip route add 9.9.9.9 via 6.6.6.6
$ ping -I 6.6.6.6 9.9.9.9
PING 9.9.9.9 (9.9.9.9) from 6.6.6.6 : 56(84) bytes of data.
3 packets transmitted, 0 received, 100% packet loss, time 2079ms
$ arp
Address H
On Tue, Jun 16, 2020 at 6:24 PM Hao Luo wrote:
>
> Andrii,
>
> Do you think we need to put the kernel's variables in one single
> DATASEC in vmlinux BTF? It looks like all the ksyms in the program
> will be under one ".ksyms" section, so we are not able to tell whether
> a ksym is from a percpu se
Hi David,
> It seems logical to me that what the chip does is align up the total
> sub-packet length to a multiple of 4 or larger, and then add those two
> prefix padding bytes. Otherwise the prefix padding won't necessarily
> and reliably align the IP header after the link level header.
Yep, th
The pull request you sent on Tue, 16 Jun 2020 16:25:51 -0700 (PDT):
> git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net refs/heads/master
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/69119673bd50b176ded34032fadd41530fb5af21
Thank you!
--
Deet-doot-dot, I am
On Wed, Jun 17, 2020 at 01:01:30AM +0200, David Sterba wrote:
> On Tue, Jun 16, 2020 at 11:53:50AM -0700, Joe Perches wrote:
> > On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
> > > v4:
> > > - Break out the memzero_explicit() change as suggested by Dan Carpenter
> > > so that it can
From: Vladimir Oltean
This action requires the VLAN awareness state of the switch to be of the
same type as the key that's being added:
- If the switch is unaware of VLAN, then the tc filter key must only
contain the destination MAC address.
- If the switch is VLAN-aware, the key must also con
From: Vladimir Oltean
This action requires the VLAN awareness state of the switch to be of the
same type as the key that's being added:
- If the switch is unaware of VLAN, then the tc filter key must only
contain the destination MAC address.
- If the switch is VLAN-aware, the key must also con
From: Vladimir Oltean
This fixes a ridiculous situation where the driver, in VLAN-unaware
mode, would refuse accepting any tc filter:
tc filter replace dev sw1p3 ingress flower skip_sw \
dst_mac 42:be:24:9b:76:20 \
action gate (...)
Error: sja1105: Can only gate based on {DMAC, V
From: Vladimir Oltean
This shouldn't be there.
Fixes: 834f8933d5dd ("net: dsa: sja1105: implement tc-gate using time-triggered
virtual links")
Signed-off-by: Vladimir Oltean
---
drivers/net/dsa/sja1105/sja1105_vl.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/net/dsa/sja1105
1) Don't get per-cpu pointer with preemption enabled in nft_set_pipapo,
fix from Stefano Brivio.
2) Fix memory leak in ctnetlink, from Pablo Neira Ayuso.
3) Multiple definitions of MPTCP_PM_MAX_ADDR, from Geliang Tang.
4) Accidently disabling NAPI in non-error paths of macb_open(), from
From: Jeff Kirsher
Date: Tue, 16 Jun 2020 15:53:51 -0700
> This series contains fixes to e1000 and e1000e.
>
> Chen fixes an e1000e issue where systems could be waken via WoL, even
> though the user has disabled the wakeup bit via sysfs.
>
> Vaibhav Gupta updates the e1000 driver to clean up th
On Tue, Jun 16, 2020 at 3:23 PM Daniel Borkmann wrote:
>
> On 6/16/20 11:27 PM, Andrii Nakryiko wrote:
> > On Tue, Jun 16, 2020 at 1:21 PM Daniel Borkmann
> > wrote:
> >> On 6/16/20 7:04 AM, Andrii Nakryiko wrote:
> >>> Add selftest that validates variable-length data reading and
> >>> concaten
> The host_kbox_drv driver serves the function of a black box. When a
> panic or mce event happen to the system, it will record the event
> time, system's status and system logs and send them to BMC before
> the OS shutdown. This driver depends on the host_edms_drv driver.
This does not belong in
On Tue, Jun 16, 2020 at 3:53 PM Stanislav Fomichev wrote:
> + if (ctx->level == SOL_IP && ctx->optname == IP_FREEBIND) {
> + if (optval > optval_end) {
same issue as before ?
see reply in v3.
> + /* For optval > PAGE_SIZE, the actual data
> +
On Tue, Jun 16, 2020 at 3:53 PM Stanislav Fomichev wrote:
>
> Attaching to these hooks can break iptables because its optval is
> usually quite big, or at least bigger than the current PAGE_SIZE limit.
> David also mentioned some SCTP options can be big (around 256k).
>
> There are two possible wa
On Mon, Jun 15, 2020 at 09:41:38AM -0700, Stanislav Fomichev wrote:
> On Fri, Jun 12, 2020 at 8:50 PM Alexei Starovoitov
> wrote:
> [ .. ]
> > > > It's probably ok, but makes me uneasy about verifier consequences.
> > > > ctx->optval is PTR_TO_PACKET and it's a valid pointer from verifier pov.
> >
On Tue, Jun 16, 2020 at 11:53:50AM -0700, Joe Perches wrote:
> On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
> > v4:
> > - Break out the memzero_explicit() change as suggested by Dan Carpenter
> > so that it can be backported to stable.
> > - Drop the "crypto: Remove unnecessary me
Make use of the struct_size() helper instead of an open-coded version
in order to avoid any potential type mistakes.
This code was detected with the help of Coccinelle and, audited and
fixed manually.
Signed-off-by: Gustavo A. R. Silva
---
drivers/net/ethernet/stmicro/stmmac/stmmac_selftests.c
> This cdev_veth_drv driver is one of the communication drivers used
> by BMA software. It depends on the host_edma_drv driver. It will
> create a char device once loaded, offering interfaces (open, close,
> read, write and poll) to BMA to send/receive RESTful messages
> between BMC software. When
From: Arnd Bergmann
The CONFIG_PM_SLEEP #ifdef checks in this file are inconsistent,
leading to a warning about sometimes unused function:
drivers/net/ethernet/intel/e1000e/netdev.c:137:13: error: unused function
'e1000e_check_me' [-Werror,-Wunused-function]
Rather than adding more #ifdefs, ju
This series contains fixes to e1000 and e1000e.
Chen fixes an e1000e issue where systems could be waken via WoL, even
though the user has disabled the wakeup bit via sysfs.
Vaibhav Gupta updates the e1000 driver to clean up the legacy Power
Management hooks.
Arnd Bergmann cleans up the inconsist
From: Vaibhav Gupta
With legacy PM hooks, it was the responsibility of a driver to manage PCI
states and also the device's power state. The generic approach is to let PCI
core handle the work.
e1000_suspend() calls __e1000_shutdown() to perform intermediate tasks.
__e1000_shutdown() modifies the
From: Chen Yu
Currently the system will be woken up via WOL(Wake On LAN) even if the
device wakeup ability has been disabled via sysfs:
cat /sys/devices/pci:00/:00:1f.6/power/wakeup
disabled
The system should not be woken up if the user has explicitly
disabled the wake up ability for t
On Tue, 16 Jun 2020 at 23:25, Davide Caratti wrote:
>
> assigning a dummy value of 'clock_id' to avoid cancellation of the cycle
> timer before its initialization was a temporary solution, and we still
> need to handle the case where act_gate timer parameters are changed by
> commands like the fol
On Tue, 16 Jun 2020 at 23:25, Davide Caratti wrote:
>
> it is possible to see a KASAN use-after-free, immediately followed by a
> NULL dereference crash, with the following command:
>
> # tc action add action gate index 3 cycle-time 1ns \
> > cycle-time-ext 1ns clockid CLOCK_TAI
Make use of the struct_size() helper instead of an open-coded version
in order to avoid any potential type mistakes. Also, remove unnecessary
variable _len_.
This code was detected with the help of Coccinelle and, audited and
fixed manually.
Signed-off-by: Gustavo A. R. Silva
---
drivers/net/wi
On Tue, Jun 16, 2020 at 09:49:25PM +, Anchal Agarwal wrote:
> On Thu, Jun 04, 2020 at 09:05:48AM +0200, Roger Pau Monné wrote:
> > CAUTION: This email originated from outside of the organization. Do not
> > click links or open attachments unless you can confirm the sender and know
> > the con
On 6/16/20 11:27 PM, Andrii Nakryiko wrote:
On Tue, Jun 16, 2020 at 1:21 PM Daniel Borkmann wrote:
On 6/16/20 7:04 AM, Andrii Nakryiko wrote:
Add selftest that validates variable-length data reading and concatentation
with one big shared data array. This is a common pattern in production use f
On Tue, Jun 16, 2020 at 05:23:43PM +0200, Eugenio Perez Martin wrote:
> On Mon, Jun 15, 2020 at 6:05 PM Eugenio Pérez wrote:
> >
> > On Thu, 2020-06-11 at 07:30 -0400, Michael S. Tsirkin wrote:
> > > On Wed, Jun 10, 2020 at 06:18:32PM +0200, Eugenio Perez Martin wrote:
> > > > On Wed, Jun 10, 2020
On Thu, Jun 04, 2020 at 09:05:48AM +0200, Roger Pau Monné wrote:
> CAUTION: This email originated from outside of the organization. Do not click
> links or open attachments unless you can confirm the sender and know the
> content is safe.
>
>
>
> Hello,
>
> On Wed, Jun 03, 2020 at 11:33:52PM
On Tue, Jun 16, 2020 at 1:21 PM Daniel Borkmann wrote:
>
> On 6/16/20 7:04 AM, Andrii Nakryiko wrote:
> > Add selftest that validates variable-length data reading and concatentation
> > with one big shared data array. This is a common pattern in production use
> > for
> > monitoring and tracing a
> -Original Message-
> From: Eric Dumazet
> On 6/13/20 11:11 PM, Greg Thelen wrote:
> > Commit e086ba2fccda ("e1000e: disable s0ix entry and exit flows for ME
> > systems") added e1000e_check_me() but it's only called from
> > CONFIG_PM_SLEEP protected code. Thus builds without CONFIG_PM_
On Tue, Jun 16, 2020 at 11:53:50AM -0700, Joe Perches wrote:
> To this larger audience and last week without reply:
> https://lore.kernel.org/lkml/573b3fbd5927c643920e1364230c296b23e7584d.ca...@perches.com/
>
> Are there _any_ fastpath uses of kfree or vfree?
I worked on adding a 'free' a couple
From: Tim Harvey
Date: Tue, 16 Jun 2020 08:31:58 -0700
> Without a MODULE_DEVICE_TABLE the attributes are missing that create
> an alias for auto-loading the module in userspace via hotplug.
>
> Signed-off-by: Tim Harvey
Applied, thank you.
From: Jeremy Kerr
Date: Tue, 16 Jun 2020 17:08:23 +0800
>> Because that code in this loop makes the same calculations:
>>
>> ax_skb = skb_clone(skb, GFP_ATOMIC);
>> if (ax_skb) {
>> ax_skb->len = pkt_len;
>> ax_skb->
From: Ido Schimmel
Date: Tue, 16 Jun 2020 10:14:58 +0300
> From: Ido Schimmel
>
> The port's headroom buffers are used to store packets while they
> traverse the device's pipeline and also to store packets that are egress
> mirrored.
>
> On Spectrum-3, ports with eight lanes use two headroom b
From: Martin Varghese
Date: Tue, 16 Jun 2020 11:18:58 +0530
> From: Martin
>
> Code to initialize the conf structure while gathering the configuration
> of the device was missing.
>
> Fixes: 571912c69f0e ("net: UDP tunnel encapsulation module for tunnelling
> different protocols like MPLS, IP
From: Sven Auhagen
Date: Tue, 16 Jun 2020 06:35:29 +0200
> The remove function does not destroy all
> BM Pools when per cpu pool is active.
>
> When reloading the mvpp2 as a module the BM Pools
> are still active in hardware and due to the bug
> have twice the size now old + new.
>
> This event
On Tue, Jun 16, 2020 at 05:47:17PM +0200, Simon Horman wrote:
> On Tue, Jun 16, 2020 at 11:18:16PM +0800, wenxu wrote:
> >
> > 在 2020/6/16 22:34, Simon Horman 写道:
> > > On Tue, Jun 16, 2020 at 10:20:46PM +0800, wenxu wrote:
> > >> 在 2020/6/16 18:51, Simon Horman 写道:
> > >>> On Tue, Jun 16, 2020 at
From: Eric Dumazet
Date: Mon, 15 Jun 2020 20:37:07 -0700
> Back in 2013, we made a change that broke fast retransmit
> for non SACK flows.
>
> Indeed, for these flows, a sender needs to receive three duplicate
> ACK before starting fast retransmit. Sending ACK with different
> receive window do
On Tue, Jun 16, 2020 at 1:30 AM Lorenz Bauer wrote:
>
> On Tue, 16 Jun 2020 at 04:55, Alexei Starovoitov
> wrote:
> >
> > On Mon, Jun 15, 2020 at 7:43 AM Lorenz Bauer wrote:
> > >
> > > On Fri, 12 Jun 2020 at 23:36, Alexei Starovoitov
> > > wrote:
> > > >
> > > > On Fri, Jun 12, 2020 at 9:02 AM
On Tue, Jun 16, 2020 at 10:17:50PM +0200, Pablo Neira Ayuso wrote:
> On Tue, Jun 16, 2020 at 11:19:39AM +0800, we...@ucloud.cn wrote:
> > From: wenxu
> >
> > When a indr device add in offload success. The block->nooffloaddevcnt
> > should be 0. After the representor go away. When the dir device g
assigning a dummy value of 'clock_id' to avoid cancellation of the cycle
timer before its initialization was a temporary solution, and we still
need to handle the case where act_gate timer parameters are changed by
commands like the following one:
# tc action replace action gate
the fix consist
- patch 1/2 attempts to fix the error path of tcf_gate_init() when users
try to configure 'act_gate' rules with wrong parameters
- patch 2/2 is a follow-up of a recent fix for NULL dereference in
the error path of tcf_gate_init()
further work will introduce a tdc test for 'act_gate'.
changes
it is possible to see a KASAN use-after-free, immediately followed by a
NULL dereference crash, with the following command:
# tc action add action gate index 3 cycle-time 1ns \
> cycle-time-ext 1ns clockid CLOCK_TAI
BUG: KASAN: use-after-free in tcf_action_init_1+0x8eb/0x960
W
On 6/16/20 7:04 AM, Andrii Nakryiko wrote:
Add selftest that validates variable-length data reading and concatentation
with one big shared data array. This is a common pattern in production use for
monitoring and tracing applications, that potentially can read a lot of data,
but usually reads muc
On Tue, Jun 16, 2020 at 11:19:39AM +0800, we...@ucloud.cn wrote:
> From: wenxu
>
> When a indr device add in offload success. The block->nooffloaddevcnt
> should be 0. After the representor go away. When the dir device go away
> the flow_block UNBIND operation with -EOPNOTSUPP which lead the warn
On Tue, Jun 16, 2020 at 11:19:38AM +0800, we...@ucloud.cn wrote:
> From: wenxu
>
> In the function __flow_block_indr_cleanup, The match stataments
> this->cb_priv == cb_priv is always false, the flow_block_cb->cb_priv
> is totally different data with the flow_indr_dev->cb_priv.
>
> Store the rep
On Tue, Jun 16, 2020 at 11:19:37AM +0800, we...@ucloud.cn wrote:
> From: wenxu
>
> The cleanup operation based on the setup callback. But in the mlx5e
> driver there are tc and flowtable indrict setup callback and shared
> the same release callbacks. So when the representor is removed,
> then ide
On 6/16/20 2:53 PM, Joe Perches wrote:
On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
v4:
- Break out the memzero_explicit() change as suggested by Dan Carpenter
so that it can be backported to stable.
- Drop the "crypto: Remove unnecessary memzero_explicit()" patch for
On Tue, Jun 16, 2020 at 12:54 PM Joe Perches wrote:
>
> On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
> > v4:
> > - Break out the memzero_explicit() change as suggested by Dan Carpenter
> > so that it can be backported to stable.
> > - Drop the "crypto: Remove unnecessary memzero_
On 6/16/20 2:53 PM, Joe Perches wrote:
On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
v4:
- Break out the memzero_explicit() change as suggested by Dan Carpenter
so that it can be backported to stable.
- Drop the "crypto: Remove unnecessary memzero_explicit()" patch for
On Mon, 2020-06-15 at 21:57 -0400, Waiman Long wrote:
> v4:
> - Break out the memzero_explicit() change as suggested by Dan Carpenter
> so that it can be backported to stable.
> - Drop the "crypto: Remove unnecessary memzero_explicit()" patch for
> now as there can be a bit more discus
Hi Manivannan, David
On 02/13/20 14:44, Manivannan Sadhasivam wrote:
[...]
> + trace_printk("advertising new server [%d:%x]@[%d:%d]\n",
> + srv->service, srv->instance, srv->node, srv->port);
I can't tell exactly from the discussion whether this is the version that got
merg
On 6/16/20 2:09 PM, Andrew Morton wrote:
On Tue, 16 Jun 2020 11:43:11 -0400 Waiman Long wrote:
As said by Linus:
A symmetric naming is only helpful if it implies symmetries in use.
Otherwise it's actively misleading.
In "kzalloc()", the z is meaningful and an important part of what
On Tue, 16 Jun 2020 11:43:11 -0400 Waiman Long wrote:
> As said by Linus:
>
> A symmetric naming is only helpful if it implies symmetries in use.
> Otherwise it's actively misleading.
>
> In "kzalloc()", the z is meaningful and an important part of what the
> caller wants.
>
> In "kz
When we clone a socket in sk_clone_lock(), its sk_cgrp_data is
copied, so the cgroup refcnt must be taken too. And, unlike the
sk_alloc() path, sock_update_netprioidx() is not called here.
Therefore, it is safe and necessary to grab the cgroup refcnt
even when cgroup_sk_alloc is disabled.
sk_clone
Similar to ip_vti, IPIP and IPIP6 tunnels processing can easily
be done with .cb_handler for xfrm interface.
Signed-off-by: Xin Long
---
net/xfrm/xfrm_interface.c | 24
1 file changed, 24 insertions(+)
diff --git a/net/xfrm/xfrm_interface.c b/net/xfrm/xfrm_interface.c
i
Similar to ip6_vti, IP6IP6 and IP6IP tunnels processing can easily
be done with .cb_handler for xfrm interface.
Signed-off-by: Xin Long
---
net/xfrm/xfrm_interface.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/net/xfrm/xfrm_interface.c b/net/xfrm/xfrm_inte
For IPIP6 tunnel processing, the functions called will be the
same as that for IPIP tunnel's. So reuse it and register it
with family == AF_INET6.
Signed-off-by: Xin Long
---
net/ipv4/ip_vti.c | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/net/ipv4/ip_vti.c
For IP6IP tunnel processing, the functions called will be the
same as that for IP6IP6 tunnel's. So reuse it and register it
with family == AF_INET.
Signed-off-by: Xin Long
---
net/ipv6/ip6_vti.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/net/ipv6/ip6_vti.c b/n
Now ipip and ipv6 tunnels processing is supported by xfrm4/6_tunnel,
but not in vti and xfrmi. This feature is needed by processing those
uncompressed small fragments and packets when using comp protocol.
It means vti and xfrmi won't be able to accept small fragments or
packets when using comp prot
This patch is to register a callback function tunnel4_rcv_cb with
is_ipip set in a xfrm_input_afinfo object for tunnel4 and tunnel64.
It will be called by xfrm_rcv_cb() from xfrm_input() when family
is AF_INET and proto is IPPROTO_IPIP or IPPROTO_IPV6.
Signed-off-by: Xin Long
---
include/net/xf
This patch is to add a new member is_ipip to struct xfrm_input_afinfo,
to allow another group family of callback functions to be registered
with is_ipip set.
This will be used for doing a callback for struct xfrm(6)_tunnel of
ipip/ipv6 tunnels in xfrm_input() by calling xfrm_rcv_cb(), which is
nee
This patch is to register a callback function tunnel6_rcv_cb with
is_ipip set in a xfrm_input_afinfo object for tunnel6 and tunnel46.
It will be called by xfrm_rcv_cb() from xfrm_input() when family
is AF_INET6 and proto is IPPROTO_IPIP or IPPROTO_IPV6.
Signed-off-by: Xin Long
---
include/net/x
With tunnel4_input_afinfo added, IPIP tunnel processing in
ip_vti can be easily done with .cb_handler. So replace the
processing by calling ip_tunnel_rcv() with it.
Signed-off-by: Xin Long
---
net/ipv4/ip_vti.c | 33 -
1 file changed, 8 insertions(+), 25 deletions
Similar to IPIP tunnel's processing, this patch is to support
IP6IP6 tunnel processing with .cb_handler.
Signed-off-by: Xin Long
---
net/ipv6/ip6_vti.c | 25 +
1 file changed, 25 insertions(+)
diff --git a/net/ipv6/ip6_vti.c b/net/ipv6/ip6_vti.c
index 1147f64..2161648 10
This way we can have trampoline on function
that has arguments with types like:
kuid_t uid
kgid_t gid
which unwind into small structs like:
typedef struct {
uid_t val;
} kuid_t;
typedef struct {
gid_t val;
} kgid_t;
And we can use them in bpftrace like:
(assuming d_
The child tunnel if_id will be used for xfrm interface's lookup
when processing the IP(6)IP(6) packets in the next patches.
Signed-off-by: Xin Long
---
net/ipv4/ipcomp.c | 1 +
net/ipv6/ipcomp6.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/net/ipv4/ipcomp.c b/net/ipv4/ipcomp.c
index 5
On Tue, Jun 16, 2020 at 08:02:58AM +0200, Steffen Klassert wrote:
> On Mon, Jun 15, 2020 at 03:13:15PM -0700, Eric Biggers wrote:
> > This series consolidates and modernizes the lists of crypto algorithms
> > that are selected by the IPsec kconfig options, and adds CRYPTO_SEQIV
> > since it no long
1 - 100 of 225 matches
Mail list logo
