[Patch net] smc: move unhash as early as possible in smc_release()

2019-01-05 Thread Cong Wang
In smc_release() we release smc->clcsock before unhash the smc sock, but a parallel smc_diag_dump() may be still reading smc->clcsock, therefore this could cause a use-after-free as reported by syzbot. Reported-and-tested-by: syzbot+fbd1e5476e4c94c7b...@syzkaller.appspotmail.com Fixes: 51f1de79ad8

Re: [PATCH] tipc: fix memory leak in tipc_nl_compat_publ_dump

2019-01-05 Thread Ying Xue
On 1/6/19 12:52 AM, Gustavo A. R. Silva wrote: > There is a memory leak in case genlmsg_put fails. > > Fix this by freeing *args* before return. > > Addresses-Coverity-ID: 1476406 ("Resource leak") > Fixes: 46273cf7e009 ("tipc: fix a missing check of genlmsg_put") > Signed-off-by: Gustavo A. R. S

Re: [PATCH net 1/4] umh: add exit routine for UMH process

2019-01-05 Thread Taehee Yoo
On Sun, 6 Jan 2019 at 07:10, David Miller wrote: > > From: Taehee Yoo > Date: Mon, 31 Dec 2018 01:31:43 +0900 > > > +void exit_umh(struct task_struct *tsk) > > +{ > > + struct umh_info *info; > > + pid_t pid = tsk->pid; > > + > > + mutex_lock(&umh_list_lock); > > + list_for_each_e

Re: [PATCH bpf 0/2] bpf fix to reject mixing ptr / scalar alu op on unpriv

2019-01-05 Thread Alexei Starovoitov
On Sun, Jan 06, 2019 at 12:54:36AM +0100, Daniel Borkmann wrote: > Follow-up fix to 979d63d50c0c ("bpf: prevent out of bounds speculation > on pointer arithmetic") in order to reject a corner case for sanitation > when ptr / scalars are mixed in the same alu op. Applied, Thanks

[PATCH 1/1] net: bridge: fix a bug on using a neighbour cache entry without checking its state

2019-01-05 Thread kchen
From: JianJhen Chen When handling DNAT'ed packets on a bridge device, the neighbour cache entry from lookup was used without checking its state. It means that a cache entry in the NUD_STALE state will be used directly instead of entering the NUD_DELAY state to confirm the reachability of the neig

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Florian Fainelli
On January 5, 2019 5:21:03 AM PST, Heiner Kallweit wrote: >During a bug analysis we came across the fact that there's no guarantee >that reading from the ID registers returns a valid value if PHY is >powered down. When reading invalid values we may load no or the wrong >PHY driver. Therefore le

[PATCH v3 3/4] rtl8723ae: Re-introduce the adaptive rate control

2019-01-05 Thread Bernd Edlinger
This re-introduces the function rtl8723e_dm_refresh_rate_adaptive_mask. This function was present in a previous version of the code base, it works just fine for me -- as long as it is not using stale data. Unlike the original version of this function it avoids using dm.undec_sm_pwdb when no beaco

[PATCH v3 4/4] rtlwifi: Don't clear num_rx_inperiod too early

2019-01-05 Thread Bernd Edlinger
This patch moves the clearing of rtlpriv->link_info.num_rx_inperiod in rtl_watchdog_wq_callback a few lines down. This is necessary since it is still used in the "AP off" detection code block. Moved clearing of rtlpriv->link_info.num_rx_inperiod as well for consistency. Signed-off-by: Bernd Edlin

[PATCH v3 2/4] rtl8723ae: Dont use old data for input gain control

2019-01-05 Thread Bernd Edlinger
When no beacon was received, the value in dm.undec_sm_pwdb is most likely out of date and should not be used to adjust the input path. Assume instead that the signal level is low. Fix the state machine in rtl8723e_dm_cck_packet_detection_thresh which did not clear pre_cck_fa_state when changing cu

[PATCH v3 1/4] rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Bernd Edlinger
This appears to trigger a firmware bug and causes severe problems with rtl8723ae PCI devices. When the power save mode is activated for longer periods of time the firmware stops to receive any packets. This problem was exposed by commit 873ffe154ae0 ("rtlwifi: Fix logic error in enter/exit power-

[PATCH v3 0/4] rtlwifi: Fix issues with rtl8723ae

2019-01-05 Thread Bernd Edlinger
Currently the rtl8723ae driver is broken (since v4.7). Connection to AP is lost very often, especially when the signal level is not very good. The main issue is the power save mode is basically not working, and seems to trigger a firmware bug. So I had to take out the FW LPS mode handling. While

[PATCH bpf 0/2] bpf fix to reject mixing ptr / scalar alu op on unpriv

2019-01-05 Thread Daniel Borkmann
Follow-up fix to 979d63d50c0c ("bpf: prevent out of bounds speculation on pointer arithmetic") in order to reject a corner case for sanitation when ptr / scalars are mixed in the same alu op. Thanks! Daniel Borkmann (2): bpf: fix sanitation of alu op with pointer / scalar type from different pa

[PATCH bpf 1/2] bpf: fix sanitation of alu op with pointer / scalar type from different paths

2019-01-05 Thread Daniel Borkmann
While 979d63d50c0c ("bpf: prevent out of bounds speculation on pointer arithmetic") took care of rejecting alu op on pointer when e.g. pointer came from two different map values with different map properties such as value size, Jann reported that a case was not covered yet when a given alu op is us

[PATCH bpf 2/2] bpf: add various test cases for alu op on mixed dst register types

2019-01-05 Thread Daniel Borkmann
Add couple of test_verifier tests to check sanitation of alu op insn with pointer and scalar type coming from different paths. This also includes BPF insns of the test reproducer provided by Jann Horn. Signed-off-by: Daniel Borkmann Acked-by: Alexei Starovoitov --- tools/testing/selftests/bpf/t

Re: [nf-next] netfilter: Add support for inner IPv6 packet match

2019-01-05 Thread David R. Bild
On Jan 25, 2018 at 08:13:00PM, Pabel wrote: > On Thu, Jan 18, 2018 at 04:13:25PM +0100, Ahmed Abdelsalam wrote: > [...] > > diff --git a/include/uapi/linux/netfilter_ipv6/ip6t_inner6.h > > b/include/uapi/linux/netfilter_ipv6/ip6t_inner6.h > > new file mode 100644 > > index 000..7017fa4 > > ---

Re: [PATCH 3/5] doc: networking: update references to files describing offloads

2019-01-05 Thread Otto Sabart
On 05. Jan (Saturday) v 14:15:16 -0800 2019, David Miller wrote: > From: Otto Sabart > Date: Fri, 4 Jan 2019 11:42:29 +0100 > > > Update reference to checksum-offloads.rst file which was converted into > > RST. > > > > Whole kernel code was grepped for references using: > > $ grep -r "\(segmenta

[PATCH v2 3/4] doc: networking: shorten the main title in offloads documents

2019-01-05 Thread Otto Sabart
The titles do not look very nice in the table of contents generated by Sphinx. I also think it is obvious that the documents are describing offloads in the Linux Networking Stack. Signed-off-by: Otto Sabart --- Documentation/networking/checksum-offloads.rst | 6 +++--- Documentation/network

[PATCH v2 4/4] doc: networking: add offload documents into main index file

2019-01-05 Thread Otto Sabart
This patch just adds references to offload documents into main table of contents in network documentation. Signed-off-by: Otto Sabart --- Documentation/networking/index.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/networking/index.rst b/Documentation/networking/index.

[PATCH v2 1/4] doc: networking: prepare offload documents for conversion into RST

2019-01-05 Thread Otto Sabart
Add small number of markups which are sufficient for conversion into reStructuredText. Unfortunately there was necessary to restructure all sections in checksum-offloads.txt file and create paragraphs separated by newline. There also must not be a space at the beginning of paragpraph. There are n

[PATCH v2 2/4] doc: networking: convert offload files into RST and update references

2019-01-05 Thread Otto Sabart
This patch renames offload files. This is necessary for Sphinx. Also update reference to checksum-offloads.rst file. Whole kernel code was grepped for references using: $ grep -r "\(segmentation\|checksum\)-offloads.txt" . There should be no other references to {segmentation,checksum}-offloads.t

[PATCH v2 0/4] doc: network: integrate offload documents into doc tree

2019-01-05 Thread Otto Sabart
Changes in v2: - Patch #3 and #2 were combined into patch #2. --- This series of patches integrates checksum-offloads and segmentation-offloads documents into documentation tree. These patches do not change semantics of these documents. There are only changes which are needed for successful inte

Re: [PATCH] PCI: Add no-D3 quirk for Mellanox ConnectX-[45]

2019-01-05 Thread Benjamin Herrenschmidt
On Sat, 2019-01-05 at 10:51 -0700, Jason Gunthorpe wrote: > > > Interesting. I've investigated this further, though I don't have as > > many new clues as I'd like. The problem occurs reliably, at least on > > one particular type of machine (a POWER8 "Garrison" with ConnectX-4). > > I don't yet k

Re: [PATCH net 1/4] umh: add exit routine for UMH process

2019-01-05 Thread David Miller
From: Taehee Yoo Date: Mon, 31 Dec 2018 01:31:43 +0900 > +void exit_umh(struct task_struct *tsk) > +{ > + struct umh_info *info; > + pid_t pid = tsk->pid; > + > + mutex_lock(&umh_list_lock); > + list_for_each_entry(info, &umh_list, list) { So this is probably too expensive of a c

Re: [PATCH 3/5] doc: networking: update references to files describing offloads

2019-01-05 Thread David Miller
From: Otto Sabart Date: Fri, 4 Jan 2019 11:42:29 +0100 > Update reference to checksum-offloads.rst file which was converted into > RST. > > Whole kernel code was grepped for references using: > $ grep -r "\(segmentation\|checksum\)-offloads.txt" . > > There should be no other references > to {s

Re: [PATCH net] ipv6: Take rcu_read_lock in __inet6_bind for mapped addresses

2019-01-05 Thread David Miller
From: David Ahern Date: Sat, 5 Jan 2019 07:35:04 -0800 > From: David Ahern > > I realized the last patch calls dev_get_by_index_rcu in a branch not > holding the rcu lock. Add the calls to rcu_read_lock and rcu_read_unlock. > > Fixes: ec90ad334986 ("ipv6: Consider sk_bound_dev_if when binding

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread David Miller
From: Andrew Lunn Date: Sat, 5 Jan 2019 23:04:12 +0100 > I tend to be more paranoid after listening to recent discussions about > this. At LPC and online, there have been comments that patches to > stable are more likely to break something than patches going via the > normal merge window. Normal

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Andrew Lunn
> Quite a few of the latest net commits don't meet the strict criteria > for a fix (as documented). Means: The risk that a problem could > occur isn't sufficient, at least one user has to actually face a > problem. So it seems net vs. net-next criteria is somewhat flexible. > Therefore I wasn't sur

Re: [RFT][PATCH V2 09/10] net: dsa: microchip: Factor out regmap config generation into common header

2019-01-05 Thread Marek Vasut
On 12/21/18 6:23 AM, Marek Vasut wrote: > On 12/21/2018 05:16 AM, tristram...@microchip.com wrote: >>> + { \ >>> + .val_bits = (width),\ >>> + .reg_stride = (width) / 8,

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread David Miller
From: Heiner Kallweit Date: Sat, 5 Jan 2019 22:07:50 +0100 > Regarding net vs. net-next: > Quite a few of the latest net commits don't meet the strict criteria > for a fix (as documented). Means: The risk that a problem could > occur isn't sufficient, at least one user has to actually face a > pr

Re: [PATCH v2 0/4] rtlwifi: Fix issues with rtl8723ae

2019-01-05 Thread Larry Finger
On 1/5/19 12:38 PM, Bernd Edlinger wrote: Currently the rtl8723ae driver is broken (since v4.7). Connection to AP is lost very often, especially when the signal level is not very good. The main issue is the power save mode is basically not working, and seems to trigger a firmware bug. So I had

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Heiner Kallweit
On 05.01.2019 18:33, Andrew Lunn wrote: > On Sat, Jan 05, 2019 at 02:21:03PM +0100, Heiner Kallweit wrote: >> During a bug analysis we came across the fact that there's no guarantee >> that reading from the ID registers returns a valid value if PHY is >> powered down. When reading invalid values we

Re: ip netns exec hides mount points from child processes

2019-01-05 Thread Eric W. Biederman
Naja Melan writes: > hi, > > I have been using network namespaces for a while, mostly with good results. > Recently I ran into a problem where the cgroup mount points are missing for > software that needs it (runc). > > I discovered that ip netns exec creates a mount namespace to bind mount >

Re: [PATCH v2 2/4] rtlwifi: rtl8723ae: Don't use dm.undec_sm_pwdb for input

2019-01-05 Thread Andrew Lunn
On Sat, Jan 05, 2019 at 06:39:02PM +, Bernd Edlinger wrote: > gain control when no beacon was received in the connected state Hi Bernd Your patch subject line is too long, so it got chopped in half. Please use something shorter. Andrew

[PATCH v2 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Bernd Edlinger
This appears to trigger a firmware bug and causes severe problems with rtl8723ae PCI devices. When the power save mode is activated for longer periods of time the firmware stops to receive any packets. This problem was exposed by commit 873ffe154ae0 ("rtlwifi: Fix logic error in enter/exit power-

[PATCH v2 4/4] rtlwifi: Move the clearing of

2019-01-05 Thread Bernd Edlinger
rtlpriv->link_info.num_rx_inperiod in rtl_watchdog_wq_callback a few lines down This is necessary since it is still used in the "AP off" detection code block. Moved clearing of rtlpriv->link_info.num_rx_inperiod as well for consistency. Signed-off-by: Bernd Edlinger --- drivers/net/wireless/r

[PATCH v2 3/4] rtlwifi: rtl8723ae: Re-introduce

2019-01-05 Thread Bernd Edlinger
rtl8723e_dm_refresh_rate_adaptive_mask This function was present in a previous version of the code base, it works just fine for me -- as long as it is not using stale data. Fixed a style nit in rtl8723e_dm_init_rate_adaptive_mask. Signed-off-by: Bernd Edlinger --- .../net/wireless/realtek/rtl

[PATCH v2 2/4] rtlwifi: rtl8723ae: Don't use dm.undec_sm_pwdb for input

2019-01-05 Thread Bernd Edlinger
gain control when no beacon was received in the connected state When no beacon was received, the value in dm.undec_sm_pwdb is most likely out of date and should not be used to adjust the input path. Assume instead that the signal level is low. Fix the state machine in rtl8723e_dm_cck_packet_dete

[PATCH v2 0/4] rtlwifi: Fix issues with rtl8723ae

2019-01-05 Thread Bernd Edlinger
Currently the rtl8723ae driver is broken (since v4.7). Connection to AP is lost very often, especially when the signal level is not very good. The main issue is the power save mode is basically not working, and seems to trigger a firmware bug. So I had to take out the FW LPS mode handling. While

Re: [Patch net v2] mlx5: fixup checksum for short ethernet frame padding

2019-01-05 Thread Nikola Ciprich
Hi Saeed, > Most likely the same issue, we are finalizing the patch initially > proposed by Cong, you can find it here, I plan to submit it next week, > after all the regression tests. > > https://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux.git/commit/?h=topic/csum-fix > > It would be n

Re: [PATCH] PCI: Add no-D3 quirk for Mellanox ConnectX-[45]

2019-01-05 Thread Jason Gunthorpe
On Fri, Jan 04, 2019 at 02:44:01PM +1100, David Gibson wrote: > On Thu, Dec 06, 2018 at 08:45:09AM +0200, Leon Romanovsky wrote: > > On Thu, Dec 06, 2018 at 03:19:51PM +1100, David Gibson wrote: > > > Mellanox ConnectX-5 IB cards (MT27800) seem to cause a call trace when > > > unbound from their re

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Andrew Lunn
On Sat, Jan 05, 2019 at 02:21:03PM +0100, Heiner Kallweit wrote: > During a bug analysis we came across the fact that there's no guarantee > that reading from the ID registers returns a valid value if PHY is > powered down. When reading invalid values we may load no or the wrong > PHY driver. There

[PATCH] tipc: fix memory leak in tipc_nl_compat_publ_dump

2019-01-05 Thread Gustavo A. R. Silva
There is a memory leak in case genlmsg_put fails. Fix this by freeing *args* before return. Addresses-Coverity-ID: 1476406 ("Resource leak") Fixes: 46273cf7e009 ("tipc: fix a missing check of genlmsg_put") Signed-off-by: Gustavo A. R. Silva --- net/tipc/netlink_compat.c | 4 +++- 1 file changed

Re: [PATCH 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Larry Finger
On 1/5/19 10:30 AM, Bernd Edlinger wrote: On 1/5/19 5:13 PM, Larry Finger wrote: but this works: modprobe rtl8723ae debug_mask=0x debug_level=5 swlps=1 fwlps=0 Yes, I think that is a better thing to do now. If and when Realtek finds a firmware bug, and when the new firmware is readil

Re: [PATCH 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Bernd Edlinger
On 1/5/19 5:13 PM, Larry Finger wrote: >> but this works: >> >> modprobe rtl8723ae debug_mask=0x debug_level=5 swlps=1 fwlps=0 > > Yes, I think that is a better thing to do now. If and when Realtek finds a > firmware bug, and when the new firmware is readily available, then there will >

Re: [PATCH 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Larry Finger
On 1/5/19 5:31 AM, Bernd Edlinger wrote: On 1/5/19 3:44 AM, Larry Finger wrote: On 1/4/19 6:48 AM, Bernd Edlinger wrote: This appears to trigger a firmware bug and causes severe problems with rtl8723ae PCI devices. When the power save mode is activated for longer periods of time the firmware s

Re: [PATCH bpf-next RFCv3 2/6] veth: support AF_XDP TX copy-mode.

2019-01-05 Thread William Tu
Hi Toshiaki, Thanks a lot for the feedback. On Tue, Jan 1, 2019 at 5:44 AM Toshiaki Makita wrote: > > Hi, William. Nice work. > I have some feedback and questions. > > > + while (peer_rq->xsk_umem && budget--) { > > + unsigned int inner_xdp_xmit = 0; > > + unsigned in

Re: [PATCH 1/2 net-next] net: Don't return pfmemalloc pages to the page pool.

2019-01-05 Thread Jesper Dangaard Brouer
On Thu, 20 Dec 2018 14:21:32 -0800 Jonathan Lemon wrote: > Return pfmemalloc pages back to the page allocator, instead of holding them > in the page pool. > > Signed-off-by: Jonathan Lemon > --- > net/core/page_pool.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ne

[PATCH net] ipv6: Take rcu_read_lock in __inet6_bind for mapped addresses

2019-01-05 Thread David Ahern
From: David Ahern I realized the last patch calls dev_get_by_index_rcu in a branch not holding the rcu lock. Add the calls to rcu_read_lock and rcu_read_unlock. Fixes: ec90ad334986 ("ipv6: Consider sk_bound_dev_if when binding a socket to a v4 mapped address") Signed-off-by: David Ahern --- n

[PATCH] [v2] netfilter: ipset: fix a missing check of nla_parse

2019-01-05 Thread Aditya Pakki
When nla_parse fails, we should not use the results (the first argument). The fix checks if it fails, and if so, returns its error code upstream. Signed-off-by: Aditya Pakki --- net/netfilter/ipset/ip_set_core.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/net/net

stm32f7 / stmmac dma soft reset hangs

2019-01-05 Thread Christopher Friedt
Hi list! I've been tracing down an ethernet issue on the stm32f746-discovery board. Several additional device tree entries were required, so there may be a patch forthcoming for those in the near future, but then the ethernet mac seemed to fail on dma initialization[1] after smmac_open() is called

Re: r8152: data corruption in various scenarios

2019-01-05 Thread Mark Lord
On 2019-01-05 9:14 a.m., Mark Lord wrote: > A couple of years back, I reported data corruption resulting from > a change in kernel 3.16 which enabled hardware checksums in the r8152 driver. > This was happening on an embedded system that was using a r8152 USB dongle. > > At the time, it was very d

r8152: data corruption in various scenarios

2019-01-05 Thread Mark Lord
A couple of years back, I reported data corruption resulting from a change in kernel 3.16 which enabled hardware checksums in the r8152 driver. This was happening on an embedded system that was using a r8152 USB dongle. At the time, it was very difficult to figure out what could possibly be causin

Re: [PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Heiner Kallweit
On 05.01.2019 14:21, Heiner Kallweit wrote: > During a bug analysis we came across the fact that there's no guarantee > that reading from the ID registers returns a valid value if PHY is > powered down. When reading invalid values we may load no or the wrong > PHY driver. Therefore let's play safe

[PATCH] cdc_ether: trivial whitespace readability fix

2019-01-05 Thread Bjørn Mork
This function is unreadable enough without indenting mismatches and unnecessary line breaks. Signed-off-by: Bjørn Mork --- drivers/net/usb/cdc_ether.c | 26 +++--- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/drivers/net/usb/cdc_ether.c b/drivers/net/usb/cd

[PATCH net] net: phy: ensure PHY is powered up when reading ID registers

2019-01-05 Thread Heiner Kallweit
During a bug analysis we came across the fact that there's no guarantee that reading from the ID registers returns a valid value if PHY is powered down. When reading invalid values we may load no or the wrong PHY driver. Therefore let's play safe and power up the PHY before reading the ID registers

Re: [RFC bpf-next v3 3/9] tools: bpftool: add probes for kernel configuration options

2019-01-05 Thread Quentin Monnet
2019-01-04 09:36 UTC-0800 ~ Y Song On Fri, Jan 4, 2019 at 6:27 AM Quentin Monnet wrote: 2019-01-03 22:35 UTC-0800 ~ Y Song On Thu, Jan 3, 2019 at 9:27 AM Quentin Monnet wrote: Add probes to dump a number of options set (or not set) for compiling the kernel image. These parameters provide

[iproute2 PATCH] ipneigh: print dst for AF_BRIDGE

2019-01-05 Thread Tobias Jungel
In case a neighbour message is of family AF_BRIDE the NDA_DST attribute was not printed so far. With this patch the family is evaluated to pass the correct family to format_host_rta. Signed-off-by: Tobias Jungel --- ip/ipneigh.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-)

Re: [PATCH 1/4] rtlwifi: rtl8723ae: Take the FW LPS mode handling out

2019-01-05 Thread Bernd Edlinger
On 1/5/19 3:44 AM, Larry Finger wrote: > On 1/4/19 6:48 AM, Bernd Edlinger wrote: >> This appears to trigger a firmware bug and causes severe >> problems with rtl8723ae PCI devices. >> >> When the power save mode is activated for longer periods >> of time the firmware stops to receive any packets.

Re: [PATCH v3] can: gw: ensure DLC boundaries after CAN frame modification

2019-01-05 Thread Marc Kleine-Budde
On 1/4/19 10:41 PM, David Miller wrote: > From: Marc Kleine-Budde > Date: Fri, 4 Jan 2019 15:55:26 +0100 > >> From: Oliver Hartkopp >> >> Muyu Yu provided a POC where user root with CAP_NET_ADMIN can create a CAN >> frame modification rule that makes the data length code a higher value than >>

Re: INFO: rcu detected stall in ndisc_alloc_skb

2019-01-05 Thread Tetsuo Handa
On 2019/01/03 2:06, Tetsuo Handa wrote: > On 2018/12/31 17:24, Dmitry Vyukov wrote: Since this involves OOMs and looks like a one-off induced memory corruption: #syz dup: kernel panic: corrupted stack end in wb_workfn >>> >>> Why? >>> >>> RCU stall in this case is likely t

Re: [PATCH ipsec 7/7] xfrm: policy: fix infinite loop when merging src-nodes

2019-01-05 Thread Florian Westphal
Cong Wang wrote: > > - hlist_for_each_entry(tmp, &node->hhead, bydst) > > - tmp->bydst_reinsert = true; > > - hlist_for_each_entry(tmp, &n->hhead, bydst) > > + hlist_for_each_entry(tmp, &n->hhead, bydst

Re: [PATCH ipsec 5/7] xfrm: policy: fix reinsertion on node merge

2019-01-05 Thread Florian Westphal
Cong Wang wrote: > > - hlist_for_each_entry(tmp, &v->hhead, bydst) > > - tmp->bydst_reinsert = true; > > - hlist_for_each_entry(tmp, &n->hhead, bydst) > > + hlist_for_each_entry(tmp, &v->hhead, bydst) { > > > hlist_for_each_entry_safe()? Good question. Its not n

Re: [PATCH ipsec 4/7] xfrm: policy: delete inexact policies from inexact list on hash rebuild

2019-01-05 Thread Florian Westphal
Cong Wang wrote: > On Fri, Jan 4, 2019 at 5:19 AM Florian Westphal wrote: > > > > An xfrm hash rebuild has to reset the inexact policy list before the > > policies get re-inserted: A change of hash thresholds will result in > > policies to get moved from inexact tree to the policy hash table. > >

Re: [PATCH lora-next 3/5] net: lora: sx130x: Add PicoCell serdev driver

2019-01-05 Thread Ben Whitten
Hi, On 04/01/2019 20:21, Andreas Färber wrote: The picoGW reference MCU firmware implements a USB CDC or UART interface with a set of serial commands. It can be found on multiple mPCIe cards as well as USB adapters. https://github.com/Lora-net/picoGW_mcu That MCU design superseded earlier a

Re: WARNING in mem_cgroup_update_lru_size

2019-01-05 Thread Dmitry Vyukov
On Fri, Jan 4, 2019 at 11:58 PM syzbot wrote: > > syzbot has found a reproducer for the following crash on: > > HEAD commit:96d4f267e40f Remove 'type' argument from access_ok() funct.. > git tree: net > console output: https://syzkaller.appspot.com/x/log.txt?x=160c9a80c0 > kernel con