4.4.223-rc1 review
Subsystems (5)
Greg Kroah-Hartman
maintainerPATCH 161/312 iio:ad7797: Use correct attribute_group
In reply to Greg Kroah-HartmanFrom: YueHaibing <[email protected]>
[ Upstream commit 28535877ac5b2b84f0d394fd67a5ec71c0c48b10 ]
It should use ad7797_attribute_group in ad7797_info,
according to commit ("iio:ad7793: Add support for the ad7796 and ad7797").
Scale is fixed for the ad7796 and not programmable, hence should not have the scale_available attribute.
--- drivers/iio/adc/ad7793.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/iio/adc/ad7793.c +1 -1
@@ -579,7 +579,7 @@ static const struct iio_info ad7797_info = { .read_raw = &ad7793_read_raw, .write_raw = &ad7793_write_raw, .write_raw_get_fmt = &ad7793_write_raw_get_fmt, - .attrs = &ad7793_attribute_group, + .attrs = &ad7797_attribute_group, .validate_trigger = ad_sd_validate_trigger, .driver_module = THIS_MODULE, };
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 162/312 selftests/ipc: Fix test failure seen after initial test run
In reply to Greg Kroah-HartmanFrom: Tyler Hicks <[email protected]>
[ Upstream commit b87080eab4c1377706c113fc9c0157f19ea8fed1 ]
After successfully running the IPC msgque test once, subsequent runs result in a test failure:
$ sudo ./run_kselftest.sh TAP version 13 1..1 # selftests: ipc: msgque # Failed to get stats for IPC queue with id 0 # Failed to dump queue: -22 # Bail out! # # Pass 0 Fail 0 Xfail 0 Xpass 0 Skip 0 Error 0 not ok 1 selftests: ipc: msgque # exit=1
The dump_queue() function loops through the possible message queue index values using calls to msgctl(kern_id, MSG_STAT, ...) where kern_id represents the index value. The first time the test is ran, the initial index value of 0 is valid and the test is able to complete. The index value of 0 is not valid in subsequent test runs and the loop attempts to try index values of 1, 2, 3, and so on until a valid index value is found that corresponds to the message queue created earlier in the test.
The msgctl() syscall returns -1 and sets errno to EINVAL when invalid index values are used. The test failure is caused by incorrectly comparing errno to -EINVAL when cycling through possible index values.
Fix invalid test failures on subsequent runs of the msgque test by correctly comparing errno values to a non-negated EINVAL.
--- tools/testing/selftests/ipc/msgque.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
tools/testing/selftests/ipc/msgque.c +1 -1
@@ -135,7 +135,7 @@ int dump_queue(struct msgque_data *msgque)for (kern_id =0 ; kern_id <256 ; kern_id++) { ret =msgctl (kern_id,MSG_STAT , &ds);if (ret <0 ) { -if (errno == -EINVAL ) +if (errno ==EINVAL )continue ;printf ("Failed to get stats for IPC queue with id %d\n" , kern_id);
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 163/312 wimax/i2400m: Fix potential urb refcnt leak
In reply to Greg Kroah-HartmanFrom: Xiyu Yang <[email protected]>
[ Upstream commit 7717cbec172c3554d470023b4020d5781961187e ]
i2400mu_bus_bm_wait_for_ack() invokes usb_get_urb(), which increases the refcount of the "notif_urb".
When i2400mu_bus_bm_wait_for_ack() returns, local variable "notif_urb" becomes invalid, so the refcount should be decreased to keep refcount balanced.
The issue happens in all paths of i2400mu_bus_bm_wait_for_ack(), which forget to decrease the refcnt increased by usb_get_urb(), causing a refcnt leak.
Fix this issue by calling usb_put_urb() before the i2400mu_bus_bm_wait_for_ack() returns.
--- drivers/net/wimax/i2400m/usb-fw.c | 1 + 1 file changed, 1 insertion(+)
drivers/net/wimax/i2400m/usb-fw.c +1 -0
@@ -354,6 +354,7 @@ ssize_t i2400mu_bus_bm_wait_for_ack(struct i2400m *i2400m,usb_autopm_put_interface (i2400mu->usb_iface);d_fnend (8 , dev,"(i2400m %p ack %p size %zu) = %ld\n" , i2400m, ack, ack_size, (long ) result); +usb_put_urb (¬if_urb);return result; error_exceeded:
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 164/312 cifs: protect updating server->dstaddr with a spinlock
In reply to Greg Kroah-HartmanFrom: Ronnie Sahlberg <[email protected]>
[ Upstream commit fada37f6f62995cc449b36ebba1220594bfe55fe ]
We use a spinlock while we are reading and accessing the destination address for a server. We need to also use this spinlock to protect when we are modifying this address from reconn_set_ipaddr().
--- fs/cifs/connect.c | 2 ++ 1 file changed, 2 insertions(+)
fs/cifs/connect.c +2 -0
@@ -338,8 +338,10 @@ static int reconn_set_ipaddr(struct TCP_Server_Info *server)return rc; } +spin_lock (&cifs_tcp_ses_lock); rc =cifs_convert_address ((struct sockaddr *)&server->dstaddr, ipaddr,strlen (ipaddr)); +spin_unlock (&cifs_tcp_ses_lock);kfree (ipaddr);return !rc ?-1 :0 ;
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 165/312 scripts/config: allow colons in option strings for sed
In reply to Greg Kroah-HartmanFrom: Jeremie Francois (on alpha) <[email protected]>
[ Upstream commit e461bc9f9ab105637b86065d24b0b83f182d477c ]
Sed broke on some strings as it used colon as a separator. I made it more robust by using \001, which is legit POSIX AFAIK.
E.g. ./config --set-str CONFIG_USBNET_DEVADDR "de:ad:be:ef:00:01" failed with: sed: -e expression #1, char 55: unknown option to `s'
--- scripts/config | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
scripts/config +4 -1
@@ -6,6 +6,9 @@ myname=${0##*/} # If no prefix forced, use the default CONFIG_ CONFIG_="${CONFIG_-CONFIG_}" +# We use an uncommon delimiter for sed substitutions +SED_DELIM=$(echo -en "\001") + usage() { cat >&2 <<EOL Manipulate options in a .config file from the command line.
@@ -82,7 +85,7 @@ txt_subst() { local infile="$3" local tmpfile="$infile.swp" - sed -e "s:$before:$after:" "$infile" >"$tmpfile" + sed -e "s$SED_DELIM$before$SED_DELIM$after$SED_DELIM" "$infile" >"$tmpfile" # replace original file with the edited one mv "$tmpfile" "$infile" }
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 166/312 lib/mpi: Fix building for powerpc with clang
In reply to Greg Kroah-HartmanFrom: Nathan Chancellor <[email protected]>
[ Upstream commit 5990cdee689c6885b27c6d969a3d58b09002b0bc ]
0day reports over and over on an powerpc randconfig with clang:
lib/mpi/generic_mpih-mul1.c:37:13: error: invalid use of a cast in a inline asm context requiring an l-value: remove the cast or build with -fheinous-gnu-extensions
Remove the superfluous casts, which have been done previously for x86 and arm32 in commit dea632cadd12 ("lib/mpi: fix build with clang") and commit 7b7c1df2883d ("lib/mpi/longlong.h: fix building with 32-bit x86").
--- lib/mpi/longlong.h | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-)
lib/mpi/longlong.h +17 -17
@@ -756,22 +756,22 @@ do { \ do { \ if (__builtin_constant_p(bh) && (bh) == 0) \ __asm__ ("{a%I4|add%I4c} %1,%3,%4\n\t{aze|addze} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "%r" ((USItype)(ah)), \ "%r" ((USItype)(al)), \ "rI" ((USItype)(bl))); \ else if (__builtin_constant_p(bh) && (bh) == ~(USItype) 0) \ __asm__ ("{a%I4|add%I4c} %1,%3,%4\n\t{ame|addme} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "%r" ((USItype)(ah)), \ "%r" ((USItype)(al)), \ "rI" ((USItype)(bl))); \ else \ __asm__ ("{a%I5|add%I5c} %1,%4,%5\n\t{ae|adde} %0,%2,%3" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "%r" ((USItype)(ah)), \ "r" ((USItype)(bh)), \ "%r" ((USItype)(al)), \
@@ -781,36 +781,36 @@ do { \ do { \ if (__builtin_constant_p(ah) && (ah) == 0) \ __asm__ ("{sf%I3|subf%I3c} %1,%4,%3\n\t{sfze|subfze} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "r" ((USItype)(bh)), \ "rI" ((USItype)(al)), \ "r" ((USItype)(bl))); \ else if (__builtin_constant_p(ah) && (ah) == ~(USItype) 0) \ __asm__ ("{sf%I3|subf%I3c} %1,%4,%3\n\t{sfme|subfme} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "r" ((USItype)(bh)), \ "rI" ((USItype)(al)), \ "r" ((USItype)(bl))); \ else if (__builtin_constant_p(bh) && (bh) == 0) \ __asm__ ("{sf%I3|subf%I3c} %1,%4,%3\n\t{ame|addme} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "r" ((USItype)(ah)), \ "rI" ((USItype)(al)), \ "r" ((USItype)(bl))); \ else if (__builtin_constant_p(bh) && (bh) == ~(USItype) 0) \ __asm__ ("{sf%I3|subf%I3c} %1,%4,%3\n\t{aze|addze} %0,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "r" ((USItype)(ah)), \ "rI" ((USItype)(al)), \ "r" ((USItype)(bl))); \ else \ __asm__ ("{sf%I4|subf%I4c} %1,%5,%4\n\t{sfe|subfe} %0,%3,%2" \ - : "=r" ((USItype)(sh)), \ - "=&r" ((USItype)(sl)) \ + : "=r" (sh), \ + "=&r" (sl) \ : "r" ((USItype)(ah)), \ "r" ((USItype)(bh)), \ "rI" ((USItype)(al)), \
@@ -821,7 +821,7 @@ do { \ do { \ USItype __m0 = (m0), __m1 = (m1); \ __asm__ ("mulhwu %0,%1,%2" \ - : "=r" ((USItype) ph) \ + : "=r" (ph) \ : "%r" (__m0), \ "r" (__m1)); \ (pl) = __m0 * __m1; \
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 167/312 net: bcmgenet: suppress warnings on failed Rx SKB allocations
In reply to Greg Kroah-HartmanFrom: Doug Berger <[email protected]>
[ Upstream commit ecaeceb8a8a145d93c7e136f170238229165348f ]
The driver is designed to drop Rx packets and reclaim the buffers when an allocation fails, and the network interface needs to safely handle this packet loss. Therefore, an allocation failure of Rx SKBs is relatively benign.
However, the output of the warning message occurs with a high scheduling priority that can cause excessive jitter/latency for other high priority processing.
This commit suppresses the warning messages to prevent scheduling problems while retaining the failure count in the statistics of the network interface.
--- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/ethernet/broadcom/genet/bcmgenet.c +2 -1
@@ -1575,7 +1575,8 @@ static struct sk_buff *bcmgenet_rx_refill(struct bcmgenet_priv *priv,dma_addr_t mapping;/* Allocate a new Rx skb */ - skb =netdev_alloc_skb (priv->dev, priv->rx_buf_len +SKB_ALIGNMENT ); + skb =__netdev_alloc_skb (priv->dev, priv->rx_buf_len +SKB_ALIGNMENT , +GFP_ATOMIC | __GFP_NOWARN);if (!skb) { priv->mib.alloc_rx_buff_failed++; netif_err(priv, rx_err, priv->dev,
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 168/312 net: systemport: suppress warnings on failed Rx SKB allocations
In reply to Greg Kroah-HartmanFrom: Doug Berger <[email protected]>
[ Upstream commit 3554e54a46125030c534820c297ed7f6c3907e24 ]
The driver is designed to drop Rx packets and reclaim the buffers when an allocation fails, and the network interface needs to safely handle this packet loss. Therefore, an allocation failure of Rx SKBs is relatively benign.
However, the output of the warning message occurs with a high scheduling priority that can cause excessive jitter/latency for other high priority processing.
This commit suppresses the warning messages to prevent scheduling problems while retaining the failure count in the statistics of the network interface.
--- drivers/net/ethernet/broadcom/bcmsysport.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/ethernet/broadcom/bcmsysport.c +2 -1
@@ -526,7 +526,8 @@ static struct sk_buff *bcm_sysport_rx_refill(struct bcm_sysport_priv *priv,dma_addr_t mapping;/* Allocate a new SKB for a new packet */ - skb =netdev_alloc_skb (priv->netdev,RX_BUF_LENGTH ); + skb =__netdev_alloc_skb (priv->netdev,RX_BUF_LENGTH , +GFP_ATOMIC | __GFP_NOWARN);if (!skb) { priv->mib.alloc_rx_buff_failed++;netif_err (priv, rx_err, ndev,"SKB alloc failed\n" );
Signature · 1 line
2.20.1
Greg Kroah-Hartman
maintainerPATCH 169/312 rc: allow rc modules to be loaded if rc-main is not a module
In reply to Greg Kroah-HartmanFrom: Russell King <[email protected]>
commit 2ff56fadd94cdaeeaeccbc0a9b703a0101ada128 upstream.
rc-main mistakenly uses #ifdef MODULE to determine whether it should load the rc keymap modules. This symbol is only defined if rc-main is being built as a module itself, and bears no relation to whether the rc keymaps are modules.
Fix this to use CONFIG_MODULES instead.
--- drivers/media/rc/rc-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/media/rc/rc-main.c +1 -1
@@ -61,7 +61,7 @@ struct rc_map *rc_map_get(const char *nastruct rc_map_list *map; map =seek_rc_map (name); -#ifdef MODULE +#ifdef CONFIG_MODULES if (!map) {int rc =request_module ("%s" , name);if (rc <0 ) {
Greg Kroah-Hartman
maintainerPATCH 170/312 lirc_imon: do not leave imon_probe() with mutex held
In reply to Greg Kroah-HartmanFrom: Alexey Khoroshilov <[email protected]>
commit b833d0df943d70682e288c38c96b8e7bfff4023a upstream.
Commit af8a819a2513 ("[media] lirc_imon: simplify error handling code") lost mutex_unlock(&context->ctx_lock), so imon_probe() exits with the context->ctx_lock mutex acquired.
The patch adds mutex_unlock(&context->ctx_lock) back.
Found by Linux Driver Verification project (linuxtesting.org).
--- drivers/staging/media/lirc/lirc_imon.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/staging/media/lirc/lirc_imon.c +2 -0
@@ -885,12 +885,14 @@ static int imon_probe(struct usb_interfa vendor, product, ifnum, usbdev->bus->busnum, usbdev->devnum);/* Everything went fine. Just unlock and return retval (with is 0) */ +mutex_unlock (&context->ctx_lock); goto driver_unlock; unregister_lirc:lirc_unregister_driver (driver->minor); free_tx_urb: +mutex_unlock (&context->ctx_lock);usb_free_urb (tx_urb); free_rx_urb:
Greg Kroah-Hartman
maintainerPATCH 171/312 am437x-vpfe: fix an uninitialized variable bug
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit e4bccada44c177cde31b9a236b7dfd7f76d403ed upstream.
If we are doing V4L2_FIELD_NONE then "ret" is used uninitialized.
--- drivers/media/platform/am437x/am437x-vpfe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/media/platform/am437x/am437x-vpfe.c +1 -1
@@ -1047,7 +1047,7 @@ static int vpfe_get_ccdc_image_format(ststatic int vpfe_config_ccdc_image_format (struct vpfe_device *vpfe) {enum ccdc_frmfmt frm_fmt =CCDC_FRMFMT_INTERLACED ; -int ret; +int ret =0 ;vpfe_dbg (2 , vpfe,"vpfe_config_ccdc_image_format\n" );
Greg Kroah-Hartman
maintainerPATCH 172/312 cx23885: uninitialized variable in cx23885_av_work_handler()
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 60587bd0680507f48ae3a7360983228fd207de8a upstream.
The "handled" variable could be uninitialized if the interrupt_service_routine() call back hasn't been implimented or if it has been implemented but doesn't initialize "handled" to zero at the start. For example, adv76xx_isr() only sets "handled" to true.
--- drivers/media/pci/cx23885/cx23885-av.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/media/pci/cx23885/cx23885-av.c +1 -1
@@ -24,7 +24,7 @@ void cx23885_av_work_handler(struct work {struct cx23885_dev *dev = container_of(work, struct cx23885_dev, cx25840_work); -bool handled; +bool handled = false;v4l2_subdev_call (dev->sd_cx25840, core, interrupt_service_routine,PCI_MSK_AV_CORE , &handled);
Greg Kroah-Hartman
maintainerPATCH 173/312 ath9k_htc: check for underflow in ath9k_htc_rx_msg()
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 3a318426e09a9c9266fe6440842e11238f640a20 upstream.
We check for overflow here, but we don't check for underflow so it causes a static checker warning.
--- drivers/net/wireless/ath/ath9k/htc_hst.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/ath/ath9k/htc_hst.c +1 -1
@@ -414,7 +414,7 @@ void ath9k_htc_rx_msg(struct htc_targetreturn ; } -if (epid >=ENDPOINT_MAX ) { +if (epid <0 || epid >=ENDPOINT_MAX ) {if (pipe_id !=USB_REG_IN_PIPE )dev_kfree_skb_any (skb);else
Greg Kroah-Hartman
maintainerPATCH 174/312 VFIO: platform: reset: fix a warning message condition
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 967628827f404b3063016c138ccc7b06c54350f8 upstream.
This loop ends with count set to -1 and not zero so the warning message isn't printed when it should be. I've fixed this by change the postop to a preop.
--- drivers/vfio/platform/reset/vfio_platform_amdxgbe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/vfio/platform/reset/vfio_platform_amdxgbe.c +1 -1
@@ -110,7 +110,7 @@ int vfio_platform_amdxgbe_reset(struct vusleep_range (10 ,15 ); count =2000 ; -while (count-- && (ioread32 (xgmac_regs->ioaddr +DMA_MR ) &1 )) +while (--count && (ioread32 (xgmac_regs->ioaddr +DMA_MR ) &1 ))usleep_range (500 ,600 );if (!count)
Greg Kroah-Hartman
maintainerPATCH 175/312 net: moxa: fix an error code
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 1d3cd1773fddfdc9ffb0c2dec9a954c7a54bc207 upstream.
We accidentally return IS_ERR(priv->base) which is 1 instead of PTR_ERR(priv->base) which is the error code.
--- drivers/net/ethernet/moxa/moxart_ether.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/moxa/moxart_ether.c +2 -2
@@ -460,9 +460,9 @@ static int moxart_mac_probe(struct platf res =platform_get_resource (pdev,IORESOURCE_MEM ,0 ); ndev->base_addr = res->start; priv->base =devm_ioremap_resource (p_dev, res); - ret =IS_ERR (priv->base); -if (ret) { +if (IS_ERR (priv->base)) {dev_err (p_dev,"devm_ioremap_resource failed\n" ); + ret =PTR_ERR (priv->base); goto init_fail; }
Greg Kroah-Hartman
maintainerPATCH 176/312 mfd: lp8788-irq: Uninitialized variable in irq handler
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 22aab38e7b59fd79ce1045006be69a9abab58e5a upstream.
Instead to being true/false, the "handled" is true/uninitialized. Presumably this doesn't cause that many problems in real life because normally we handle the IRQ.
--- drivers/mfd/lp8788-irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/mfd/lp8788-irq.c +1 -1
@@ -112,7 +112,7 @@ static irqreturn_t lp8788_irq_handler(instruct lp8788_irq_data *irqd = ptr;struct lp8788 *lp = irqd->lp;u8 status[NUM_REGS ], addr, mask; -bool handled; +bool handled = false;int i;if (lp8788_read_multi_bytes (lp,LP8788_INT_1 , status,NUM_REGS ))
Greg Kroah-Hartman
maintainerPATCH 177/312 ethernet: micrel: fix some error codes
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 3af0d554c1ce11e9d0953381ff566271f9ab81a9 upstream.
There were two issues here:
1) dma_mapping_error() return true/false but we want to return -ENOMEM
2) If dmaengine_prep_slave_sg() failed then "err" wasn't set but presumably that should be -ENOMEM as well.
I changed the success path to "return 0;" instead of "return ret;" for clarity.
--- drivers/net/ethernet/micrel/ks8842.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
drivers/net/ethernet/micrel/ks8842.c +6 -4
@@ -561,8 +561,8 @@ static int __ks8842_start_new_rx_dma(strsg_init_table (sg,1 );sg_dma_address (sg) =dma_map_single (adapter->dev, ctl->skb->data,DMA_BUFFER_SIZE ,DMA_FROM_DEVICE ); - err =dma_mapping_error (adapter->dev,sg_dma_address (sg)); -if (unlikely (err)) { +if (dma_mapping_error (adapter->dev,sg_dma_address (sg))) { + err = -ENOMEM ;sg_dma_address (sg) =0 ; goto out; }
@@ -572,8 +572,10 @@ static int __ks8842_start_new_rx_dma(str ctl->adesc =dmaengine_prep_slave_sg (ctl->chan, sg,1 ,DMA_DEV_TO_MEM ,DMA_PREP_INTERRUPT ); -if (!ctl->adesc) +if (!ctl->adesc) { + err = -ENOMEM ; goto out; + } ctl->adesc->callback_param = netdev; ctl->adesc->callback = ks8842_dma_rx_cb;
@@ -584,7 +586,7 @@ static int __ks8842_start_new_rx_dma(str goto out; } -return err; +return 0 ; out:if (sg_dma_address (sg)) dma_unmap_single(adapter->dev, sg_dma_address(sg),
Greg Kroah-Hartman
maintainerPATCH 178/312 power: ipaq-micro-battery: freeing the wrong variable
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit b9223da41794030a5dfd5106c34ed1b98255e2ae upstream.
We accidentally free "micro_ac_power" which is an error pointer and it leads to an oops. We intended to free "micro_batt_power".
--- drivers/power/ipaq_micro_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/power/ipaq_micro_battery.c +1 -1
@@ -261,7 +261,7 @@ static int micro_batt_probe(struct platfreturn 0 ; ac_err: -power_supply_unregister (micro_ac_power); +power_supply_unregister (micro_batt_power); batt_err:cancel_delayed_work_sync (&mb->update);destroy_workqueue (mb->wq);
Greg Kroah-Hartman
maintainerPATCH 179/312 i40e: fix an uninitialized variable bug
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 1c306f7f62a38ee5f05f0ee994dfe82d654cf47c upstream.
We removed this initialization but it is required. Let's put it back.
--- drivers/net/ethernet/intel/i40e/i40e_hmc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/intel/i40e/i40e_hmc.c +1 -1
@@ -49,7 +49,7 @@ i40e_status i40e_add_sd_table_entry(strustruct i40e_hmc_sd_entry *sd_entry;bool dma_mem_alloc_done = false;struct i40e_dma_mem mem; -i40e_status ret_code; +i40e_status ret_code =I40E_SUCCESS ;u64 alloc_len;if (NULL == hmc_info->sd_table.sd_entry) {
Greg Kroah-Hartman
maintainerPATCH 180/312 qede: uninitialized variable in qede_start_xmit()
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 810810ffb2f6d46365d0790bbe77698a5534393a upstream.
"data_split" was never set to false. It's just uninitialized.
--- drivers/net/ethernet/qlogic/qede/qede_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/qlogic/qede/qede_main.c +1 -1
@@ -400,7 +400,7 @@ netdev_tx_t qede_start_xmit(struct sk_buu8 xmit_type;u16 idx;u16 hlen; -bool data_split; +bool data_split = false;/* Get tx-queue context and netdev index */ txq_index =skb_get_queue_mapping (skb);
Greg Kroah-Hartman
maintainerPATCH 181/312 qlcnic: potential NULL dereference in qlcnic_83xx_get_minidump_template()
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 5f46feab87bb105d6a217d966b327fdc56696802 upstream.
If qlcnic_fw_cmd_get_minidump_temp() fails then "fw_dump->tmpl_hdr" is NULL or possibly freed. It can lead to an oops later.
--- drivers/net/ethernet/qlogic/qlcnic/qlcnic_minidump.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)
drivers/net/ethernet/qlogic/qlcnic/qlcnic_minidump.c +6 -2
@@ -1419,6 +1419,7 @@ void qlcnic_83xx_get_minidump_template(sstruct qlcnic_fw_dump *fw_dump = &ahw->fw_dump;struct pci_dev *pdev = adapter->pdev;bool extended = false; +int ret; prev_version = adapter->fw_version; current_version =qlcnic_83xx_get_fw_version (adapter);
@@ -1429,8 +1430,11 @@ void qlcnic_83xx_get_minidump_template(sif (qlcnic_83xx_md_check_extended_dump_capability (adapter)) extended = !qlcnic_83xx_extend_md_capab (adapter); -if (!qlcnic_fw_cmd_get_minidump_temp (adapter)) -dev_info (&pdev->dev,"Supports FW dump capability\n" ); + ret =qlcnic_fw_cmd_get_minidump_temp (adapter); +if (ret) +return ; + +dev_info (&pdev->dev,"Supports FW dump capability\n" );/* Once we have minidump template with extended iSCSI dump * capability, update the minidump capture mask to 0x1f as
Greg Kroah-Hartman
maintainerPATCH 182/312 qlcnic: use the correct ring in qlcnic_83xx_process_rcv_ring_diag()
In reply to Greg Kroah-HartmanFrom: Dan Carpenter <[email protected]>
commit 5b4d10f5e0369ed79434593b7cd8e85eebbe473f upstream.
There is a static checker warning here "warn: mask and shift to zero" and the code sets "ring" to zero every time. From looking at how QLCNIC_FETCH_RING_ID() is used in qlcnic_83xx_process_rcv_ring() the qlcnic_83xx_hndl() should be removed.
--- drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c +1 -1
@@ -2220,7 +2220,7 @@ void qlcnic_83xx_process_rcv_ring_diag(sif (!opcode)return ; - ring =QLCNIC_FETCH_RING_ID (qlcnic_83xx_hndl (sts_data[0 ])); + ring =QLCNIC_FETCH_RING_ID (sts_data[0 ]);qlcnic_83xx_process_rcv_diag (adapter, ring, sts_data); desc = &sds_ring->desc_head[consumer]; desc->status_desc_data[0 ] =cpu_to_le64 (STATUS_OWNER_PHANTOM );
Greg Kroah-Hartman
maintainerPATCH 183/312 target: Fix a memory leak in target_dev_lba_map_store()
In reply to Greg Kroah-HartmanFrom: Bart Van Assche <[email protected]>
commit f0a8afecb29ad0005e7e946228a0ef5422058b85 upstream.
strsep() modifies its first argument. Make the pointer passed to kfree() match the return value of kmalloc().
--- drivers/target/target_core_configfs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/target/target_core_configfs.c +3 -3
@@ -1980,14 +1980,14 @@ static ssize_t target_dev_lba_map_store(struct se_device *dev =to_device (item);struct t10_alua_lba_map *lba_map =NULL ;struct list_head lba_list; -char *map_entries, *ptr; +char *map_entries, *orig, *ptr;char state;int pg_num =-1 , pg;int ret =0 , num =0 , pg_id, alua_state;unsigned long start_lba =-1 , end_lba =-1 ;unsigned long segment_size =-1 , segment_mult =-1 ; - map_entries =kstrdup (page,GFP_KERNEL ); + orig = map_entries =kstrdup (page,GFP_KERNEL );if (!map_entries)return -ENOMEM ;
@@ -2085,7 +2085,7 @@ out: }else core_alua_set_lba_map (dev, &lba_list, segment_size, segment_mult); -kfree (map_entries); +kfree (orig);return count; }
Greg Kroah-Hartman
maintainerPATCH 184/312 memory/tegra: Add number of TLB lines for Tegra124
In reply to Greg Kroah-HartmanFrom: Vince Hsu <[email protected]>
commit e2127ae7a5e80eb53ad431c39145767391da40cd upstream.
Tegra124 was accidentally left out when the number of TLB lines was parameterized in commit 11cec15bf3fb ("iommu/tegra-smmu: Parameterize number of TLB lines"). Fortunately this doesn't cause any noticeable regressions upstream, presumably because there aren't any use-cases that exercise enough pressure on the SMMU. But it is a regression nonetheless, so let's fix it.
--- drivers/memory/tegra/tegra124.c | 1 + 1 file changed, 1 insertion(+)
drivers/memory/tegra/tegra124.c +1 -0
@@ -1007,6 +1007,7 @@ static const struct tegra_smmu_soc tegra .num_swgroups = ARRAY_SIZE(tegra124_swgroups), .supports_round_robin_arbitration = true, .supports_request_limit = true, + .num_tlb_lines = 32, .num_asids = 128, };
Greg Kroah-Hartman
maintainerPATCH 185/312 pinctrl: bcm2835: Fix memory leak in error path
In reply to Greg Kroah-HartmanFrom: Stefan Wahren <[email protected]>
commit 53653c6b91750debba6dc4503003c851257fd340 upstream.
In case of an invalid pin value bcm2835_pctl_dt_node_to_map() would leak the pull configs of already assigned pins. So avoid this by calling the free map function in error case.
--- drivers/pinctrl/bcm/pinctrl-bcm2835.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pinctrl/bcm/pinctrl-bcm2835.c +1 -1
@@ -795,7 +795,7 @@ static int bcm2835_pctl_dt_node_to_map(sreturn 0 ; out: -kfree (maps); +bcm2835_pctl_dt_free_map (pctldev, maps, num_pins * maps_per_pin);return err; }
Greg Kroah-Hartman
maintainerPATCH 186/312 be2net: Dont leak iomapped memory on removal.
In reply to Greg Kroah-HartmanFrom: Douglas Miller <[email protected]>
commit a69bf3c5b49ef488970c74e26ba0ec12f08491c2 upstream.
The adapter->pcicfg resource is either mapped via pci_iomap() or derived from adapter->db. During be_remove() this resource was ignored and so could remain mapped after remove.
Add a flag to track whether adapter->pcicfg was mapped or not, then use that flag in be_unmap_pci_bars() to unmap if required.
--- drivers/net/ethernet/emulex/benet/be.h | 1 + drivers/net/ethernet/emulex/benet/be_main.c | 4 ++++ 2 files changed, 5 insertions(+)
drivers/net/ethernet/emulex/benet/be.h +1 -0
@@ -531,6 +531,7 @@ struct be_adapter {struct delayed_work be_err_detection_work;u8 err_flags; +bool pcicfg_mapped;/* pcicfg obtained via pci_iomap() */ u32 flags;u32 cmd_privileges;/* Ethtool knobs and info */
drivers/net/ethernet/emulex/benet/be_main.c +4 -0
@@ -5526,6 +5526,8 @@ static void be_unmap_pci_bars(struct be_pci_iounmap (adapter->pdev, adapter->csr);if (adapter->db)pci_iounmap (adapter->pdev, adapter->db); +if (adapter->pcicfg && adapter->pcicfg_mapped) +pci_iounmap (adapter->pdev, adapter->pcicfg); }static int db_bar (struct be_adapter *adapter)
@@ -5577,8 +5579,10 @@ static int be_map_pci_bars(struct be_adaif (!addr) goto pci_map_err; adapter->pcicfg = addr; + adapter->pcicfg_mapped = true; }else { adapter->pcicfg = adapter->db +SRIOV_VF_PCICFG_OFFSET ; + adapter->pcicfg_mapped = false; } }
Greg Kroah-Hartman
maintainerPATCH 187/312 ipv4: Fix memory leak in exception case for splitting tries
In reply to Greg Kroah-HartmanFrom: Alexander Duyck <[email protected]>
commit 3114cdfe66c156345b0ae34e2990472f277e0c1b upstream.
Fix a small memory leak that can occur where we leak a fib_alias in the event of us not being able to insert it into the local table.
--- net/ipv4/fib_trie.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
net/ipv4/fib_trie.c +3 -1
@@ -1714,8 +1714,10 @@ struct fib_table *fib_trie_unmerge(struc local_l =fib_find_node (lt, &local_tp, l->key);if (fib_insert_alias (lt, local_tp, local_l, new_fa, -NULL , l->key)) +NULL , l->key)) { +kmem_cache_free (fn_alias_kmem, new_fa); goto out; + } }/* stop loop if key wrapped back to 0 */
Greg Kroah-Hartman
maintainerPATCH 188/312 flow_dissector: Check for IP fragmentation even if not using IPv4 address
In reply to Greg Kroah-HartmanFrom: Alexander Duyck <[email protected]>
commit 918c023f29ab2dd8c63cfcc6a1239ee15933871a upstream.
This patch corrects the logic for the IPv4 parsing so that it is consistent with how we handle IPv6. Specifically if we do not have the flow key indicating we want the addresses we still may need to take a look at the IP fragmentation bits and to see if we should stop after we have recognized the L3 header.
--- net/core/flow_dissector.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-)
net/core/flow_dissector.c +9 -8
@@ -178,15 +178,16 @@ ip: ip_proto = iph->protocol; -if (!dissector_uses_key (flow_dissector, -FLOW_DISSECTOR_KEY_IPV4_ADDRS )) -break ; +if (dissector_uses_key (flow_dissector, +FLOW_DISSECTOR_KEY_IPV4_ADDRS )) { + key_addrs =skb_flow_dissector_target (flow_dissector, +FLOW_DISSECTOR_KEY_IPV4_ADDRS , + target_container); - key_addrs =skb_flow_dissector_target (flow_dissector, -FLOW_DISSECTOR_KEY_IPV4_ADDRS , target_container); -memcpy (&key_addrs->v4addrs, &iph->saddr, -sizeof (key_addrs->v4addrs)); - key_control->addr_type =FLOW_DISSECTOR_KEY_IPV4_ADDRS ; +memcpy (&key_addrs->v4addrs, &iph->saddr, +sizeof (key_addrs->v4addrs)); + key_control->addr_type =FLOW_DISSECTOR_KEY_IPV4_ADDRS ; + }if (ip_is_fragment (iph)) { key_control->flags |=FLOW_DIS_IS_FRAGMENT ;
Greg Kroah-Hartman
maintainerPATCH 189/312 ipv4: fix checksum annotation in udp4_csum_init
In reply to Greg Kroah-HartmanFrom: Hannes Frederic Sowa <[email protected]>
commit b46d9f625b07f843c706c2c7d0210a90ccdf143b upstream.
--- net/ipv4/udp.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
net/ipv4/udp.c +5 -2
@@ -1754,8 +1754,11 @@ static inline int udp4_csum_init(struct } } -return skb_checksum_init_zero_check (skb, proto, uh->check, - inet_compute_pseudo); +/* Note, we are only interested in != 0 or == 0, thus the +* force to int. +*/ +return (__force int )skb_checksum_init_zero_check (skb, proto, uh->check, + inet_compute_pseudo); }/*
Greg Kroah-Hartman
maintainerPATCH 190/312 ipv4: do not abuse GFP_ATOMIC in inet_netconf_notify_devconf()
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit fa17806cde76fb1087532f07e72aa757a30e0500 upstream.
inet_forward_change() runs with RTNL held.
We are allowed to sleep if required.
If we use __in_dev_get_rtnl() instead of __in_dev_get_rcu(), we no longer have to use GFP_ATOMIC allocations in inet_netconf_notify_devconf(), meaning we are less likely to miss notifications under memory pressure, and wont touch precious memory reserves either and risk dropping incoming packets.
inet_netconf_get_devconf() can also use GFP_KERNEL allocation.
--- net/ipv4/devinet.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)
net/ipv4/devinet.c +6 -6
@@ -1814,7 +1814,7 @@ void inet_netconf_notify_devconf(structstruct sk_buff *skb;int err = -ENOBUFS ; - skb =nlmsg_new (inet_netconf_msgsize_devconf (type),GFP_ATOMIC ); + skb =nlmsg_new (inet_netconf_msgsize_devconf (type),GFP_KERNEL );if (!skb) goto errout;
@@ -1826,7 +1826,7 @@ void inet_netconf_notify_devconf(structkfree_skb (skb); goto errout; } -rtnl_notify (skb, net,0 ,RTNLGRP_IPV4_NETCONF ,NULL ,GFP_ATOMIC ); +rtnl_notify (skb, net,0 ,RTNLGRP_IPV4_NETCONF ,NULL ,GFP_KERNEL );return ; errout:if (err <0 )
@@ -1883,7 +1883,7 @@ static int inet_netconf_get_devconf(stru } err = -ENOBUFS ; - skb =nlmsg_new (inet_netconf_msgsize_devconf (-1 ),GFP_ATOMIC ); + skb =nlmsg_new (inet_netconf_msgsize_devconf (-1 ),GFP_KERNEL );if (!skb) goto errout;
@@ -2007,16 +2007,16 @@ static void inet_forward_change(struct nfor_each_netdev (net, dev) {struct in_device *in_dev; +if (on)dev_disable_lro (dev); -rcu_read_lock (); - in_dev =__in_dev_get_rcu (dev); + + in_dev =__in_dev_get_rtnl (dev);if (in_dev) {IN_DEV_CONF_SET (in_dev,FORWARDING , on);inet_netconf_notify_devconf (net,NETCONFA_FORWARDING , dev->ifindex, &in_dev->cnf); } -rcu_read_unlock (); } }
Greg Kroah-Hartman
maintainerPATCH 191/312 ipv4: accept u8 in IP_TOS ancillary data
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit e895cdce683161081e3626c4f5a5c55cb72089f8 upstream.
In commit f02db315b8d8 ("ipv4: IP_TOS and IP_TTL can be specified as ancillary data") Francesco added IP_TOS values specified as integer.
However, kernel sends to userspace (at recvmsg() time) an IP_TOS value in a single byte, when IP_RECVTOS is set on the socket.
It can be very useful to reflect all ancillary options as given by the kernel in a subsequent sendmsg(), instead of aborting the sendmsg() with EINVAL after Francesco patch.
So this patch extends IP_TOS ancillary to accept an u8, so that an UDP server can simply reuse same ancillary block without having to mangle it.
Jesper can then augment
https://github.com/netoptimizer/network-testing/blob/master/src/udp_example02.c
to add TOS reflection ;)
--- net/ipv4/ip_sockglue.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
net/ipv4/ip_sockglue.c +5 -2
@@ -279,9 +279,12 @@ int ip_cmsg_send(struct net *net, struct ipc->ttl = val;break ;case IP_TOS : -if (cmsg->cmsg_len !=CMSG_LEN (sizeof (int ))) +if (cmsg->cmsg_len ==CMSG_LEN (sizeof (int ))) + val = *(int *)CMSG_DATA (cmsg); +else if (cmsg->cmsg_len ==CMSG_LEN (sizeof (u8))) + val = *(u8 *)CMSG_DATA (cmsg); +else return -EINVAL ; - val = *(int *)CMSG_DATA (cmsg);if (val <0 || val >255 )return -EINVAL ; ipc->tos = val;
Greg Kroah-Hartman
maintainerPATCH 192/312 net: vrf: Fix dev refcnt leak due to IPv6 prefix route
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit 4f7f34eaab9f68c9bcd45386b15c414c38b40587 upstream.
ifupdown2 found a kernel bug with IPv6 routes and movement from the main table to the VRF table. Sequence of events:
Create the interface and add addresses:
ip link add dev eth4.105 link eth4 type vlan id 105
ip addr add dev eth4.105 8.105.105.10/24
ip -6 addr add dev eth4.105 2008:105:105::10/64At this point IPv6 has inserted a prefix route in the main table even though the interface is 'down'. From there the VRF device is created:
ip link add dev vrf105 type vrf table 105
ip addr add dev vrf105 9.9.105.10/32
ip -6 addr add dev vrf105 2000:9:105::10/128
ip link set vrf105 up
Then the interface is enslaved, while still in the 'down' state:
ip link set dev eth4.105 master vrf105
Since the device is down the VRF driver cycling the device does not send the NETDEV_UP and NETDEV_DOWN but rather the NETDEV_CHANGE event which does not flush the routes inserted prior.
When the link is brought up
ip link set dev eth4.105 up
the prefix route is added in the VRF table, but does not remove the route from the main table.
Fix by handling the NETDEV_CHANGEUPPER event similar what was implemented for IPv4 in 7f49e7a38b77 ("net: Flush local routes when device changes vrf association")
--- net/ipv6/addrconf.c | 10 ++++++++++ 1 file changed, 10 insertions(+)
net/ipv6/addrconf.c +10 -0
@@ -3146,6 +3146,7 @@ static int addrconf_notify(struct notifi void *ptr) {struct net_device *dev =netdev_notifier_info_to_dev (ptr); +struct netdev_notifier_changeupper_info *info;struct inet6_dev *idev =__in6_dev_get (dev);struct net *net =dev_net (dev);int run_pending =0 ;
@@ -3307,6 +3308,15 @@ static int addrconf_notify(struct notificase NETDEV_POST_TYPE_CHANGE :addrconf_type_change (dev, event);break ; + +case NETDEV_CHANGEUPPER : + info = ptr; + +/* flush all routes if dev is linked to or unlinked from +* an L3 master device (e.g., VRF) +*/ +if (info->upper_dev &&netif_is_l3_master (info->upper_dev)) +addrconf_ifdown (dev,0 ); }return NOTIFY_OK ;
Greg Kroah-Hartman
maintainerPATCH 193/312 ipv6: fix checksum annotation in udp6_csum_init
In reply to Greg Kroah-HartmanFrom: Hannes Frederic Sowa <[email protected]>
commit c148d16369ff0095eca950d17968ba1d56a47b53 upstream.
--- net/ipv6/ip6_checksum.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
net/ipv6/ip6_checksum.c +5 -2
@@ -84,9 +84,12 @@ int udp6_csum_init(struct sk_buff *skb,* we accept a checksum of zero here. When we find the socket * for the UDP packet we'll check if that socket allows zero checksum * for IPv6 (set by socket option). +* +* Note, we are only interested in != 0 or == 0, thus the +* force to int. */ -return skb_checksum_init_zero_check (skb, proto, uh->check, - ip6_compute_pseudo); +return (__force int )skb_checksum_init_zero_check (skb, proto, uh->check, + ip6_compute_pseudo); }EXPORT_SYMBOL (udp6_csum_init);
Greg Kroah-Hartman
maintainerPATCH 194/312 ipv6: do not abuse GFP_ATOMIC in inet6_netconf_notify_devconf()
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit 927265bc6cd6374c9bafc43408ece4e92311b149 upstream.
All inet6_netconf_notify_devconf() callers are in process context, so we can use GFP_KERNEL allocations if we take care of not holding a rwlock while not needed in ip6mr (we hold RTNL there)
--- net/ipv6/addrconf.c | 4 ++-- net/ipv6/ip6mr.c | 13 +++++++------ 2 files changed, 9 insertions(+), 8 deletions(-)
net/ipv6/addrconf.c +2 -2
@@ -540,7 +540,7 @@ void inet6_netconf_notify_devconf(structstruct sk_buff *skb;int err = -ENOBUFS ; - skb =nlmsg_new (inet6_netconf_msgsize_devconf (type),GFP_ATOMIC ); + skb =nlmsg_new (inet6_netconf_msgsize_devconf (type),GFP_KERNEL );if (!skb) goto errout;
@@ -552,7 +552,7 @@ void inet6_netconf_notify_devconf(structkfree_skb (skb); goto errout; } -rtnl_notify (skb, net,0 ,RTNLGRP_IPV6_NETCONF ,NULL ,GFP_ATOMIC ); +rtnl_notify (skb, net,0 ,RTNLGRP_IPV6_NETCONF ,NULL ,GFP_KERNEL );return ; errout:rtnl_set_sk_err (net,RTNLGRP_IPV6_NETCONF , err);
net/ipv6/ip6mr.c +7 -6
@@ -1594,14 +1594,15 @@ static int ip6mr_sk_init(struct mr6_tablif (likely (mrt->mroute6_sk ==NULL )) { mrt->mroute6_sk = sk; net->ipv6.devconf_all->mc_forwarding++; -inet6_netconf_notify_devconf (net,NETCONFA_MC_FORWARDING , -NETCONFA_IFINDEX_ALL , - net->ipv6.devconf_all); - } -else + }else { err = -EADDRINUSE ; + }write_unlock_bh (&mrt_lock); +if (!err) +inet6_netconf_notify_devconf (net,NETCONFA_MC_FORWARDING , +NETCONFA_IFINDEX_ALL , + net->ipv6.devconf_all);rtnl_unlock ();return err;
@@ -1619,11 +1620,11 @@ int ip6mr_sk_done(struct sock *sk)write_lock_bh (&mrt_lock); mrt->mroute6_sk =NULL ; net->ipv6.devconf_all->mc_forwarding--; +write_unlock_bh (&mrt_lock);inet6_netconf_notify_devconf (net,NETCONFA_MC_FORWARDING ,NETCONFA_IFINDEX_ALL , net->ipv6.devconf_all); -write_unlock_bh (&mrt_lock);mroute_clean_tables (mrt, false); err =0 ;
Greg Kroah-Hartman
maintainerPATCH 195/312 ipv6: add missing netconf notif when all is updated
In reply to Greg Kroah-HartmanFrom: Nicolas Dichtel <[email protected]>
commit d26c638c16cb54f6fb1507e27df93ede692db572 upstream.
The 'default' value was not advertised.
--- net/ipv6/addrconf.c | 7 +++++++ 1 file changed, 7 insertions(+)
net/ipv6/addrconf.c +7 -0
@@ -771,7 +771,14 @@ static int addrconf_fixup_forwarding(str }if (p == &net->ipv6.devconf_all->forwarding) { +int old_dflt = net->ipv6.devconf_dflt->forwarding; + net->ipv6.devconf_dflt->forwarding = newf; +if ((!newf) ^ (!old_dflt)) +inet6_netconf_notify_devconf (net,NETCONFA_FORWARDING , +NETCONFA_IFINDEX_DEFAULT , + net->ipv6.devconf_dflt); +addrconf_forward_change (net, newf);if ((!newf) ^ (!old)) inet6_netconf_notify_devconf(net, NETCONFA_FORWARDING,
Greg Kroah-Hartman
maintainerPATCH 196/312 net: ipv6: Fix processing of RAs in presence of VRF
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit 830218c1add1da16519b71909e5cf21522b7d062 upstream.
rt6_add_route_info and rt6_add_dflt_router were updated to pull the FIB table from the device index, but the corresponding rt6_get_route_info and rt6_get_dflt_router functions were not leading to the failure to process RA's:
ICMPv6: RA: ndisc_router_discovery failed to add default route
Fix the 'get' functions by using the table id associated with the device when applicable.
Also, now that default routes can be added to tables other than the default table, rt6_purge_dflt_routers needs to be updated as well to look at all tables. To handle that efficiently, add a flag to the table denoting if it is has a default route via RA.
--- include/net/ip6_fib.h | 2 + net/ipv6/route.c | 68 +++++++++++++++++++++++++++++++++++--------------- 2 files changed, 50 insertions(+), 20 deletions(-)
include/net/ip6_fib.h +2 -0
@@ -258,6 +258,8 @@ struct fib6_table {rwlock_t tb6_lock;struct fib6_node tb6_root;struct inet_peer_base tb6_peers; +unsigned int flags; +#define RT6_TABLE_HAS_DFLT_ROUTER BIT(0) };#define RT6_TABLE_UNSPEC RT_TABLE_UNSPEC
net/ipv6/route.c +48 -20
148 lines not on this page · raw message
Greg Kroah-Hartman
maintainerPATCH 197/312 netfilter: nf_tables: fix a wrong check to skip the inactive rules
In reply to Greg Kroah-HartmanFrom: Liping Zhang <[email protected]>
commit 8fff1722f705ce5023a0d6d77a31a9d013be2a34 upstream.
nft_genmask_cur has already done left-shift operator on the gencursor, so there's no need to do left-shift operator on it again.
--- net/netfilter/nf_tables_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/netfilter/nf_tables_core.c +1 -1
@@ -130,7 +130,7 @@ next_rule:list_for_each_entry_continue_rcu (rule, &chain->rules, list) {/* This rule is not active, skip. */ -if (unlikely (rule->genmask & (1 << gencursor))) +if (unlikely (rule->genmask & gencursor))continue ; rulenum++;
Greg Kroah-Hartman
maintainerPATCH 198/312 netfilter: nft_dynset: fix panic if NFT_SET_HASH is not enabled
In reply to Greg Kroah-HartmanFrom: Liping Zhang <[email protected]>
commit bb6a6e8e091353770074608c1d1bfde0e20b8154 upstream.
When CONFIG_NFT_SET_HASH is not enabled and I input the following rule:
"nft add rule filter output flow table test {ip daddr counter }", kernel panic happened on my system:
BUG: unable to handle kernel NULL pointer dereference at (null) IP: [< (null)>] (null) [...] Call Trace: [<ffffffffa0590466>] ? nft_dynset_eval+0x56/0x100 [nf_tables] [<ffffffffa05851bb>] nft_do_chain+0xfb/0x4e0 [nf_tables] [<ffffffffa0432f01>] ? nf_conntrack_tuple_taken+0x61/0x210 [nf_conntrack] [<ffffffffa0459ea6>] ? get_unique_tuple+0x136/0x560 [nf_nat] [<ffffffffa043bca1>] ? __nf_ct_ext_add_length+0x111/0x130 [nf_conntrack] [<ffffffffa045a357>] ? nf_nat_setup_info+0x87/0x3b0 [nf_nat] [<ffffffff81761e27>] ? ipt_do_table+0x327/0x610 [<ffffffffa045a6d7>] ? __nf_nat_alloc_null_binding+0x57/0x80 [nf_nat] [<ffffffffa059f21f>] nft_ipv4_output+0xaf/0xd0 [nf_tables_ipv4] [<ffffffff81702515>] nf_iterate+0x55/0x60 [<ffffffff81702593>] nf_hook_slow+0x73/0xd0
Because in rbtree type set, ops->update is not implemented. So just keep it simple, in such case, report -EOPNOTSUPP to the user space.
--- net/netfilter/nft_dynset.c | 3 +++ 1 file changed, 3 insertions(+)
net/netfilter/nft_dynset.c +3 -0
@@ -121,6 +121,9 @@ static int nft_dynset_init(const structreturn PTR_ERR (set); } +if (set->ops->update ==NULL ) +return -EOPNOTSUPP ; +if (set->flags &NFT_SET_CONSTANT )return -EBUSY ;
Greg Kroah-Hartman
maintainerPATCH 199/312 netfilter: nf_tables: destroy the set if fail to add transaction
In reply to Greg Kroah-HartmanFrom: Liping Zhang <[email protected]>
commit c17c3cdff10b9f59ef1244a14604f10949f17117 upstream.
When the memory is exhausted, then we will fail to add the NFT_MSG_NEWSET transaction. In such case, we should destroy the set before we free it.
--- net/netfilter/nf_tables_api.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
net/netfilter/nf_tables_api.c +3 -1
@@ -2849,12 +2849,14 @@ static int nf_tables_newset(struct net * err =nft_trans_set_add (&ctx,NFT_MSG_NEWSET , set);if (err <0 ) - goto err2; + goto err3;list_add_tail_rcu (&set->list, &table->sets); table->use++;return 0 ; +err3: + ops->destroy (set); err2:kfree (set); err1:
Greg Kroah-Hartman
maintainerPATCH 200/312 netfilter: nft_dup: do not use sreg_dev if the user doesnt specify it
In reply to Greg Kroah-HartmanFrom: Liping Zhang <[email protected]>
commit b73b8a1ba598236296a46103d81c10d629d9a470 upstream.
The NFTA_DUP_SREG_DEV attribute is not a must option, so we should use it in routing lookup only when the user specify it.
--- net/ipv4/netfilter/nft_dup_ipv4.c | 6 ++++-- net/ipv6/netfilter/nft_dup_ipv6.c | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-)
net/ipv4/netfilter/nft_dup_ipv4.c +4 -2
@@ -28,7 +28,7 @@ static void nft_dup_ipv4_eval(const strustruct in_addr gw = { .s_addr = (__force __be32 )regs->data[priv->sreg_addr], }; -int oif = regs->data[priv->sreg_dev]; +int oif = priv->sreg_dev ? regs->data[priv->sreg_dev] :-1 ;nf_dup_ipv4 (pkt->net, pkt->skb, pkt->hook, &gw, oif); }
@@ -59,7 +59,9 @@ static int nft_dup_ipv4_dump(struct sk_b {struct nft_dup_ipv4 *priv =nft_expr_priv (expr); -if (nft_dump_register (skb,NFTA_DUP_SREG_ADDR , priv->sreg_addr) || +if (nft_dump_register (skb,NFTA_DUP_SREG_ADDR , priv->sreg_addr)) + goto nla_put_failure; +if (priv->sreg_dev &&nft_dump_register (skb,NFTA_DUP_SREG_DEV , priv->sreg_dev)) goto nla_put_failure;
net/ipv6/netfilter/nft_dup_ipv6.c +4 -2
@@ -26,7 +26,7 @@ static void nft_dup_ipv6_eval(const stru {struct nft_dup_ipv6 *priv =nft_expr_priv (expr);struct in6_addr *gw = (struct in6_addr *)®s->data[priv->sreg_addr]; -int oif = regs->data[priv->sreg_dev]; +int oif = priv->sreg_dev ? regs->data[priv->sreg_dev] :-1 ;nf_dup_ipv6 (pkt->net, pkt->skb, pkt->hook, gw, oif); }
@@ -57,7 +57,9 @@ static int nft_dup_ipv6_dump(struct sk_b {struct nft_dup_ipv6 *priv =nft_expr_priv (expr); -if (nft_dump_register (skb,NFTA_DUP_SREG_ADDR , priv->sreg_addr) || +if (nft_dump_register (skb,NFTA_DUP_SREG_ADDR , priv->sreg_addr)) + goto nla_put_failure; +if (priv->sreg_dev &&nft_dump_register (skb,NFTA_DUP_SREG_DEV , priv->sreg_dev)) goto nla_put_failure;
Greg Kroah-Hartman
maintainerPATCH 201/312 udp: restore UDPlite many-cast delivery
In reply to Greg Kroah-HartmanFrom: Pablo Neira <[email protected]>
commit 73e2d5e34b6cdd1080038daf3d6d6d744a9eefe6 upstream.
Honor udptable parameter that is passed to __udp*_lib_mcast_deliver(), otherwise udplite broadcast/multicast use the wrong table and it breaks.
--- net/ipv4/udp.c | 6 +++--- net/ipv6/udp.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-)
net/ipv4/udp.c +3 -3
@@ -1685,10 +1685,10 @@ static int __udp4_lib_mcast_deliver(struif (use_hash2) { hash2_any =udp4_portaddr_hash (net,htonl (INADDR_ANY ), hnum) & - udp_table.mask; - hash2 =udp4_portaddr_hash (net, daddr, hnum) & udp_table.mask; + udptable->mask; + hash2 =udp4_portaddr_hash (net, daddr, hnum) & udptable->mask; start_lookup: - hslot = &udp_table.hash2[hash2]; + hslot = &udptable->hash2[hash2]; offset = offsetof(typeof(*sk), __sk_common.skc_portaddr_node); }
net/ipv6/udp.c +3 -3
@@ -801,10 +801,10 @@ static int __udp6_lib_mcast_deliver(struif (use_hash2) { hash2_any =udp6_portaddr_hash (net, &in6addr_any, hnum) & - udp_table.mask; - hash2 =udp6_portaddr_hash (net, daddr, hnum) & udp_table.mask; + udptable->mask; + hash2 =udp6_portaddr_hash (net, daddr, hnum) & udptable->mask; start_lookup: - hslot = &udp_table.hash2[hash2]; + hslot = &udptable->hash2[hash2]; offset = offsetof(typeof(*sk), __sk_common.skc_portaddr_node); }
Greg Kroah-Hartman
maintainerPATCH 202/312 clk: st: avoid uninitialized variable use
In reply to Greg Kroah-HartmanFrom: Arnd Bergmann <[email protected]>
commit 2dd52d7f6f9d9d03a82a68040ac3d221dd79af94 upstream.
quadfs_pll_fs660c32_round_rate prints a few structure members that are never initialized, and also doesn't print the only one it cares about. We get a gcc warning about the ones that are printed:
clk/st/clkgen-fsyn.c:560:93: warning: 'params.sdiv' may be used uninitialized in this function clk/st/clkgen-fsyn.c:560:93: warning: 'params.mdiv' may be used uninitialized in this function clk/st/clkgen-fsyn.c:560:93: warning: 'params.pe' may be used uninitialized in this function clk/st/clkgen-fsyn.c:560:93: warning: 'params.nsdiv' may be used uninitialized in this function
This changes the code to no longer print uninitialized data, and for good measure it also prints the ndiv member that is being set.
--- drivers/clk/st/clkgen-fsyn.c | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-)
drivers/clk/st/clkgen-fsyn.c +9 -8
@@ -549,19 +549,20 @@ static int clk_fs660c32_vco_get_params(ureturn 0 ; } -static long quadfs_pll_fs660c32_round_rate (struct clk_hw *hw,unsigned long rate - ,unsigned long *prate) +static long quadfs_pll_fs660c32_round_rate (struct clk_hw *hw, +unsigned long rate, +unsigned long *prate) {struct stm_fs params; -if (!clk_fs660c32_vco_get_params (*prate, rate, ¶ms)) -clk_fs660c32_vco_get_rate (*prate, ¶ms, &rate); +if (clk_fs660c32_vco_get_params (*prate, rate, ¶ms)) +return rate; -pr_debug ("%s: %s new rate %ld [sdiv=0x%x,md=0x%x,pe=0x%x,nsdiv3=%u]\n" , +clk_fs660c32_vco_get_rate (*prate, ¶ms, &rate); + +pr_debug ("%s: %s new rate %ld [ndiv=%u]\n" , __func__,clk_hw_get_name (hw), - rate, (unsigned int )params.sdiv, - (unsigned int )params.mdiv, - (unsigned int )params.pe, (unsigned int )params.nsdiv); + rate, (unsigned int )params.ndiv);return rate; }
Greg Kroah-Hartman
maintainerPATCH 203/312 clk: gpio: handle error codes for of_clk_get_parent_count()
In reply to Greg Kroah-HartmanFrom: Brian Norris <[email protected]>
commit 0b2e78865d92e2d70542cb1d4d7af1d4ea0a286d upstream.
We might make bad memory allocations if we get (e.g.) -ENOSYS from of_clk_get_parent_count().
Noticed by Coverity.
--- drivers/clk/clk-gpio.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
drivers/clk/clk-gpio.c +4 -2
@@ -287,12 +287,14 @@ static void __init of_gpio_clk_setup(strconst char **parent_names;int i, num_parents; + num_parents =of_clk_get_parent_count (node); +if (num_parents <0 ) +return ; + data =kzalloc (sizeof (*data),GFP_KERNEL );if (!data)return ; - num_parents =of_clk_get_parent_count (node); - parent_names =kcalloc (num_parents,sizeof (char *),GFP_KERNEL );if (!parent_names)return ;
Greg Kroah-Hartman
maintainerPATCH 204/312 clk: ti: omap3+: dpll: use non-locking version of clk_get_rate
In reply to Greg Kroah-HartmanFrom: Tero Kristo <[email protected]>
commit a0d54c3899aaeb047969d9479263c6bcf385c331 upstream.
As the code in this file is being executed within irq context in some cases, we must avoid the clk_get_rate which uses mutex internally. Switch the code to use clk_hw_get_rate instead which is non-locking.
This fixes an issue where PM runtime will hang the system if enabled with a serial console before a suspend-resume cycle.
--- drivers/clk/ti/dpll3xxx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/clk/ti/dpll3xxx.c +2 -1
@@ -437,7 +437,8 @@ int omap3_noncore_dpll_enable(struct clk parent =clk_hw_get_parent (hw); -if (clk_hw_get_rate (hw) ==clk_get_rate (dd->clk_bypass)) { +if (clk_hw_get_rate (hw) == +clk_hw_get_rate (__clk_get_hw (dd->clk_bypass))) {WARN_ON (parent !=__clk_get_hw (dd->clk_bypass)); r =_omap3_noncore_dpll_bypass (clk); }else {
Greg Kroah-Hartman
maintainerPATCH 205/312 clk: multiplier: Prevent the multiplier from under / over flowing
In reply to Greg Kroah-HartmanFrom: Maxime Ripard <[email protected]>
commit 25f77a3aa4cb948666bf8e7fd972533ea487c3bd upstream.
In the current multiplier base clock implementation, if the CLK_SET_RATE_PARENT flag isn't set, the code will not make sure that the multiplier computed remains within the boundaries of our clock.
This means that if the clock we want to reach is below the parent rate, or if the multiplier is above the maximum that we can reach, we will end up with a completely bogus one that the clock cannot achieve.
--- drivers/clk/clk-multiplier.c | 20 +++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-)
drivers/clk/clk-multiplier.c +17 -3
@@ -54,14 +54,28 @@ static unsigned long __bestmult(struct cunsigned long *best_parent_rate, u8 width, unsigned long flags) { + struct clk_multiplier *mult = to_clk_multiplier(hw);unsigned long orig_parent_rate = *best_parent_rate;unsigned long parent_rate, current_rate, best_rate = ~0 ;unsigned int i, bestmult =0 ; +unsigned int maxmult = (1 << width) -1 ; -if (!(clk_hw_get_flags (hw) &CLK_SET_RATE_PARENT )) -return rate / *best_parent_rate; +if (!(clk_hw_get_flags (hw) &CLK_SET_RATE_PARENT )) { + bestmult = rate / orig_parent_rate; -for (i =1 ; i < ((1 << width) -1 ); i++) { +/* Make sure we don't end up with a 0 multiplier */ +if ((bestmult ==0 ) && + !(mult->flags &CLK_MULTIPLIER_ZERO_BYPASS )) + bestmult =1 ; + +/* Make sure we don't overflow the multiplier */ +if (bestmult > maxmult) + bestmult = maxmult; + +return bestmult; + } + +for (i =1 ; i < maxmult; i++) {if (rate == orig_parent_rate * i) {/* * This is the best case for us if we have a
Greg Kroah-Hartman
maintainerPATCH 206/312 clk: imx: clk-pllv3: fix incorrect handle of enet powerdown bit
In reply to Greg Kroah-HartmanFrom: Dong Aisheng <[email protected]>
commit b3e76bdc0b2190e67427d31cd740debd01c03631 upstream.
After commit f53947456f98 ("ARM: clk: imx: update pllv3 to support imx7"), the former used BM_PLL_POWER bit is not correct anymore for IMX7 ENET. Instead, pll->powerdown holds the correct bit, so using powerdown bit in clk_pllv3_{prepare | unprepare} functions.
--- drivers/clk/imx/clk-pllv3.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
drivers/clk/imx/clk-pllv3.c +4 -4
@@ -76,9 +76,9 @@ static int clk_pllv3_prepare(struct clk_ val =readl_relaxed (pll->base);if (pll->powerup_set) - val |=BM_PLL_POWER ; + val |= pll->powerdown;else - val &= ~BM_PLL_POWER ; + val &= ~pll->powerdown;writel_relaxed (val, pll->base);return clk_pllv3_wait_lock (pll);
@@ -91,9 +91,9 @@ static void clk_pllv3_unprepare(struct c val =readl_relaxed (pll->base);if (pll->powerup_set) - val &= ~BM_PLL_POWER ; + val &= ~pll->powerdown;else - val |=BM_PLL_POWER ; + val |= pll->powerdown;writel_relaxed (val, pll->base); }
Greg Kroah-Hartman
maintainerPATCH 207/312 clk: xgene: Dont call __pa on ioremaped address
In reply to Greg Kroah-HartmanFrom: Laura Abbott <[email protected]>
commit 06b113e9f28f8657715919087a3f54b77d1634ed upstream.
ioremaped addresses are not linearly mapped so the physical address can not be figured out via __pa. More generally, there is no guarantee that backing value of an ioremapped address is a physical address at all. The value here is only used for debugging so just drop the call to __pa on the ioremapped address.
--- drivers/clk/clk-xgene.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-)
drivers/clk/clk-xgene.c +4 -6
@@ -218,22 +218,20 @@ static int xgene_clk_enable(struct clk_hstruct xgene_clk *pclk =to_xgene_clk (hw);unsigned long flags =0 ;u32 data; -phys_addr_t reg;if (pclk->lock)spin_lock_irqsave (pclk->lock, flags);if (pclk->param.csr_reg !=NULL ) {pr_debug ("%s clock enabled\n" ,clk_hw_get_name (hw)); - reg =__pa (pclk->param.csr_reg);/* First enable the clock */ data =xgene_clk_read (pclk->param.csr_reg + pclk->param.reg_clk_offset); data |= pclk->param.reg_clk_mask;xgene_clk_write (data, pclk->param.csr_reg + pclk->param.reg_clk_offset); -pr_debug ("%s clock PADDR base %pa clk offset 0x%08X mask 0x%08X value 0x%08X\n" , -clk_hw_get_name (hw), ®, +pr_debug ("%s clk offset 0x%08X mask 0x%08X value 0x%08X\n" , +clk_hw_get_name (hw), pclk->param.reg_clk_offset, pclk->param.reg_clk_mask, data);
@@ -243,8 +241,8 @@ static int xgene_clk_enable(struct clk_h data &= ~pclk->param.reg_csr_mask;xgene_clk_write (data, pclk->param.csr_reg + pclk->param.reg_csr_offset); -pr_debug ("%s CSR RESET PADDR base %pa csr offset 0x%08X mask 0x%08X value 0x%08X\n" , -clk_hw_get_name (hw), ®, +pr_debug ("%s csr offset 0x%08X mask 0x%08X value 0x%08X\n" , +clk_hw_get_name (hw), pclk->param.reg_csr_offset, pclk->param.reg_csr_mask, data); }
Greg Kroah-Hartman
maintainerPATCH 208/312 cls_bpf: reset class and reuse major in da
In reply to Greg Kroah-HartmanFrom: Daniel Borkmann <[email protected]>
commit 3a461da1d03e7a857edfa6a002040d07e118c639 upstream.
There are two issues with the current code. First one is that we need to set res->class to 0 in case we use non-default classid matching.
This is important for the case where cls_bpf was initially set up with an optional binding to a default class with tcf_bind_filter(), where the underlying qdisc implements bind_tcf() that fills res->class and tests for it later on when doing the classification. Convention for these cases is that after tc_classify() was called, such qdiscs (atm, drr, qfq, cbq, hfsc, htb) first test class, and if 0, then they lookup based on classid.
Second, there's a bug with da mode, where res->classid is only assigned a 16 bit minor, but it needs to expand to the full 32 bit major/minor combination instead, therefore we need to expand with the bound major. This is fine as classes belonging to a classful qdisc must share the same major.
--- net/sched/cls_bpf.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-)
net/sched/cls_bpf.c +8 -5
@@ -107,8 +107,9 @@ static int cls_bpf_classify(struct sk_bu }if (prog->exts_integrated) { - res->class = prog->res.class; - res->classid =qdisc_skb_cb (skb)->tc_classid; + res->class =0 ; + res->classid =TC_H_MAJ (prog->res.classid) | +qdisc_skb_cb (skb)->tc_classid; ret =cls_bpf_exec_opcode (filter_res);if (ret ==TC_ACT_UNSPEC )
@@ -118,10 +119,12 @@ static int cls_bpf_classify(struct sk_buif (filter_res ==0 )continue ; - - *res = prog->res; -if (filter_res !=-1 ) +if (filter_res !=-1 ) { + res->class =0 ; res->classid = filter_res; + }else { + *res = prog->res; + } ret =tcf_exts_exec (skb, &prog->exts, res);if (ret <0 )
Greg Kroah-Hartman
maintainerPATCH 209/312 arm64: bpf: jit JMP_JSET_{X,K}
In reply to Greg Kroah-HartmanFrom: Zi Shen Lim <[email protected]>
commit 98397fc547e3f4553553a30ea56fa34d613f0a4c upstream.
Original implementation commit e54bcde3d69d ("arm64: eBPF JIT compiler") had the relevant code paths, but due to an oversight always fail jiting.
As a result, we had been falling back to BPF interpreter whenever a BPF program has JMP_JSET_{X,K} instructions.
With this fix, we confirm that the corresponding tests in lib/test_bpf continue to pass, and also jited.
...
[ 2.784553] test_bpf: #30 JSET jited:1 188 192 197 PASS [ 2.791373] test_bpf: #31 tcpdump port 22 jited:1 325 677 625 PASS [ 2.808800] test_bpf: #32 tcpdump complex jited:1 323 731 991 PASS ... [ 3.190759] test_bpf: #237 JMP_JSET_K: if (0x3 & 0x2) return 1 jited:1 110 PASS [ 3.192524] test_bpf: #238 JMP_JSET_K: if (0x3 & 0xffffffff) return 1 jited:1 98 PASS [ 3.211014] test_bpf: #249 JMP_JSET_X: if (0x3 & 0x2) return 1 jited:1 120 PASS [ 3.212973] test_bpf: #250 JMP_JSET_X: if (0x3 & 0xffffffff) return 1 jited:1 89 PASS
...
--- arch/arm64/net/bpf_jit_comp.c | 1 + 1 file changed, 1 insertion(+)
arch/arm64/net/bpf_jit_comp.c +1 -0
@@ -482,6 +482,7 @@ emit_cond_jmp:case BPF_JGE : jmp_cond =A64_COND_CS ;break ; +case BPF_JSET :case BPF_JNE : jmp_cond =A64_COND_NE ;break ;
Greg Kroah-Hartman
maintainerPATCH 210/312 bpf, trace: check event type in bpf_perf_event_read
In reply to Greg Kroah-HartmanFrom: Alexei Starovoitov <[email protected]>
commit ad572d174787daa59e24b8b5c83028c09cdb5ddb upstream.
similar to bpf_perf_event_output() the bpf_perf_event_read() helper needs to check the type of the perf_event before reading the counter.
--- kernel/trace/bpf_trace.c | 4 ++++ 1 file changed, 4 insertions(+)
kernel/trace/bpf_trace.c +4 -0
@@ -206,6 +206,10 @@ static u64 bpf_perf_event_read(u64 r1, u event->pmu->count)return -EINVAL ; +if (unlikely (event->attr.type !=PERF_TYPE_HARDWARE && + event->attr.type !=PERF_TYPE_RAW )) +return -EINVAL ; +/* * we don't know if the function is run successfully by the * return value. It can be judged in other places, such as
Greg Kroah-Hartman
maintainerPATCH 211/312 bpf: fix map not being uncharged during map creation failure
In reply to Greg Kroah-HartmanFrom: Daniel Borkmann <[email protected]>
commit 20b2b24f91f70e7d3f0918c077546cb21bd73a87 upstream.
In map_create(), we first find and create the map, then once that suceeded, we charge it to the user's RLIMIT_MEMLOCK, and then fetch a new anon fd through anon_inode_getfd(). The problem is, once the latter fails f.e. due to RLIMIT_NOFILE limit, then we only destruct the map via map->ops->map_free(), but without uncharging the previously locked memory first. That means that the user_struct allocation is leaked as well as the accounted RLIMIT_MEMLOCK memory not released. Make the label names in the fix consistent with bpf_prog_load().
--- kernel/bpf/syscall.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
kernel/bpf/syscall.c +3 -1
@@ -152,7 +152,7 @@ static int map_create(union bpf_attr *at err =bpf_map_charge_memlock (map);if (err) - goto free_map; + goto free_map_nouncharge; err =bpf_map_new_fd (map);if (err <0 )
@@ -162,6 +162,8 @@ static int map_create(union bpf_attr *atreturn err; free_map: +bpf_map_uncharge_memlock (map); +free_map_nouncharge: map->ops->map_free (map);return err; }
Greg Kroah-Hartman
maintainerPATCH 212/312 net/mlx4_core: Fix potential corruption in counters database
In reply to Greg Kroah-HartmanFrom: Eran Ben Elisha <[email protected]>
commit 6b94bab0ee8d5def6a2aac0ef6204ee6e24386b6 upstream.
The error flow in procedure handle_existing_counter() is wrong.
The procedure should exit after encountering the error, not continue as if everything is OK.
--- drivers/net/ethernet/mellanox/mlx4/resource_tracker.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
drivers/net/ethernet/mellanox/mlx4/resource_tracker.c +6 -4
@@ -918,11 +918,13 @@ static int handle_existing_counter(strucspin_lock_irq (mlx4_tlock (dev)); r =find_res (dev, counter_index,RES_COUNTER ); -if (!r || r->owner != slave) +if (!r || r->owner != slave) { ret = -EINVAL ; - counter = container_of(r, struct res_counter, com); -if (!counter->port) - counter->port = port; + }else { + counter = container_of(r, struct res_counter, com); +if (!counter->port) + counter->port = port; + }spin_unlock_irq (mlx4_tlock (dev));return ret;
Greg Kroah-Hartman
maintainerPATCH 213/312 net/mlx4_core: Fix access to uninitialized index
In reply to Greg Kroah-HartmanFrom: Tariq Toukan <[email protected]>
commit 2bb07e155bb3e0c722c806723f737cf8020961ef upstream.
Prevent using uninitialized or negative index when handling steering entries.
--- drivers/net/ethernet/mellanox/mlx4/mcg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/mellanox/mlx4/mcg.c +2 -2
@@ -1109,7 +1109,7 @@ int mlx4_qp_attach_common(struct mlx4_destruct mlx4_cmd_mailbox *mailbox;struct mlx4_mgm *mgm;u32 members_count; -int index, prev; +int index =-1 , prev;int link =0 ;int i;int err;
@@ -1188,7 +1188,7 @@ int mlx4_qp_attach_common(struct mlx4_de goto out; out: -if (prot ==MLX4_PROT_ETH ) { +if (prot ==MLX4_PROT_ETH && index !=-1 ) {/* manage the steering entry for promisc mode */ if (new_entry) err = new_steering_entry(dev, port, steer,
Greg Kroah-Hartman
maintainerPATCH 214/312 net/mlx4_en: Fix the return value of a failure in VLAN VID add/kill
In reply to Greg Kroah-HartmanFrom: Kamal Heib <[email protected]>
commit 93c098af09455ea7bdc6f0f6b08f6ac14fa06cf4 upstream.
Modify mlx4_en_vlan_rx_[add/kill]_vid to return error value in case of failure.
--- drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-)
drivers/net/ethernet/mellanox/mlx4/en_netdev.c +11 -7
@@ -424,14 +424,18 @@ static int mlx4_en_vlan_rx_add_vid(strucmutex_lock (&mdev->state_lock);if (mdev->device_up && priv->port_up) { err =mlx4_SET_VLAN_FLTR (mdev->dev, priv); -if (err) +if (err) {en_err (priv,"Failed configuring VLAN filter\n" ); + goto out; + } } -if (mlx4_register_vlan (mdev->dev, priv->port, vid, &idx)) -en_dbg (HW , priv,"failed adding vlan %d\n" , vid); -mutex_unlock (&mdev->state_lock); + err =mlx4_register_vlan (mdev->dev, priv->port, vid, &idx); +if (err) +en_dbg (HW , priv,"Failed adding vlan %d\n" , vid); -return 0 ; +out: +mutex_unlock (&mdev->state_lock); +return err; } static int mlx4_en_vlan_rx_kill_vid(struct net_device *dev,
@@ -439,7 +443,7 @@ static int mlx4_en_vlan_rx_kill_vid(stru {struct mlx4_en_priv *priv =netdev_priv (dev);struct mlx4_en_dev *mdev = priv->mdev; -int err; +int err =0 ;en_dbg (HW , priv,"Killing VID:%d\n" , vid);
@@ -456,7 +460,7 @@ static int mlx4_en_vlan_rx_kill_vid(stru }mutex_unlock (&mdev->state_lock); -return 0 ; +return err; }static void mlx4_en_u64_to_mac (unsigned char dst_mac[ETH_ALEN +2 ],u64 src_mac)
Greg Kroah-Hartman
maintainerPATCH 215/312 net/mlx4_core: Check device state before unregistering it
In reply to Greg Kroah-HartmanFrom: Alex Vesker <[email protected]>
commit 9b022a6e0f26af108b9105b16b310393c898d9bd upstream.
Verify that the device state is registered before un-registering it. This check is required to prevent an OOPS on flows that do re-registration of the device and its previous state was unregistered.
--- drivers/net/ethernet/mellanox/mlx4/intf.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/net/ethernet/mellanox/mlx4/intf.c +3 -0
@@ -217,6 +217,9 @@ void mlx4_unregister_device(struct mlx4_struct mlx4_priv *priv =mlx4_priv (dev);struct mlx4_interface *intf; +if (!(dev->persist->interface_state &MLX4_INTERFACE_STATE_UP )) +return ; +mlx4_stop_catas_poll (dev);if (dev->persist->interface_state &MLX4_INTERFACE_STATE_DELETION &&mlx4_is_slave (dev)) {
Greg Kroah-Hartman
maintainerPATCH 216/312 net/mlx4_core: Fix the resource-type enum in res tracker to conform to FW spec
In reply to Greg Kroah-HartmanFrom: Jack Morgenstein <[email protected]>
commit aa0c08feae8161b945520ada753d0dfe62b14fe7 upstream.
The resource type enum in the resource tracker was incorrect. RES_EQ was put in the position of RES_NPORT_ID (a FC resource).
Since the remaining resources maintain their current values, and RES_EQ is not passed from slaves to the hypervisor in any FW command, this change affects only the hypervisor. Therefore, there is no backwards-compatibility issue.
--- drivers/net/ethernet/mellanox/mlx4/mlx4.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlx4/mlx4.h +2 -1
@@ -143,9 +143,10 @@ enum mlx4_resource {RES_MTT ,RES_MAC ,RES_VLAN , -RES_EQ , +RES_NPORT_ID ,RES_COUNTER ,RES_FS_RULE , +RES_EQ , MLX4_NUM_OF_RESOURCE_TYPE };
Greg Kroah-Hartman
maintainerPATCH 217/312 net/mlx4_en: Process all completions in RX rings after port goes up
In reply to Greg Kroah-HartmanFrom: Erez Shitrit <[email protected]>
commit 8d59de8f7bb3db296331c665779c653b0c8d13ba upstream.
Currently there is a race between incoming traffic and initialization flow. HW is able to receive the packets after INIT_PORT is done and unicast steering is configured. Before we set priv->port_up NAPI is not scheduled and receive queues become full. Therefore we never get new interrupts about the completions.
This issue could happen if running heavy traffic during bringing port up.
The resolution is to schedule NAPI once port_up is set. If receive queues were full this will process all cqes and release them.
--- drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 7 +++++++ 1 file changed, 7 insertions(+)
drivers/net/ethernet/mellanox/mlx4/en_netdev.c +7 -0
@@ -1720,6 +1720,13 @@ int mlx4_en_start_port(struct net_devicevxlan_get_rx_port (dev);#endif priv->port_up = true; + +/* Process all completions if exist to prevent +* the queues freezing if they are full +*/ +for (i =0 ; i < priv->rx_ring_num; i++) +napi_schedule (&priv->rx_cq[i]->napi); +netif_tx_start_all_queues (dev);netif_device_attach (dev);
Greg Kroah-Hartman
maintainerPATCH 218/312 net/mlx4_core: Do not access comm channel if it has not yet been initialized
In reply to Greg Kroah-HartmanFrom: Jack Morgenstein <[email protected]>
commit 81d184199e328fdad5633da139a10337327154e0 upstream.
In the Hypervisor, there are several FW commands which are invoked before the comm channel is initialized (in mlx4_multi_func_init). These include MOD_STAT_CONFIG, QUERY_DEV_CAP, INIT_HCA, and others.
If any of these commands fails, say with a timeout, the Hypervisor driver enters the internal error reset flow. In this flow, the driver attempts to notify all slaves via the comm channel that an internal error has occurred.
Since the comm channel has not yet been initialized (i.e., mapped via ioremap), this will cause dereferencing a NULL pointer.
To fix this, do not access the comm channel in the internal error flow if it has not yet been initialized.
--- drivers/net/ethernet/mellanox/mlx4/cmd.c | 9 +++++++++ 1 file changed, 9 insertions(+)
drivers/net/ethernet/mellanox/mlx4/cmd.c +9 -0
@@ -2451,6 +2451,7 @@ err_comm_admin:kfree (priv->mfunc.master.slave_state); err_comm:iounmap (priv->mfunc.comm); + priv->mfunc.comm =NULL ; err_vhcr: dma_free_coherent(&dev->persist->pdev->dev, PAGE_SIZE, priv->mfunc.vhcr,
@@ -2518,6 +2519,13 @@ void mlx4_report_internal_err_comm_eventint slave;u32 slave_read; +/* If the comm channel has not yet been initialized, +* skip reporting the internal error event to all +* the communication channels. +*/ +if (!priv->mfunc.comm) +return ; +/* Report an internal error event to all * communication channels. */
@@ -2552,6 +2560,7 @@ void mlx4_multi_func_cleanup(struct mlx4 }iounmap (priv->mfunc.comm); + priv->mfunc.comm =NULL ; }void mlx4_cmd_cleanup (struct mlx4_dev *dev,int cleanup_mask)
Greg Kroah-Hartman
maintainerPATCH 219/312 net/mlx4_en: Fix potential deadlock in port statistics flow
In reply to Greg Kroah-HartmanFrom: Jack Morgenstein <[email protected]>
commit d2582a03939ed0a80ffcd3ea5345505bc8067c54 upstream.
mlx4_en_DUMP_ETH_STATS took the *counter mutex* and then called the FW command, with WRAPPED attribute. As a result, the fw command is wrapped on the Hypervisor when it calls mlx4_en_DUMP_ETH_STATS. The FW command wrapper flow on the hypervisor takes the *slave_cmd_mutex* during processing.
At the same time, a VF could be in the process of coming up, and could call mlx4_QUERY_FUNC_CAP. On the hypervisor, the command flow takes the *slave_cmd_mutex*, then executes mlx4_QUERY_FUNC_CAP_wrapper. mlx4_QUERY_FUNC_CAP wrapper calls mlx4_get_default_counter_index(), which takes the *counter mutex*. DEADLOCK.
The fix is that the DUMP_ETH_STATS fw command should be called with the NATIVE attribute, so that on the hypervisor, this command does not enter the wrapper flow.
Since the Hypervisor no longer goes through the wrapper code, we also simply return 0 in mlx4_DUMP_ETH_STATS_wrapper (i.e.the function succeeds, but the returned data will be all zeroes).
No need to test if it is the Hypervisor going through the wrapper.
--- drivers/net/ethernet/mellanox/mlx4/en_port.c | 4 ++-- drivers/net/ethernet/mellanox/mlx4/mlx4.h | 2 -- drivers/net/ethernet/mellanox/mlx4/port.c | 13 +------------ 3 files changed, 3 insertions(+), 16 deletions(-)
drivers/net/ethernet/mellanox/mlx4/en_port.c +2 -2
@@ -164,7 +164,7 @@ int mlx4_en_DUMP_ETH_STATS(struct mlx4_ereturn PTR_ERR (mailbox); err =mlx4_cmd_box (mdev->dev,0 , mailbox->dma, in_mod,0 ,MLX4_CMD_DUMP_ETH_STATS ,MLX4_CMD_TIME_CLASS_B , -MLX4_CMD_WRAPPED ); +MLX4_CMD_NATIVE );if (err) goto out;
@@ -325,7 +325,7 @@ int mlx4_en_DUMP_ETH_STATS(struct mlx4_e err =mlx4_cmd_box (mdev->dev,0 , mailbox->dma, in_mod |MLX4_DUMP_ETH_STATS_FLOW_CONTROL ,0 ,MLX4_CMD_DUMP_ETH_STATS , -MLX4_CMD_TIME_CLASS_B ,MLX4_CMD_WRAPPED ); +MLX4_CMD_TIME_CLASS_B ,MLX4_CMD_NATIVE );if (err) goto out; }
drivers/net/ethernet/mellanox/mlx4/mlx4.h +0 -2
@@ -1313,8 +1313,6 @@ int mlx4_SET_VLAN_FLTR_wrapper(struct ml struct mlx4_cmd_info *cmd);int mlx4_common_set_vlan_fltr (struct mlx4_dev *dev,int function,int port,void *buf); -int mlx4_common_dump_eth_stats (struct mlx4_dev *dev,int slave,u32 in_mod, -struct mlx4_cmd_mailbox *outbox); int mlx4_DUMP_ETH_STATS_wrapper(struct mlx4_dev *dev, int slave, struct mlx4_vhcr *vhcr, struct mlx4_cmd_mailbox *inbox,
drivers/net/ethernet/mellanox/mlx4/port.c +1 -12
@@ -1155,24 +1155,13 @@ int mlx4_SET_VLAN_FLTR_wrapper(struct mlreturn err; } -int mlx4_common_dump_eth_stats (struct mlx4_dev *dev,int slave, -u32 in_mod,struct mlx4_cmd_mailbox *outbox) -{ -return mlx4_cmd_box (dev,0 , outbox->dma, in_mod,0 , -MLX4_CMD_DUMP_ETH_STATS ,MLX4_CMD_TIME_CLASS_B , -MLX4_CMD_NATIVE ); -} -int mlx4_DUMP_ETH_STATS_wrapper (struct mlx4_dev *dev,int slave,struct mlx4_vhcr *vhcr,struct mlx4_cmd_mailbox *inbox,struct mlx4_cmd_mailbox *outbox,struct mlx4_cmd_info *cmd) { -if (slave != dev->caps.function) -return 0 ; -return mlx4_common_dump_eth_stats (dev, slave, - vhcr->in_modifier, outbox); +return 0 ; } int mlx4_get_slave_from_roce_gid(struct mlx4_dev *dev, int port, u8 *gid,
Greg Kroah-Hartman
maintainerPATCH 220/312 net/mlx4: Fix uninitialized fields in rule when adding promiscuous mode to device managed flow steering
In reply to Greg Kroah-HartmanFrom: Jack Morgenstein <[email protected]>
commit 44b911e77793d686b481608770d0c55c18055ba0 upstream.
In procedure mlx4_flow_steer_promisc_add(), several fields were left uninitialized in the rule structure. Correctly initialize these fields.
--- drivers/net/ethernet/mellanox/mlx4/mcg.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlx4/mcg.c +6 -1
@@ -1464,7 +1464,12 @@ EXPORT_SYMBOL_GPL(mlx4_multicast_detach)int mlx4_flow_steer_promisc_add (struct mlx4_dev *dev,u8 port,u32 qpn,enum mlx4_net_trans_promisc_mode mode) { -struct mlx4_net_trans_rule rule; +struct mlx4_net_trans_rule rule = { + .queue_mode =MLX4_NET_TRANS_Q_FIFO , + .exclusive =0 , + .allow_loopback =1 , + }; +u64 *regid_p;switch (mode) {
Greg Kroah-Hartman
maintainerPATCH 221/312 net/mlx4_core: Fix QUERY FUNC CAP flags
In reply to Greg Kroah-HartmanFrom: Moshe Shemesh <[email protected]>
commit c9cc599a96a6822c52cd72ed31dd7f813d792b4f upstream.
Separate QUERY_FUNC_CAP flags0 from QUERY_FUNC_CAP flags, as 'flags' is already used for another set of flags in FUNC CAP, while phv bit should be part of a different set of flags.
Remove QUERY_FUNC_CAP port_flags field, as it is not in use.
--- drivers/net/ethernet/mellanox/mlx4/fw.c | 5 ++--- drivers/net/ethernet/mellanox/mlx4/fw.h | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-)
drivers/net/ethernet/mellanox/mlx4/fw.c +2 -3
@@ -610,8 +610,7 @@ int mlx4_QUERY_FUNC_CAP(struct mlx4_devMLX4_GET (func_cap->phys_port_id, outbox,QUERY_FUNC_CAP_PHYS_PORT_ID ); -MLX4_GET (field, outbox,QUERY_FUNC_CAP_FLAGS0_OFFSET ); - func_cap->flags |= (field &QUERY_FUNC_CAP_PHV_BIT ); +MLX4_GET (func_cap->flags0, outbox,QUERY_FUNC_CAP_FLAGS0_OFFSET );/* All other resources are allocated by the master, but we still report * 'num' and 'reserved' capabilities as follows:
@@ -2840,7 +2839,7 @@ int get_phv_bit(struct mlx4_dev *dev, u8memset (&func_cap,0 ,sizeof (func_cap)); err =mlx4_QUERY_FUNC_CAP (dev, port, &func_cap);if (!err) - *phv = func_cap.flags &QUERY_FUNC_CAP_PHV_BIT ; + *phv = func_cap.flags0 &QUERY_FUNC_CAP_PHV_BIT ;return err; }EXPORT_SYMBOL (get_phv_bit);
drivers/net/ethernet/mellanox/mlx4/fw.h +1 -1
@@ -150,7 +150,7 @@ struct mlx4_func_cap {u32 qp1_proxy_qpn;u32 reserved_lkey;u8 physical_port; -u8 port_flags; +u8 flags0;u8 flags1;u64 phys_port_id;u32 extra_flags;
Greg Kroah-Hartman
maintainerPATCH 222/312 mlxsw: switchx2: Fix misuse of hard_header_len
In reply to Greg Kroah-HartmanFrom: Yotam Gigi <[email protected]>
commit 251d41c58b765f00d73b1b4230cad256e25f2735 upstream.
In order to specify that the mlxsw switchx2 driver needs additional headroom for packets, there have been use of the hard_header_len field of the netdevice struct.
This commit changes that to use needed_headroom instead, as this is the correct way to do that.
--- drivers/net/ethernet/mellanox/mlxsw/switchx2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlxsw/switchx2.c +1 -1
@@ -993,7 +993,7 @@ static int mlxsw_sx_port_create(struct m/* Each packet needs to have a Tx header (metadata) on top all other * headers. */ - dev->hard_header_len +=MLXSW_TXHDR_LEN ; + dev->needed_headroom =MLXSW_TXHDR_LEN ; err =mlxsw_sx_port_module_check (mlxsw_sx_port, &usable);if (err) {
Greg Kroah-Hartman
maintainerPATCH 223/312 mlxsw: switchx2: Fix ethernet port initialization
In reply to Greg Kroah-HartmanFrom: Elad Raz <[email protected]>
commit 7fb6a36bab6b0b158f93eb13faa1b440f8b26009 upstream.
When creating an ethernet port fails, we must move the port to disable, otherwise putting the port in switch partition 0 (ETH) or 1 (IB) will always fails.
--- drivers/net/ethernet/mellanox/mlxsw/switchx2.c | 1 + 1 file changed, 1 insertion(+)
drivers/net/ethernet/mellanox/mlxsw/switchx2.c +1 -0
@@ -1074,6 +1074,7 @@ err_port_stp_state_set: err_port_admin_status_set: err_port_mtu_set: err_port_speed_set: +mlxsw_sx_port_swid_set (mlxsw_sx_port,MLXSW_PORT_SWID_DISABLED_PORT ); err_port_swid_set: err_port_system_port_mapping_set: port_not_usable:
Greg Kroah-Hartman
maintainerPATCH 224/312 sched/fair: Fix calc_cfs_shares() fixed point arithmetics width confusion
In reply to Greg Kroah-HartmanFrom: Peter Zijlstra <[email protected]>
commit ea1dc6fc6242f991656e35e2ed3d90ec1cd13418 upstream.
Commit:
fde7d22e01aa ("sched/fair: Fix overly small weight for interactive group entities")
did something non-obvious but also did it buggy yet latent.
The problem was exposed for real by a later commit in the v4.7 merge window:
2159197d6677 ("sched/core: Enable increased load resolution on 64-bit kernels")
... after which tg->load_avg and cfs_rq->load.weight had different units (10 bit fixed point and 20 bit fixed point resp.).
Add a comment to explain the use of cfs_rq->load.weight over the 'natural' cfs_rq->avg.load_avg and add scale_load_down() to correct for the difference in unit.
Since this is (now, as per a previous commit) the only user of calc_tg_weight(), collapse it.
The effects of this bug should be randomly inconsistent SMP-balancing of cgroups workloads.
--- kernel/sched/fair.c | 27 +++++++++++---------------- 1 file changed, 11 insertions(+), 16 deletions(-)
kernel/sched/fair.c +11 -16
@@ -2394,28 +2394,22 @@ account_entity_dequeue(struct cfs_rq *cf #ifdef CONFIG_FAIR_GROUP_SCHED # ifdef CONFIG_SMP -static inline long calc_tg_weight(struct task_group *tg, struct cfs_rq *cfs_rq) +static long calc_cfs_shares(struct cfs_rq *cfs_rq, struct task_group *tg) { - long tg_weight; + long tg_weight, load, shares; /* - * Use this CPU's real-time load instead of the last load contribution - * as the updating of the contribution is delayed, and we will use the - * the real-time load to calc the share. See update_tg_load_avg(). + * This really should be: cfs_rq->avg.load_avg, but instead we use + * cfs_rq->load.weight, which is its upper bound. This helps ramp up + * the shares for small weight interactive tasks. */ - tg_weight = atomic_long_read(&tg->load_avg); - tg_weight -= cfs_rq->tg_load_avg_contrib; - tg_weight += cfs_rq->load.weight; - - return tg_weight; -} + load = scale_load_down(cfs_rq->load.weight); -static long calc_cfs_shares(struct cfs_rq *cfs_rq, struct task_group *tg) -{ - long tg_weight, load, shares; + tg_weight = atomic_long_read(&tg->load_avg); - tg_weight = calc_tg_weight(tg, cfs_rq); - load = cfs_rq->load.weight; + /* Ensure tg_weight >= load */ + tg_weight -= cfs_rq->tg_load_avg_contrib; + tg_weight += load; shares = (tg->shares * load); if (tg_weight)
@@ -2434,6 +2428,7 @@ static inline long calc_cfs_shares(struc return tg->shares; }# endif /* CONFIG_SMP */ +static void reweight_entity (struct cfs_rq *cfs_rq,struct sched_entity *se,unsigned long weight) {
Greg Kroah-Hartman
maintainerPATCH 225/312 net_sched: flower: Avoid dissection of unmasked keys
In reply to Greg Kroah-HartmanFrom: Hadar Hen Zion <[email protected]>
commit 339ba878cfb01b68de3d281ba33fd5e4c9f76546 upstream.
The current flower implementation checks the mask range and set all the keys included in that range as "used_keys", even if a specific key in the range has a zero mask.
This behavior can cause a false positive return value of dissector_uses_key function and unnecessary dissection in __skb_flow_dissect.
This patch checks explicitly the mask of each key and "used_keys" will be set accordingly.
--- net/sched/cls_flower.c | 28 +++++++++++++--------------- 1 file changed, 13 insertions(+), 15 deletions(-)
net/sched/cls_flower.c +13 -15
@@ -351,12 +351,10 @@ static int fl_init_hashtable(struct cls_#define FL_KEY_MEMBER_OFFSET (member) offsetof(struct fl_flow_key, member)#define FL_KEY_MEMBER_SIZE (member) (sizeof(((struct fl_flow_key *) 0)->member)) -#define FL_KEY_MEMBER_END_OFFSET (member) \ - (FL_KEY_MEMBER_OFFSET(member) + FL_KEY_MEMBER_SIZE(member)) -#define FL_KEY_IN_RANGE (mask, member) \ - (FL_KEY_MEMBER_OFFSET(member) <= (mask)->range.end && \ - FL_KEY_MEMBER_END_OFFSET(member) >= (mask)->range.start) +#define FL_KEY_IS_MASKED (mask, member) \ + memchr_inv(((char *)mask) + FL_KEY_MEMBER_OFFSET(member), \ + 0, FL_KEY_MEMBER_SIZE(member)) \#define FL_KEY_SET (keys, cnt, id, member) \ do { \
@@ -365,9 +363,9 @@ static int fl_init_hashtable(struct cls_ cnt++; \ }while (0 ); -#define FL_KEY_SET_IF_IN_RANGE (mask, keys, cnt, id, member) \ +#define FL_KEY_SET_IF_MASKED (mask, keys, cnt, id, member) \ do { \ - if (FL_KEY_IN_RANGE(mask, member)) \ + if (FL_KEY_IS_MASKED(mask, member)) \ FL_KEY_SET(keys, cnt, id, member); \ } while(0);
@@ -379,14 +377,14 @@ static void fl_init_dissector(struct clsFL_KEY_SET (keys, cnt,FLOW_DISSECTOR_KEY_CONTROL , control);FL_KEY_SET (keys, cnt,FLOW_DISSECTOR_KEY_BASIC , basic); -FL_KEY_SET_IF_IN_RANGE (mask, keys, cnt, -FLOW_DISSECTOR_KEY_ETH_ADDRS , eth); -FL_KEY_SET_IF_IN_RANGE (mask, keys, cnt, -FLOW_DISSECTOR_KEY_IPV4_ADDRS , ipv4); -FL_KEY_SET_IF_IN_RANGE (mask, keys, cnt, -FLOW_DISSECTOR_KEY_IPV6_ADDRS , ipv6); -FL_KEY_SET_IF_IN_RANGE (mask, keys, cnt, -FLOW_DISSECTOR_KEY_PORTS , tp); +FL_KEY_SET_IF_MASKED (&mask->key, keys, cnt, +FLOW_DISSECTOR_KEY_ETH_ADDRS , eth); +FL_KEY_SET_IF_MASKED (&mask->key, keys, cnt, +FLOW_DISSECTOR_KEY_IPV4_ADDRS , ipv4); +FL_KEY_SET_IF_MASKED (&mask->key, keys, cnt, +FLOW_DISSECTOR_KEY_IPV6_ADDRS , ipv6); +FL_KEY_SET_IF_MASKED (&mask->key, keys, cnt, +FLOW_DISSECTOR_KEY_PORTS , tp);skb_flow_dissector_init (&head->dissector, keys, cnt); }
Greg Kroah-Hartman
maintainerPATCH 226/312 pkt_sched: fq: use proper locking in fq_dump_stats()
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit 695b4ec0f0a9cf29deabd3ac075911d58b31f42b upstream.
When fq is used on 32bit kernels, we need to lock the qdisc before copying 64bit fields.
Otherwise "tc -s qdisc ..." might report bogus values.
--- net/sched/sch_fq.c | 32 ++++++++++++++++++-------------- 1 file changed, 18 insertions(+), 14 deletions(-)
net/sched/sch_fq.c +18 -14
@@ -830,20 +830,24 @@ nla_put_failure:static int fq_dump_stats (struct Qdisc *sch,struct gnet_dump *d) {struct fq_sched_data *q =qdisc_priv (sch); -u64 now =ktime_get_ns (); -struct tc_fq_qd_stats st = { - .gc_flows = q->stat_gc_flows, - .highprio_packets = q->stat_internal_packets, - .tcp_retrans = q->stat_tcp_retrans, - .throttled = q->stat_throttled, - .flows_plimit = q->stat_flows_plimit, - .pkts_too_long = q->stat_pkts_too_long, - .allocation_errors = q->stat_allocation_errors, - .flows = q->flows, - .inactive_flows = q->inactive_flows, - .throttled_flows = q->throttled_flows, - .time_next_delayed_flow = q->time_next_delayed_flow - now, - }; +struct tc_fq_qd_stats st; + +sch_tree_lock (sch); + + st.gc_flows = q->stat_gc_flows; + st.highprio_packets = q->stat_internal_packets; + st.tcp_retrans = q->stat_tcp_retrans; + st.throttled = q->stat_throttled; + st.flows_plimit = q->stat_flows_plimit; + st.pkts_too_long = q->stat_pkts_too_long; + st.allocation_errors = q->stat_allocation_errors; + st.time_next_delayed_flow = q->time_next_delayed_flow -ktime_get_ns (); + st.flows = q->flows; + st.inactive_flows = q->inactive_flows; + st.throttled_flows = q->throttled_flows; + st.pad =0 ; + +sch_tree_unlock (sch);return gnet_stats_copy_app (d, &st,sizeof (st)); }
Greg Kroah-Hartman
maintainerPATCH 227/312 sched/preempt: Fix preempt_count manipulations
In reply to Greg Kroah-HartmanFrom: Peter Zijlstra <[email protected]>
commit 2e636d5e66c35dfcbaf617aa8fa963f6847478fe upstream.
Vikram reported that his ARM64 compiler managed to 'optimize' away the preempt_count manipulations in code like:
preempt_enable_no_resched(); put_user(); preempt_disable();
Irrespective of that fact that that is horrible code that should be fixed for many reasons, it does highlight a deficiency in the generic preempt_count manipulators. As it is never right to combine/elide preempt_count manipulations like this.
Therefore sprinkle some volatile in the two generic accessors to ensure the compiler is aware of the fact that the preempt_count is observed outside of the regular program-order view and thus cannot be optimized away like this.
x86; the only arch not using the generic code is not affected as we do all this in asm in order to use the segment base per-cpu stuff.
--- include/asm-generic/preempt.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
include/asm-generic/preempt.h +2 -2
@@ -7,10 +7,10 @@ static __always_inline int preempt_count(void) { -return current_thread_info ()->preempt_count; +return READ_ONCE (current_thread_info ()->preempt_count); } -static __always_inline int *preempt_count_ptr(void) +static __always_inline volatile int *preempt_count_ptr(void) {return ¤t_thread_info ()->preempt_count; }
Greg Kroah-Hartman
maintainerPATCH 228/312 power: bq27xxx: fix reading for bq27000 and bq27010
In reply to Greg Kroah-HartmanFrom: H. Nikolaus Schaller <[email protected]>
commit 549d7b317c761dbf4ed0c2945aec3acc9ca7ae14 upstream.
bug: the driver reports funny capacity values:
root@letux:/sys/class/power_supply/bq27000-battery# cat uevent POWER_SUPPLY_NAME=bq27000-battery POWER_SUPPLY_STATUS=Charging POWER_SUPPLY_PRESENT=1 POWER_SUPPLY_VOLTAGE_NOW=3702000 POWER_SUPPLY_CURRENT_NOW=-464635 POWER_SUPPLY_CAPACITY=1536 <- over 100% is magic POWER_SUPPLY_CAPACITY_LEVEL=Normal POWER_SUPPLY_TEMP=311 POWER_SUPPLY_TIME_TO_FULL_NOW=10440 POWER_SUPPLY_TECHNOLOGY=Li-ion POWER_SUPPLY_CHARGE_FULL=805450 POWER_SUPPLY_CHARGE_NOW=1068 POWER_SUPPLY_CHARGE_FULL_DESIGN=8844998 <- battery has just 1200 mAh POWER_SUPPLY_CYCLE_COUNT=21 POWER_SUPPLY_ENERGY_NOW=0 POWER_SUPPLY_POWER_AVG=0 POWER_SUPPLY_HEALTH=Good POWER_SUPPLY_MANUFACTURER=Texas Instruments
reason: the state of charge and the design capacity register are single byte only. The design capacity returns the higer order byte.
tested: GTA04 with Openmoko/FIC HF08x battery (using hdq)
--- drivers/power/bq27xxx_battery.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-)
drivers/power/bq27xxx_battery.c +9 -3
@@ -471,7 +471,10 @@ static int bq27xxx_battery_read_soc(stru {int soc; - soc =bq27xxx_read (di,BQ27XXX_REG_SOC , false); +if (di->chip ==BQ27000 || di->chip ==BQ27010 ) + soc =bq27xxx_read (di,BQ27XXX_REG_SOC , true); +else + soc =bq27xxx_read (di,BQ27XXX_REG_SOC , false);if (soc <0 )dev_dbg (di->dev,"error reading State-of-Charge\n" );
@@ -536,7 +539,10 @@ static int bq27xxx_battery_read_dcap(str {int dcap; - dcap =bq27xxx_read (di,BQ27XXX_REG_DCAP , false); +if (di->chip ==BQ27000 || di->chip ==BQ27010 ) + dcap =bq27xxx_read (di,BQ27XXX_REG_DCAP , true); +else + dcap =bq27xxx_read (di,BQ27XXX_REG_DCAP , false);if (dcap <0 ) {dev_dbg (di->dev,"error reading initial last measured discharge\n" );
@@ -544,7 +550,7 @@ static int bq27xxx_battery_read_dcap(str }if (di->chip ==BQ27000 || di->chip ==BQ27010 ) - dcap *=BQ27XXX_CURRENT_CONSTANT /BQ27XXX_RS ; + dcap = (dcap <<8 ) *BQ27XXX_CURRENT_CONSTANT /BQ27XXX_RS ;else dcap *=1000 ;
Greg Kroah-Hartman
maintainerPATCH 229/312 power: bq27xxx: fix register numbers of bq27500
In reply to Greg Kroah-HartmanFrom: H. Nikolaus Schaller <[email protected]>
commit 099867a16a0fa9fd5aafc32e3b1a6f8a90f17834 upstream.
bug: according to data sheet some register numbers are wrong.
tested: no
--- drivers/power/bq27xxx_battery.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/power/bq27xxx_battery.c +3 -3
@@ -198,10 +198,10 @@ static u8 bq27500_regs[] = {INVALID_REG_ADDR ,/* TTECP - NA */ 0x0c ,/* NAC */ 0x12 ,/* LMD(FCC) */ -0x1e ,/* CYCT */ +0x2a ,/* CYCT */ INVALID_REG_ADDR ,/* AE - NA */ -0x20 ,/* SOC(RSOC) */ -0x2e ,/* DCAP(ILMD) */ +0x2c ,/* SOC(RSOC) */ +0x3c ,/* DCAP(ILMD) */ INVALID_REG_ADDR, /* AP - NA */ };
Greg Kroah-Hartman
maintainerPATCH 230/312 power: test_power: correctly handle empty writes
In reply to Greg Kroah-HartmanFrom: Sasha Levin <[email protected]>
commit 6b9140f39c2aaf76791197fbab0839c0e4af56e8 upstream.
Writing 0 length data into test_power makes it access an invalid array location and kill the system.
--- drivers/power/test_power.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/power/test_power.c +2 -0
@@ -301,6 +301,8 @@ static int map_get_value(struct battery_ buf[MAX_KEYLENGTH -1 ] ='\0' ; cr =strnlen (buf,MAX_KEYLENGTH ) -1 ; +if (cr <0 ) +return def_val;if (buf[cr] =='\n' ) buf[cr] ='\0' ;
Greg Kroah-Hartman
maintainerPATCH 231/312 power: bq27xxx_battery: Fix bq27541 AveragePower register address
In reply to Greg Kroah-HartmanFrom: Liu Xiang <[email protected]>
commit 265b60497a57da56a4be7d5c72983ae89dc0765e upstream.
Currently in bq27541 driver, the average power register address is incorrectly set to 0x76, which would result in an error:
bq27xxx-battery 2-0055: error reading average power register 10: -11 According to the bq27541 datasheet, fix this problem by setting the average power register address to 0x24.
--- drivers/power/bq27xxx_battery.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/power/bq27xxx_battery.c +1 -1
@@ -242,7 +242,7 @@ static u8 bq27541_regs[] = {INVALID_REG_ADDR ,/* AE - NA */ 0x2c ,/* SOC(RSOC) */ 0x3c ,/* DCAP */ - 0x76, /* AP */ + 0x24, /* AP */ }; static u8 bq27545_regs[] = {
Greg Kroah-Hartman
maintainerPATCH 232/312 power_supply: tps65217-charger: Fix NULL deref during property export
In reply to Greg Kroah-HartmanFrom: Marcin Niestroj <[email protected]>
commit 362761299eea7dfc3a4870551de36e08758b9254 upstream.
This bug leads to:
[ 1.906411] Unable to handle kernel NULL pointer dereference at virtual address 0000000c [ 1.914878] pgd = c0004000 [ 1.917786] [0000000c] *pgd=00000000 [ 1.921536] Internal error: Oops: 5 [#1] SMP ARM [ 1.926357] Modules linked in: [ 1.929556] CPU: 0 PID: 14 Comm: kworker/0:1 Not tainted 4.4.5 #18 [ 1.936006] Hardware name: Generic AM33XX (Flattened Device Tree) [ 1.942383] Workqueue: events power_supply_changed_work [ 1.947842] task: de2c41c0 ti: de2c8000 task.ti: de2c8000 [ 1.953483] PC is at tps65217_ac_get_property+0x14/0x28 [ 1.958937] LR is at tps65217_ac_get_property+0x10/0x28
Driver was trying to use drv_data in property get handler. However drv_data was not set, so it caused NULL pointer dereference. This patch properly sets drv_data during probe by power_supply_config parameter, so the property get handler works as desired.
--- drivers/power/tps65217_charger.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
drivers/power/tps65217_charger.c +5 -1
@@ -197,6 +197,7 @@ static int tps65217_charger_probe(struct {struct tps65217 *tps =dev_get_drvdata (pdev->dev.parent);struct tps65217_charger *charger; +struct power_supply_config cfg = {};int ret;dev_dbg (&pdev->dev,"%s\n" , __func__);
@@ -209,9 +210,12 @@ static int tps65217_charger_probe(struct charger->tps = tps; charger->dev = &pdev->dev; + cfg.of_node = pdev->dev.of_node; + cfg.drv_data = charger; + charger->ac =devm_power_supply_register (&pdev->dev, &tps65217_charger_desc, -NULL ); + &cfg);if (IS_ERR (charger->ac)) {dev_err (&pdev->dev,"failed: power supply register\n" );return PTR_ERR (charger->ac);
Greg Kroah-Hartman
maintainerPATCH 233/312 net: vrf: Fix dst reference counting
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit 9ab179d83b4e31ea277a123492e419067c2f129a upstream.
Vivek reported a kernel exception deleting a VRF with an active connection through it. The root cause is that the socket has a cached reference to a dst that is destroyed. Converting the dst_destroy to dst_release and letting proper reference counting kick in does not work as the dst has a reference to the device which needs to be released as well.
I talked to Hannes about this at netdev and he pointed out the ipv4 and ipv6 dst handling has dst_ifdown for just this scenario. Rather than continuing with the reinvented dst wheel in VRF just remove it and leverage the ipv4 and ipv6 versions.
--- drivers/net/vrf.c | 177 ++++-------------------------------------------- include/net/ip6_route.h | 3 include/net/route.h | 3 net/ipv4/route.c | 7 + net/ipv6/route.c | 7 + 5 files changed, 30 insertions(+), 167 deletions(-)
drivers/net/vrf.c +16 -161
@@ -71,41 +71,6 @@ struct pcpu_dstats {struct u64_stats_sync syncp; }; -static struct dst_entry *vrf_ip_check (struct dst_entry *dst,u32 cookie) -{ -return dst; -} - -static int vrf_ip_local_out (struct net *net,struct sock *sk,struct sk_buff *skb) -{ -return ip_local_out (net, sk, skb); -} - -static unsigned int vrf_v4_mtu (const struct dst_entry *dst) -{ -/* TO-DO: return max ethernet size? */ -return dst->dev->mtu; -} - -static void vrf_dst_destroy (struct dst_entry *dst) -{ -/* our dst lives forever - or until the device is closed */ -} - -static unsigned int vrf_default_advmss (const struct dst_entry *dst) -{ -return 65535 -40 ; -} - -static struct dst_ops vrf_dst_ops = { - .family =AF_INET , - .local_out = vrf_ip_local_out, - .check = vrf_ip_check, - .mtu = vrf_v4_mtu, - .destroy = vrf_dst_destroy, - .default_advmss = vrf_default_advmss, -}; -/* neighbor handling is done with actual device; do not want * to flip skb->dev for those ndisc packets. This really fails * for multiple next protocols (e.g., NEXTHDR_HOP). But it is
@@ -363,46 +328,6 @@ static netdev_tx_t vrf_xmit(struct sk_bu }#if IS_ENABLED (CONFIG_IPV6 ) -static struct dst_entry *vrf_ip6_check (struct dst_entry *dst,u32 cookie) -{ -return dst; -} - -static struct dst_ops vrf_dst_ops6 = { - .family =AF_INET6 , - .local_out = ip6_local_out, - .check = vrf_ip6_check, - .mtu = vrf_v4_mtu, - .destroy = vrf_dst_destroy, - .default_advmss = vrf_default_advmss, -}; - -static int init_dst_ops6_kmem_cachep (void ) -{ - vrf_dst_ops6.kmem_cachep =kmem_cache_create ("vrf_ip6_dst_cache" , -sizeof (struct rt6_info ), -0 , -SLAB_HWCACHE_ALIGN , -NULL ); - -if (!vrf_dst_ops6.kmem_cachep) -return -ENOMEM ; - -return 0 ; -} - -static void free_dst_ops6_kmem_cachep (void ) -{ -kmem_cache_destroy (vrf_dst_ops6.kmem_cachep); -} - -static int vrf_input6 (struct sk_buff *skb) -{ - skb->dev->stats.rx_errors++; -kfree_skb (skb); -return 0 ; -} -/* modelled after ip6_finish_output2 */ static int vrf_finish_output6 (struct net *net,struct sock *sk,struct sk_buff *skb)
@@ -445,67 +370,34 @@ static int vrf_output6(struct net *net, !(IP6CB(skb)->flags & IP6SKB_REROUTED)); } -static void vrf_rt6_destroy (struct net_vrf *vrf) +static void vrf_rt6_release (struct net_vrf *vrf) { -dst_destroy (&vrf->rt6->dst); -free_percpu (vrf->rt6->rt6i_pcpu); +dst_release (&vrf->rt6->dst); vrf->rt6 =NULL ; }static int vrf_rt6_create (struct net_device *dev) {struct net_vrf *vrf =netdev_priv (dev); -struct dst_entry *dst; +struct net *net =dev_net (dev);struct rt6_info *rt6; -int cpu;int rc = -ENOMEM ; - rt6 =dst_alloc (&vrf_dst_ops6, dev,0 , -DST_OBSOLETE_NONE , - (DST_HOST |DST_NOPOLICY |DST_NOXFRM )); + rt6 =ip6_dst_alloc (net, dev, +DST_HOST |DST_NOPOLICY |DST_NOXFRM |DST_NOCACHE );if (!rt6) goto out; - dst = &rt6->dst; - - rt6->rt6i_pcpu = alloc_percpu_gfp(struct rt6_info *, GFP_KERNEL); -if (!rt6->rt6i_pcpu) { -dst_destroy (dst); - goto out; - } -for_each_possible_cpu (cpu) { -struct rt6_info **p =per_cpu_ptr (rt6->rt6i_pcpu, cpu); - *p =NULL ; - } - -memset (dst +1 ,0 ,sizeof (*rt6) -sizeof (*dst)); - -INIT_LIST_HEAD (&rt6->rt6i_siblings); -INIT_LIST_HEAD (&rt6->rt6i_uncached); - - rt6->dst.input = vrf_input6; rt6->dst.output = vrf_output6; - - rt6->rt6i_table =fib6_get_table (dev_net (dev), vrf->tb_id); - -atomic_set (&rt6->dst.__refcnt,2 ); - + rt6->rt6i_table =fib6_get_table (net, vrf->tb_id); +dst_hold (&rt6->dst); vrf->rt6 = rt6; rc =0 ; out:return rc; }#else -static int init_dst_ops6_kmem_cachep (void ) -{ -return 0 ; -} - -static void free_dst_ops6_kmem_cachep (void ) -{ -} - -static void vrf_rt6_destroy (struct net_vrf *vrf) +static void vrf_rt6_release (struct net_vrf *vrf) { }
@@ -577,11 +469,11 @@ static int vrf_output(struct net *net, s !(IPCB(skb)->flags & IPSKB_REROUTED)); } -static void vrf_rtable_destroy (struct net_vrf *vrf) +static void vrf_rtable_release (struct net_vrf *vrf) {struct dst_entry *dst = (struct dst_entry *)vrf->rth; -dst_destroy (dst); +dst_release (dst); vrf->rth =NULL ; }
@@ -590,22 +482,10 @@ static struct rtable *vrf_rtable_create(struct net_vrf *vrf =netdev_priv (dev);struct rtable *rth; - rth =dst_alloc (&vrf_dst_ops, dev,2 , -DST_OBSOLETE_NONE , - (DST_HOST |DST_NOPOLICY |DST_NOXFRM )); + rth =rt_dst_alloc (dev,0 ,RTN_UNICAST ,1 ,1 ,0 );if (rth) { rth->dst.output = vrf_output; - rth->rt_genid =rt_genid_ipv4 (dev_net (dev)); - rth->rt_flags =0 ; - rth->rt_type =RTN_UNICAST ; - rth->rt_is_input =0 ; - rth->rt_iif =0 ; - rth->rt_pmtu =0 ; - rth->rt_gateway =0 ; - rth->rt_uses_gateway =0 ; rth->rt_table_id = vrf->tb_id; -INIT_LIST_HEAD (&rth->rt_uncached); - rth->rt_uncached_list =NULL ; }return rth;
@@ -739,8 +619,8 @@ static void vrf_dev_uninit(struct net_de// struct list_head *head = &queue->all_slaves; // struct slave *slave, *next; -vrf_rtable_destroy (vrf); -vrf_rt6_destroy (vrf); +vrf_rtable_release (vrf); +vrf_rt6_release (vrf);// list_for_each_entry_safe(slave, next, head, list) // vrf_del_slave(dev, slave->dev);
@@ -772,7 +652,7 @@ static int vrf_dev_init(struct net_devicreturn 0 ; out_rth: -vrf_rtable_destroy (vrf); +vrf_rtable_release (vrf); out_stats:free_percpu (dev->dstats); dev->dstats =NULL ;
@@ -805,7 +685,7 @@ static struct rtable *vrf_get_rtable(construct net_vrf *vrf =netdev_priv (dev); rth = vrf->rth; -atomic_inc (&rth->dst.__refcnt); +dst_hold (&rth->dst); }return rth;
@@ -856,7 +736,7 @@ static struct dst_entry *vrf_get_rt6_dststruct net_vrf *vrf =netdev_priv (dev); rt = vrf->rt6; -atomic_inc (&rt->dst.__refcnt); +dst_hold (&rt->dst); }return (struct dst_entry *)rt;
@@ -1003,19 +883,6 @@ static int __init vrf_init_module(void) {int rc; - vrf_dst_ops.kmem_cachep = -kmem_cache_create ("vrf_ip_dst_cache" , -sizeof (struct rtable ),0 , -SLAB_HWCACHE_ALIGN , -NULL ); - -if (!vrf_dst_ops.kmem_cachep) -return -ENOMEM ; - - rc =init_dst_ops6_kmem_cachep (); -if (rc !=0 ) - goto error2; -register_netdevice_notifier (&vrf_notifier_block); rc =rtnl_link_register (&vrf_link_ops);
@@ -1026,22 +893,10 @@ static int __init vrf_init_module(void) error:unregister_netdevice_notifier (&vrf_notifier_block); -free_dst_ops6_kmem_cachep (); -error2: -kmem_cache_destroy (vrf_dst_ops.kmem_cachep);return rc; } -static void __exit vrf_cleanup_module(void) -{ -rtnl_link_unregister (&vrf_link_ops); -unregister_netdevice_notifier (&vrf_notifier_block); -kmem_cache_destroy (vrf_dst_ops.kmem_cachep); -free_dst_ops6_kmem_cachep (); -} -module_init (vrf_init_module); -module_exit (vrf_cleanup_module);MODULE_AUTHOR ("Shrijeet Mukherjee, David Ahern" );MODULE_DESCRIPTION ("Device driver to instantiate VRF domains" );MODULE_LICENSE ("GPL" );
include/net/ip6_route.h +3 -0
@@ -103,6 +103,9 @@ void fib6_force_start_gc(struct net *netstruct rt6_info *addrconf_dst_alloc (struct inet6_dev *idev,const struct in6_addr *addr,bool anycast); +struct rt6_info *ip6_dst_alloc (struct net *net,struct net_device *dev, +int flags); +/* * support functions for ND *
include/net/route.h +3 -0
@@ -210,6 +210,9 @@ unsigned int inet_addr_type_dev_table(stvoid ip_rt_multicast_event (struct in_device *);int ip_rt_ioctl (struct net *,unsigned int cmd,void __user *arg);void ip_rt_get_source (u8 *src,struct sk_buff *skb,struct rtable *rt); +struct rtable *rt_dst_alloc (struct net_device *dev, +unsigned int flags,u16 type, +bool nopolicy,bool noxfrm,bool will_cache);struct in_ifaddr ;void fib_add_ifaddr (struct in_ifaddr *);
net/ipv4/route.c +4 -3
@@ -1500,9 +1500,9 @@ static void rt_set_nexthop(struct rtable#endif } -static struct rtable *rt_dst_alloc (struct net_device *dev, -unsigned int flags,u16 type, -bool nopolicy,bool noxfrm,bool will_cache) +struct rtable *rt_dst_alloc (struct net_device *dev, +unsigned int flags,u16 type, +bool nopolicy,bool noxfrm,bool will_cache) {struct rtable *rt;
@@ -1531,6 +1531,7 @@ static struct rtable *rt_dst_alloc(strucreturn rt; } +EXPORT_SYMBOL (rt_dst_alloc);/* called in rcu_read_lock() section */ static int ip_route_input_mc(struct sk_buff *skb, __be32 daddr, __be32 saddr,
net/ipv6/route.c +4 -3
@@ -339,9 +339,9 @@ static struct rt6_info *__ip6_dst_alloc(return rt; } -static struct rt6_info *ip6_dst_alloc (struct net *net, -struct net_device *dev, -int flags) +struct rt6_info *ip6_dst_alloc (struct net *net, +struct net_device *dev, +int flags) {struct rt6_info *rt =__ip6_dst_alloc (net, dev, flags);
@@ -365,6 +365,7 @@ static struct rt6_info *ip6_dst_alloc(streturn rt; } +EXPORT_SYMBOL (ip6_dst_alloc);static void ip6_dst_destroy (struct dst_entry *dst) {
Greg Kroah-Hartman
maintainerPATCH 234/312 net: Dont delete routes in different VRFs
In reply to Greg Kroah-HartmanFrom: Mark Tomlinson <[email protected]>
commit 5a56a0b3a45dd0cc5b2f7bec6afd053a474ed9f5 upstream.
When deleting an IP address from an interface, there is a clean-up of routes which refer to this local address. However, there was no check to see that the VRF matched. This meant that deletion wasn't confined to the VRF it should have been.
To solve this, a new field has been added to fib_info to hold a table id. When removing fib entries corresponding to a local ip address, this table id is also used in the comparison.
The table id is populated when the fib_info is created. This was already done in some places, but not in ip_rt_ioctl(). This has now been fixed.
--- include/net/ip_fib.h | 3 ++- net/ipv4/fib_frontend.c | 3 ++- net/ipv4/fib_semantics.c | 8 ++++++-- 3 files changed, 10 insertions(+), 4 deletions(-)
include/net/ip_fib.h +2 -1
@@ -112,6 +112,7 @@ struct fib_info {unsigned char fib_scope;unsigned char fib_type;__be32 fib_prefsrc; +u32 fib_tb_id;u32 fib_priority;struct dst_metrics *fib_metrics;#define fib_mtu fib_metrics->metrics[RTAX_MTU-1]
@@ -320,7 +321,7 @@ void fib_flush_external(struct net *net)/* Exported by fib_semantics.c */ int ip_fib_check_default (__be32 gw,struct net_device *dev);int fib_sync_down_dev (struct net_device *dev,unsigned long event,bool force); -int fib_sync_down_addr (struct net *net,__be32 local); +int fib_sync_down_addr (struct net_device *dev,__be32 local);int fib_sync_up (struct net_device *dev,unsigned int nh_flags);void fib_sync_mtu (struct net_device *dev,u32 orig_mtu);
net/ipv4/fib_frontend.c +2 -1
@@ -509,6 +509,7 @@ static int rtentry_to_fib_config(structif (!dev)return -ENODEV ; cfg->fc_oif = dev->ifindex; + cfg->fc_table =l3mdev_fib_table (dev);if (colon) {struct in_ifaddr *ifa;struct in_device *in_dev =__in_dev_get_rtnl (dev);
@@ -1034,7 +1035,7 @@ no_promotions:* First of all, we scan fib_info list searching * for stray nexthop entries, then ignite fib_flush. */ -if (fib_sync_down_addr (dev_net (dev), ifa->ifa_local)) +if (fib_sync_down_addr (dev, ifa->ifa_local))fib_flush (dev_net (dev)); } }
net/ipv4/fib_semantics.c +6 -2
@@ -1069,6 +1069,7 @@ struct fib_info *fib_create_info(struct fi->fib_priority = cfg->fc_priority; fi->fib_prefsrc = cfg->fc_prefsrc; fi->fib_type = cfg->fc_type; + fi->fib_tb_id = cfg->fc_table; fi->fib_nhs = nhs;change_nexthops (fi) {
@@ -1352,18 +1353,21 @@ nla_put_failure:* referring to it. * - device went down -> we must shutdown all nexthops going via it. */ -int fib_sync_down_addr (struct net *net,__be32 local) +int fib_sync_down_addr (struct net_device *dev,__be32 local) {int ret =0 ;unsigned int hash =fib_laddr_hashfn (local);struct hlist_head *head = &fib_info_laddrhash[hash]; +struct net *net =dev_net (dev); +int tb_id =l3mdev_fib_table (dev);struct fib_info *fi;if (!fib_info_laddrhash || local ==0 )return 0 ;hlist_for_each_entry (fi, head, fib_lhash) { -if (!net_eq (fi->fib_net, net)) +if (!net_eq (fi->fib_net, net) || + fi->fib_tb_id != tb_id)continue ;if (fi->fib_prefsrc == local) { fi->fib_flags |=RTNH_F_DEAD ;
Greg Kroah-Hartman
maintainerPATCH 235/312 vti6: fix input path
In reply to Greg Kroah-HartmanFrom: Nicolas Dichtel <[email protected]>
commit 63c43787d35e45562a6b5927e2edc8f4783d95b8 upstream.
Since commit 1625f4529957, vti6 is broken, all input packets are dropped (LINUX_MIB_XFRMINNOSTATES is incremented).
XFRM_TUNNEL_SKB_CB(skb)->tunnel.ip6 is set by vti6_rcv() before calling xfrm6_rcv()/xfrm6_rcv_spi(), thus we cannot set to NULL that value in xfrm6_rcv_spi().
A new function xfrm6_rcv_tnl() that enables to pass a value to xfrm6_rcv_spi() is added, so that xfrm6_rcv() is not touched (this function is used in several handlers).
--- include/net/xfrm.h | 4 +++- net/ipv6/ip6_vti.c | 4 +--- net/ipv6/xfrm6_input.c | 16 +++++++++++----- net/ipv6/xfrm6_tunnel.c | 2 +- 4 files changed, 16 insertions(+), 10 deletions(-)
include/net/xfrm.h +3 -1
@@ -1551,8 +1551,10 @@ int xfrm4_tunnel_deregister(struct xfrm_void xfrm4_local_error (struct sk_buff *skb,u32 mtu);int xfrm6_extract_header (struct sk_buff *skb);int xfrm6_extract_input (struct xfrm_state *x,struct sk_buff *skb); -int xfrm6_rcv_spi (struct sk_buff *skb,int nexthdr,__be32 spi); +int xfrm6_rcv_spi (struct sk_buff *skb,int nexthdr,__be32 spi, +struct ip6_tnl *t);int xfrm6_transport_finish (struct sk_buff *skb,int async); +int xfrm6_rcv_tnl (struct sk_buff *skb,struct ip6_tnl *t);int xfrm6_rcv (struct sk_buff *skb);int xfrm6_input_addr (struct sk_buff *skb,xfrm_address_t *daddr,xfrm_address_t *saddr,u8 proto);
net/ipv6/ip6_vti.c +1 -3
@@ -324,11 +324,9 @@ static int vti6_rcv(struct sk_buff *skb) goto discard; } -XFRM_TUNNEL_SKB_CB (skb)->tunnel.ip6 = t; -rcu_read_unlock (); -return xfrm6_rcv (skb); +return xfrm6_rcv_tnl (skb, t); }rcu_read_unlock ();return -EINVAL ;
net/ipv6/xfrm6_input.c +11 -5
@@ -21,9 +21,10 @@ int xfrm6_extract_input(struct xfrm_statreturn xfrm6_extract_header (skb); } -int xfrm6_rcv_spi (struct sk_buff *skb,int nexthdr,__be32 spi) +int xfrm6_rcv_spi (struct sk_buff *skb,int nexthdr,__be32 spi, +struct ip6_tnl *t) { -XFRM_TUNNEL_SKB_CB (skb)->tunnel.ip6 =NULL ; +XFRM_TUNNEL_SKB_CB (skb)->tunnel.ip6 = t;XFRM_SPI_SKB_CB (skb)->family =AF_INET6 ;XFRM_SPI_SKB_CB (skb)->daddroff = offsetof(struct ipv6hdr , daddr);return xfrm_input (skb, nexthdr, spi,0 );
@@ -49,13 +50,18 @@ int xfrm6_transport_finish(struct sk_bufreturn -1 ; } -int xfrm6_rcv (struct sk_buff *skb) +int xfrm6_rcv_tnl (struct sk_buff *skb,struct ip6_tnl *t) {return xfrm6_rcv_spi (skb,skb_network_header (skb)[IP6CB (skb)->nhoff], -0 ); +0 , t); } -EXPORT_SYMBOL (xfrm6_rcv); +EXPORT_SYMBOL (xfrm6_rcv_tnl); +int xfrm6_rcv (struct sk_buff *skb) +{ +return xfrm6_rcv_tnl (skb,NULL ); +} +EXPORT_SYMBOL (xfrm6_rcv);int xfrm6_input_addr (struct sk_buff *skb,xfrm_address_t *daddr,xfrm_address_t *saddr,u8 proto) {
net/ipv6/xfrm6_tunnel.c +1 -1
@@ -239,7 +239,7 @@ static int xfrm6_tunnel_rcv(struct sk_bu__be32 spi; spi =xfrm6_tunnel_spi_lookup (net, (const xfrm_address_t *)&iph->saddr); -return xfrm6_rcv_spi (skb,IPPROTO_IPV6 , spi); +return xfrm6_rcv_spi (skb,IPPROTO_IPV6 , spi,NULL ); } static int xfrm6_tunnel_err(struct sk_buff *skb, struct inet6_skb_parm *opt,
Greg Kroah-Hartman
maintainerPATCH 236/312 ipv4: Fix table id reference in fib_sync_down_addr
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit e0a312629fefa943534fc46f7bfbe6de3fdaf463 upstream.
Hendrik reported routes in the main table using source address are not removed when the address is removed. The problem is that fib_sync_down_addr does not account for devices in the default VRF which are associated with the main table. Fix by updating the table id reference.
--- net/ipv4/fib_semantics.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/ipv4/fib_semantics.c +1 -1
@@ -1358,8 +1358,8 @@ int fib_sync_down_addr(struct net_deviceint ret =0 ;unsigned int hash =fib_laddr_hashfn (local);struct hlist_head *head = &fib_info_laddrhash[hash]; +int tb_id =l3mdev_fib_table (dev) ? :RT_TABLE_MAIN ;struct net *net =dev_net (dev); -int tb_id =l3mdev_fib_table (dev);struct fib_info *fi;if (!fib_info_laddrhash || local ==0 )
Greg Kroah-Hartman
maintainerPATCH 237/312 mlx4: do not call napi_schedule() without care
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit 8cf699ec849f4ca1413cea01289bd7d37dbcc626 upstream.
Disable BH around the call to napi_schedule() to avoid following warning
[ 52.095499] NOHZ: local_softirq_pending 08 [ 52.421291] NOHZ: local_softirq_pending 08 [ 52.608313] NOHZ: local_softirq_pending 08
--- drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
drivers/net/ethernet/mellanox/mlx4/en_netdev.c +4 -1
@@ -1724,8 +1724,11 @@ int mlx4_en_start_port(struct net_device/* Process all completions if exist to prevent * the queues freezing if they are full */ -for (i =0 ; i < priv->rx_ring_num; i++) +for (i =0 ; i < priv->rx_ring_num; i++) { +local_bh_disable ();napi_schedule (&priv->rx_cq[i]->napi); +local_bh_enable (); + }netif_tx_start_all_queues (dev);netif_device_attach (dev);
Greg Kroah-Hartman
maintainerPATCH 238/312 xprtrdma: Fix backchannel allocation of extra rpcrdma_reps
In reply to Greg Kroah-HartmanFrom: Chuck Lever <[email protected]>
commit d698c4a02ee02053bbebe051322ff427a2dad56a upstream.
The backchannel code uses rpcrdma_recv_buffer_put to add new reps to the free rep list. This also decrements rb_recv_count, which spoofs the receive overrun logic in rpcrdma_buffer_get_rep.
Commit 9b06688bc3b9 ("xprtrdma: Fix additional uses of spin_lock_irqsave(rb_lock)") replaced the original open-coded list_add with a call to rpcrdma_recv_buffer_put(), but then a year later, commit 05c974669ece ("xprtrdma: Fix receive buffer accounting") added rep accounting to rpcrdma_recv_buffer_put. It was an oversight to let the backchannel continue to use this function.
The fix this, let's combine the "add to free list" logic with rpcrdma_create_rep.
Also, do not allocate RPCRDMA_MAX_BC_REQUESTS rpcrdma_reps in rpcrdma_buffer_create and then allocate additional rpcrdma_reps in rpcrdma_bc_setup_reps. Allocating the extra reps during backchannel set-up is sufficient.
--- net/sunrpc/xprtrdma/backchannel.c | 12 ++---------- net/sunrpc/xprtrdma/verbs.c | 34 ++++++++++++++++++++-------------- net/sunrpc/xprtrdma/xprt_rdma.h | 2 +- 3 files changed, 23 insertions(+), 25 deletions(-)
net/sunrpc/xprtrdma/backchannel.c +2 -10
@@ -84,21 +84,13 @@ out_fail:static int rpcrdma_bc_setup_reps (struct rpcrdma_xprt *r_xprt,unsigned int count) { -struct rpcrdma_rep *rep;int rc =0 ;while (count--) { - rep =rpcrdma_create_rep (r_xprt); -if (IS_ERR (rep)) { -pr_err ("RPC: %s: reply buffer alloc failed\n" , - __func__); - rc =PTR_ERR (rep); + rc =rpcrdma_create_rep (r_xprt); +if (rc)break ; - } - -rpcrdma_recv_buffer_put (rep); } -return rc; }
net/sunrpc/xprtrdma/verbs.c +20 -14
@@ -911,10 +911,17 @@ rpcrdma_create_req(struct rpcrdma_xprt *return req; } -struct rpcrdma_rep * -rpcrdma_create_rep (struct rpcrdma_xprt *r_xprt) +/** +* rpcrdma_create_rep - Allocate an rpcrdma_rep object +* @r_xprt: controlling transport +* +* Returns 0 on success or a negative errno on failure. +*/ +int +rpcrdma_create_rep (struct rpcrdma_xprt *r_xprt) {struct rpcrdma_create_data_internal *cdata = &r_xprt->rx_data; +struct rpcrdma_buffer *buf = &r_xprt->rx_buf;struct rpcrdma_ia *ia = &r_xprt->rx_ia;struct rpcrdma_rep *rep;int rc;
@@ -934,12 +941,18 @@ rpcrdma_create_rep(struct rpcrdma_xprt * rep->rr_device = ia->ri_device; rep->rr_rxprt = r_xprt;INIT_WORK (&rep->rr_work, rpcrdma_receive_worker); -return rep; + +spin_lock (&buf->rb_lock); +list_add (&rep->rr_list, &buf->rb_recv_bufs); +spin_unlock (&buf->rb_lock); +return 0 ; out_free:kfree (rep); out: -return ERR_PTR (rc); +dprintk ("RPC: %s: reply buffer %d alloc failed\n" , + __func__, rc); +return rc; }int
@@ -975,17 +988,10 @@ rpcrdma_buffer_create(struct rpcrdma_xpr }INIT_LIST_HEAD (&buf->rb_recv_bufs); -for (i =0 ; i < buf->rb_max_requests +2 ; i++) { -struct rpcrdma_rep *rep; - - rep =rpcrdma_create_rep (r_xprt); -if (IS_ERR (rep)) { -dprintk ("RPC: %s: reply buffer %d alloc failed\n" , - __func__, i); - rc =PTR_ERR (rep); +for (i =0 ; i <= buf->rb_max_requests; i++) { + rc =rpcrdma_create_rep (r_xprt); +if (rc) goto out; - } -list_add (&rep->rr_list, &buf->rb_recv_bufs); }return 0 ;
net/sunrpc/xprtrdma/xprt_rdma.h +1 -1
@@ -431,8 +431,8 @@ int rpcrdma_ep_post_recv(struct rpcrdma_* Buffer calls - xprtrdma/verbs.c */ struct rpcrdma_req *rpcrdma_create_req (struct rpcrdma_xprt *); -struct rpcrdma_rep *rpcrdma_create_rep (struct rpcrdma_xprt *);void rpcrdma_destroy_req (struct rpcrdma_ia *,struct rpcrdma_req *); +int rpcrdma_create_rep (struct rpcrdma_xprt *r_xprt);int rpcrdma_buffer_create (struct rpcrdma_xprt *);void rpcrdma_buffer_destroy (struct rpcrdma_buffer *);
Greg Kroah-Hartman
maintainerPATCH 239/312 ALSA: fm801: Initialize chip after IRQ handler is registered
In reply to Greg Kroah-HartmanFrom: Andy Shevchenko <[email protected]>
commit 610e1ae9b533be82b3aa118b907e0a703256913d upstream.
The commit b56fa687e02b ("ALSA: fm801: detect FM-only card earlier") rearranged initialization calls, i.e. it makes snd_fm801_chip_init() to be called before we register interrupt handler and set PCI bus mastering.
Somehow it prevents FM801-AU to work properly. Thus, partially revert initialization order changed by commit mentioned above.
--- sound/pci/fm801.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
sound/pci/fm801.c +2 -2
@@ -1225,8 +1225,6 @@ static int snd_fm801_create(struct snd_c } } -snd_fm801_chip_init (chip); -if ((chip->tea575x_tuner &TUNER_ONLY ) ==0 ) {if (devm_request_irq (&pci->dev, pci->irq, snd_fm801_interrupt,IRQF_SHARED ,KBUILD_MODNAME , chip)) {
@@ -1238,6 +1236,8 @@ static int snd_fm801_create(struct snd_cpci_set_master (pci); } +snd_fm801_chip_init (chip); +if ((err =snd_device_new (card,SNDRV_DEV_LOWLEVEL , chip, &ops)) <0 ) {snd_fm801_free (chip);return err;
Greg Kroah-Hartman
maintainerPATCH 240/312 bonding: fix length of actor system
In reply to Greg Kroah-HartmanFrom: Tobias Jungel <[email protected]>
commit 414dd6fb9a1a1b59983aea7bf0f79f0085ecc5b8 upstream.
The attribute IFLA_BOND_AD_ACTOR_SYSTEM is sent to user space having the length of sizeof(bond->params.ad_actor_system) which is 8 byte. This patch aligns the length to ETH_ALEN to have the same MAC address exposed as using sysfs.
--- drivers/net/bonding/bond_netlink.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/net/bonding/bond_netlink.c +1 -2
@@ -628,8 +628,7 @@ static int bond_fill_info(struct sk_buff goto nla_put_failure;if (nla_put (skb,IFLA_BOND_AD_ACTOR_SYSTEM , -sizeof (bond->params.ad_actor_system), - &bond->params.ad_actor_system)) +ETH_ALEN , &bond->params.ad_actor_system)) goto nla_put_failure; }if (!bond_3ad_get_active_agg_info (bond, &info)) {
Greg Kroah-Hartman
maintainerPATCH 241/312 MIPS: perf: Remove incorrect odd/even counter handling for I6400
In reply to Greg Kroah-HartmanFrom: Marcin Nowakowski <[email protected]>
commit f7a31b5e7874f77464a4eae0a8ba84b9ae0b3a54 upstream.
All performance counters on I6400 (odd and even) are capable of counting any of the available events, so drop current logic of using the extra bit to determine which counter to use.
Patchwork: https://patchwork.linux-mips.org/patch/15991/
--- arch/mips/kernel/perf_event_mipsxx.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
arch/mips/kernel/perf_event_mipsxx.c +5 -1
@@ -1606,7 +1606,6 @@ static const struct mips_perf_event *mip#endif break ;case CPU_P5600 : -case CPU_I6400 :/* 8-bit event numbers */ raw_id = config &0x1ff ; base_id = raw_id &0xff ;
@@ -1619,6 +1618,11 @@ static const struct mips_perf_event *mip raw_event.range =P ;#endif break ; +case CPU_I6400 : +/* 8-bit event numbers */ + base_id = config &0xff ; + raw_event.cntr_mask =CNTR_EVEN |CNTR_ODD ; +break ;case CPU_1004K :if (IS_BOTH_COUNTERS_1004K_EVENT (base_id)) raw_event.cntr_mask =CNTR_EVEN |CNTR_ODD ;
Greg Kroah-Hartman
maintainerPATCH 242/312 Revert "cpufreq: Drop rwsem lock around CPUFREQ_GOV_POLICY_EXIT"
In reply to Greg Kroah-HartmanFrom: Viresh Kumar <[email protected]>
commit 68e80dae09033d778b98dc88e5bfe8fdade188e5 upstream.
Earlier, when the struct freq-attr was used to represent governor attributes, the standard cpufreq show/store sysfs attribute callbacks were applied to the governor tunable attributes and they always acquire the policy->rwsem lock before carrying out the operation. That could have resulted in an ABBA deadlock if governor tunable attributes are removed under policy->rwsem while one of them is being accessed concurrently (if sysfs attributes removal wins the race, it will wait for the access to complete with policy->rwsem held while the attribute callback will block on policy->rwsem indefinitely).
We attempted to address this issue by dropping policy->rwsem around governor tunable attributes removal (that is, around invocations of the ->governor callback with the event arg equal to CPUFREQ_GOV_POLICY_EXIT) in cpufreq_set_policy(), but that opened up race conditions that had not been possible with policy->rwsem held all the time.
The previous commit, "cpufreq: governor: New sysfs show/store callbacks for governor tunables", fixed the original ABBA deadlock by adding new governor specific show/store callbacks.
We don't have to drop rwsem around invocations of governor event CPUFREQ_GOV_POLICY_EXIT anymore, and original fix can be reverted now.
--- drivers/cpufreq/cpufreq.c | 5 ----- include/linux/cpufreq.h | 4 ---- 2 files changed, 9 deletions(-)
drivers/cpufreq/cpufreq.c +0 -5
@@ -2171,10 +2171,7 @@ static int cpufreq_set_policy(struct cpureturn ret; } -up_write (&policy->rwsem); ret =__cpufreq_governor (policy,CPUFREQ_GOV_POLICY_EXIT ); -down_write (&policy->rwsem); -if (ret) {pr_err ("%s: Failed to Exit Governor: %s (%d)\n" , __func__, old_gov->name, ret);
@@ -2190,9 +2187,7 @@ static int cpufreq_set_policy(struct cpuif (!ret) goto out; -up_write (&policy->rwsem);__cpufreq_governor (policy,CPUFREQ_GOV_POLICY_EXIT ); -down_write (&policy->rwsem); }/* new governor failed, so re-start old one */
include/linux/cpufreq.h +0 -4
@@ -100,10 +100,6 @@ struct cpufreq_policy {* - Any routine that will write to the policy structure and/or may take away * the policy altogether (eg. CPU hotplug), will hold this lock in write * mode before doing so. -* -* Additional rules: -* - Lock should not be held across -* __cpufreq_governor(data, CPUFREQ_GOV_POLICY_EXIT); */ struct rw_semaphore rwsem;
Greg Kroah-Hartman
maintainerPATCH 243/312 net: dsa: mv88e6xxx: unlock DSA and CPU ports
In reply to Greg Kroah-HartmanFrom: Vivien Didelot <[email protected]>
commit 65fa40276ac1318e75264e5a204547b57c2cf397 upstream.
Locking a port generates an hardware interrupt when a new SA address is received. This enables CPU directed learning, which is needed for 802.1X MAC authentication.
To disable automatic learning on a port, the only configuration needed is to set its Port Association Vector to all zero.
Clear PAV when SA learning should be disabled instead of locking a port.
--- drivers/net/dsa/mv88e6xxx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/dsa/mv88e6xxx.c +1 -1
@@ -2066,7 +2066,7 @@ static int mv88e6xxx_setup_port(struct d reg =1 << port;/* Disable learning for DSA and CPU ports */ if (dsa_is_cpu_port (ds, port) ||dsa_is_dsa_port (ds, port)) - reg =PORT_ASSOC_VECTOR_LOCKED_PORT ; + reg =0 ; ret =_mv88e6xxx_reg_write (ds,REG_PORT (port),PORT_ASSOC_VECTOR , reg);if (ret)
Greg Kroah-Hartman
maintainerPATCH 244/312 gfs2: fix flock panic issue
In reply to Greg Kroah-HartmanFrom: Junxiao Bi <[email protected]>
commit a93a99838248bdab49db2eaac00236847670bc7f upstream.
Commit 4f6563677ae8 ("Move locks API users to locks_lock_inode_wait()") moved flock/posix lock identify code to locks_lock_inode_wait(), but missed to set fl_flags to FL_FLOCK which will cause kernel panic in locks_lock_inode_wait().
--- fs/gfs2/file.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
fs/gfs2/file.c +4 -1
@@ -1035,7 +1035,10 @@ static int do_flock(struct file *file, iif (fl_gh->gh_state == state) goto out;locks_lock_file_wait (file, - &(struct file_lock ){.fl_type =F_UNLCK }); + &(struct file_lock ) { + .fl_type =F_UNLCK , + .fl_flags =FL_FLOCK + });gfs2_glock_dq (fl_gh);gfs2_holder_reinit (state, flags, fl_gh); }else {
Greg Kroah-Hartman
maintainerPATCH 245/312 blk-mq: fix undefined behaviour in order_to_size()
In reply to Greg Kroah-HartmanFrom: Bartlomiej Zolnierkiewicz <[email protected]>
commit b3a834b1596ac668df206aa2bb1f191c31f5f5e4 upstream.
When this_order variable in blk_mq_init_rq_map() becomes zero the code incorrectly decrements the variable and passes the result to order_to_size() helper causing undefined behaviour:
UBSAN: Undefined behaviour in block/blk-mq.c:1459:27 shift exponent 4294967295 is too large for 32-bit type 'unsigned int' CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.6.0-rc6-00072-g33656a1 #22
Fix the code by checking this_order variable for not having the zero value first.
--- block/blk-mq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
block/blk-mq.c +1 -1
@@ -1491,7 +1491,7 @@ static struct blk_mq_tags *blk_mq_init_rint to_do;void *p; -while (left <order_to_size (this_order -1 ) && this_order) +while (this_order && left <order_to_size (this_order -1 )) this_order--; do {
Greg Kroah-Hartman
maintainerPATCH 246/312 dm: fix second blk_delay_queue() parameter to be in msec units not jiffies
In reply to Greg Kroah-HartmanFrom: Tahsin Erdogan <[email protected]>
commit bd9f55ea1cf6e14eb054b06ea877d2d1fa339514 upstream.
Commit d548b34b062 ("dm: reduce the queue delay used in dm_request_fn from 100ms to 10ms") always intended the value to be 10 msecs -- it just expressed it in jiffies because earlier commit 7eaceaccab ("block:
remove per-queue plugging") did.
--- drivers/md/dm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/md/dm.c +1 -1
@@ -2192,7 +2192,7 @@ static void dm_request_fn(struct request goto out; delay_and_out: -blk_delay_queue (q,HZ /100 ); +blk_delay_queue (q,10 ); out:dm_put_live_table (md, srcu_idx); }
Greg Kroah-Hartman
maintainerPATCH 247/312 dmaengine: edma: Add probe callback to edma_tptc_driver
In reply to Greg Kroah-HartmanFrom: Peter Ujfalusi <[email protected]>
commit 4fa2d09c1ae879c2ee2760ab419a4f97026dd97b upstream.
Due to changes in device and platform code drivers w/o probe will fail to load. This means that the devices for eDMA TPTCs are goign to be without driver and omap hwmod code will turn them off after the kernel finished loading:
[ 3.015900] platform 49800000.tptc: omap_device_late_idle: enabled but no driver. Idling [ 3.024671] platform 49a00000.tptc: omap_device_late_idle: enabled but no driver. Idling
This will prevent eDMA to work since the TPTCs are not enabled.
--- drivers/dma/edma.c | 6 ++++++ 1 file changed, 6 insertions(+)
drivers/dma/edma.c +6 -0
@@ -2439,7 +2439,13 @@ static struct platform_driver edma_drive }, }; +static int edma_tptc_probe (struct platform_device *pdev) +{ +return 0 ; +} +static struct platform_driver edma_tptc_driver = { + .probe = edma_tptc_probe, .driver = { .name ="edma3-tptc" , .of_match_table = edma_tptc_of_ids,
Greg Kroah-Hartman
maintainerPATCH 248/312 spi: rockchip: modify DMA max burst to 1
In reply to Greg Kroah-HartmanFrom: Addy Ke <[email protected]>
commit 80abf8880cc6e1594c11b7c417f22dde60e25312 upstream.
Generic dma controller on Rockchips' platform cannot support DMAFLUSHP instruction which make dma to flush the req of non-aligned or non-multiple of what we need. That will cause an unrecoverable dma bus error. The saftest way is to set dma max burst to 1.
--- drivers/spi/spi-rockchip.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)
drivers/spi/spi-rockchip.c +10 -2
@@ -199,6 +199,7 @@ struct rockchip_spi {struct sg_table rx_sg;struct rockchip_spi_dma_data dma_rx;struct rockchip_spi_dma_data dma_tx; +struct dma_slave_caps dma_caps; };static inline void spi_enable_chip (struct rockchip_spi *rs,int enable)
@@ -457,7 +458,10 @@ static void rockchip_spi_prepare_dma(str rxconf.direction = rs->dma_rx.direction; rxconf.src_addr = rs->dma_rx.addr; rxconf.src_addr_width = rs->n_bytes; - rxconf.src_maxburst = rs->n_bytes; +if (rs->dma_caps.max_burst >4 ) + rxconf.src_maxburst =4 ; +else + rxconf.src_maxburst =1 ;dmaengine_slave_config (rs->dma_rx.ch, &rxconf); rxdesc = dmaengine_prep_slave_sg(
@@ -474,7 +478,10 @@ static void rockchip_spi_prepare_dma(str txconf.direction = rs->dma_tx.direction; txconf.dst_addr = rs->dma_tx.addr; txconf.dst_addr_width = rs->n_bytes; - txconf.dst_maxburst = rs->n_bytes; +if (rs->dma_caps.max_burst >4 ) + txconf.dst_maxburst =4 ; +else + txconf.dst_maxburst =1 ;dmaengine_slave_config (rs->dma_tx.ch, &txconf); txdesc = dmaengine_prep_slave_sg(
@@ -738,6 +745,7 @@ static int rockchip_spi_probe(struct pla }if (rs->dma_tx.ch && rs->dma_rx.ch) { +dma_get_slave_caps (rs->dma_rx.ch, &(rs->dma_caps)); rs->dma_tx.addr = (dma_addr_t)(mem->start +ROCKCHIP_SPI_TXDR ); rs->dma_rx.addr = (dma_addr_t)(mem->start +ROCKCHIP_SPI_RXDR ); rs->dma_tx.direction =DMA_MEM_TO_DEV ;
Greg Kroah-Hartman
maintainerPATCH 249/312 openvswitch: update checksum in {push,pop}_mpls
In reply to Greg Kroah-HartmanFrom: Simon Horman <[email protected]>
commit bc7cc5999fd392cc799630d7e375b2f4e29cc398 upstream.
In the case of CHECKSUM_COMPLETE the skb checksum should be updated in {push,pop}_mpls() as they the type in the ethernet header.
As suggested by Pravin Shelar.
--- net/openvswitch/actions.c | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-)
net/openvswitch/actions.c +15 -5
@@ -137,11 +137,23 @@ static bool is_flow_key_valid(const strureturn !!key->eth.type; } +static void update_ethertype (struct sk_buff *skb,struct ethhdr *hdr, +__be16 ethertype) +{ +if (skb->ip_summed ==CHECKSUM_COMPLETE ) { +__be16 diff[] = { ~(hdr->h_proto), ethertype }; + + skb->csum = ~csum_partial ((char *)diff,sizeof (diff), + ~skb->csum); + } + + hdr->h_proto = ethertype; +} +static int push_mpls (struct sk_buff *skb,struct sw_flow_key *key,const struct ovs_action_push_mpls *mpls) {__be32 *new_mpls_lse; -struct ethhdr *hdr;/* Networking stack do not allow simultaneous Tunnel and MPLS GSO. */ if (skb->encapsulation)
@@ -160,9 +172,7 @@ static int push_mpls(struct sk_buff *skbskb_postpush_rcsum (skb, new_mpls_lse,MPLS_HLEN ); - hdr =eth_hdr (skb); - hdr->h_proto = mpls->mpls_ethertype; - +update_ethertype (skb,eth_hdr (skb), mpls->mpls_ethertype);if (!skb->inner_protocol)skb_set_inner_protocol (skb, skb->protocol); skb->protocol = mpls->mpls_ethertype;
@@ -193,7 +203,7 @@ static int pop_mpls(struct sk_buff *skb,* field correctly in the presence of VLAN tags. */ hdr = (struct ethhdr *)(skb_mpls_header (skb) -ETH_HLEN ); - hdr->h_proto = ethertype; +update_ethertype (skb, hdr, ethertype);if (eth_p_mpls (skb->protocol)) skb->protocol = ethertype;
Greg Kroah-Hartman
maintainerPATCH 250/312 cxgb4/cxgb4vf: Fixes regression in perf when tx vlan offload is disabled
In reply to Greg Kroah-HartmanFrom: Hariprasad Shenai <[email protected]>
commit 8d09e6b8b9c9969ac59496dc21e10b67fe727e7e upstream.
The commit 637d3e997351 ("cxgb4: Discard the packet if the length is greater than mtu") introduced a regression in the VLAN interface performance when Tx VLAN offload is disabled.
Check if skb is tagged, regardless of whether it is hardware accelerated or not. Presently we were checking only for hardware acclereated one, which caused performance to drop to ~0.17Mbps on a 10GbE adapter for VLAN interface, when tx vlan offload is turned off using ethtool. The ethernet head length calculation was going wrong in this case, and driver ended up dropping packets.
--- drivers/net/ethernet/chelsio/cxgb4/sge.c | 2 +- drivers/net/ethernet/chelsio/cxgb4vf/sge.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/chelsio/cxgb4/sge.c +1 -1
@@ -1165,7 +1165,7 @@ out_free: dev_kfree_skb_any(skb);/* Discard the packet if the length is greater than mtu */ max_pkt_len =ETH_HLEN + dev->mtu; -if (skb_vlan_tag_present (skb)) +if (skb_vlan_tagged (skb)) max_pkt_len +=VLAN_HLEN ;if (!skb_shinfo (skb)->gso_size && (unlikely (skb->len > max_pkt_len))) goto out_free;
drivers/net/ethernet/chelsio/cxgb4vf/sge.c +1 -1
@@ -1188,7 +1188,7 @@ int t4vf_eth_xmit(struct sk_buff *skb, s/* Discard the packet if the length is greater than mtu */ max_pkt_len =ETH_HLEN + dev->mtu; -if (skb_vlan_tag_present (skb)) +if (skb_vlan_tagged (skb)) max_pkt_len +=VLAN_HLEN ;if (!skb_shinfo (skb)->gso_size && (unlikely (skb->len > max_pkt_len))) goto out_free;
Greg Kroah-Hartman
maintainerPATCH 251/312 net: bcmgenet: fix skb_len in bcmgenet_xmit_single()
In reply to Greg Kroah-HartmanFrom: Petri Gynther <[email protected]>
commit 7dd399130efb5a454daf24075b7563d197114e39 upstream.
skb_len needs to be skb_headlen(skb) in bcmgenet_xmit_single().
Fragmented skbs can have only Ethernet + IP + TCP headers (14+20+20=54 bytes) in the linear buffer, followed by the rest in fragments. Bumping skb_len to ETH_ZLEN would be incorrect for this case, as it would introduce garbage between TCP header and the fragment data.
This also works with regular/non-fragmented small packets < ETH_ZLEN bytes. Successfully tested this on GENETv3 with 42-byte ARP frames.
For testing, I used:
ethtool -K eth0 tx-checksum-ipv4 off
ethtool -K eth0 tx-checksum-ipv6 off
echo 0 > /proc/sys/net/ipv4/tcp_timestamps
--- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/broadcom/genet/bcmgenet.c +1 -1
@@ -1346,7 +1346,7 @@ static int bcmgenet_xmit_single(struct n tx_cb_ptr->skb = skb; - skb_len =skb_headlen (skb) <ETH_ZLEN ?ETH_ZLEN :skb_headlen (skb); + skb_len =skb_headlen (skb); mapping =dma_map_single (kdev, skb->data, skb_len,DMA_TO_DEVICE ); ret =dma_mapping_error (kdev, mapping);
Greg Kroah-Hartman
maintainerPATCH 252/312 net: bcmgenet: device stats are unsigned long
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit 6517eb59b03965689e6bb16bb2d480096b3ef95d upstream.
On 64bit kernels, device stats are 64bit wide, not 32bit.
--- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
drivers/net/ethernet/broadcom/genet/bcmgenet.c +5 -1
@@ -927,7 +927,11 @@ static void bcmgenet_get_ethtool_stats(selse p = (char *)priv; p += s->stat_offset; - data[i] = *(u32 *)p; +if (sizeof (unsigned long ) !=sizeof (u32) && + s->stat_sizeof ==sizeof (unsigned long )) + data[i] = *(unsigned long *)p; +else + data[i] = *(u32 *)p; } }
Greg Kroah-Hartman
maintainerPATCH 253/312 ovs/gre: fix rtnl notifications on iface deletion
In reply to Greg Kroah-HartmanFrom: Nicolas Dichtel <[email protected]>
commit da6f1da819d4b9c081a477dec74dc468a0b44290 upstream.
The function gretap_fb_dev_create() (only used by ovs) never calls rtnl_configure_link(). The consequence is that dev->rtnl_link_state is never set to RTNL_LINK_INITIALIZED.
During the deletion phase, the function rollback_registered_many() sends a RTM_DELLINK only if dev->rtnl_link_state is set to RTNL_LINK_INITIALIZED.
--- net/ipv4/ip_gre.c | 4 ++++ 1 file changed, 4 insertions(+)
net/ipv4/ip_gre.c +4 -0
@@ -1265,6 +1265,10 @@ struct net_device *gretap_fb_dev_create(if (err) goto out; + err =rtnl_configure_link (dev,NULL ); +if (err <0 ) + goto out; +return dev; out:ip_tunnel_dellink (dev, &list_kill);
Greg Kroah-Hartman
maintainerPATCH 254/312 gre: do not assign header_ops in collect metadata mode
In reply to Greg Kroah-HartmanFrom: Jiri Benc <[email protected]>
commit a64b04d86d14c81f50f68e102f79ef301e3d0a0e upstream.
In ipgre mode (i.e. not gretap) with collect metadata flag set, the tunnel is incorrectly assumed to be mGRE in NBMA mode (see commit 6a5f44d7a048c). This is not the case, we're controlling the encapsulation addresses by lwtunnel metadata. And anyway, assigning dev->header_ops in collect metadata mode does not make sense.
Although it would be more user firendly to reject requests that specify both the collect metadata flag and a remote/local IP address, this would break current users of gretap or introduce ugly code and differences in handling ipgre and gretap configuration. Keep the current behavior of remote/local IP address being ignored in such case.
v3: Back to v1, added explanation paragraph.
v2: Reject configuration specifying both remote/local address and collect
metadata flag.
--- net/ipv4/ip_gre.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
net/ipv4/ip_gre.c +3 -2
@@ -882,7 +882,7 @@ static int ipgre_tunnel_init(struct net_netif_keep_dst (dev); dev->addr_len =4 ; -if (iph->daddr) { +if (iph->daddr && !tunnel->collect_md) {#ifdef CONFIG_NET_IPGRE_BROADCAST if (ipv4_is_multicast (iph->daddr)) {if (!iph->saddr)
@@ -891,8 +891,9 @@ static int ipgre_tunnel_init(struct net_ dev->header_ops = &ipgre_header_ops; }#endif - }else + }else if (!tunnel->collect_md) { dev->header_ops = &ipgre_header_ops; + }return ip_tunnel_init (dev); }
Greg Kroah-Hartman
maintainerPATCH 255/312 gre: build header correctly for collect metadata tunnels
In reply to Greg Kroah-HartmanFrom: Jiri Benc <[email protected]>
commit 2090714e1d6e80979dd6926be22b0de9ca432273 upstream.
In ipgre (i.e. not gretap) + collect metadata mode, the skb was assumed to contain Ethernet header and was encapsulated as ETH_P_TEB. This is not the case, the interface is ARPHRD_IPGRE and the protocol to be used for encapsulation is skb->protocol.
--- net/ipv4/ip_gre.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)
net/ipv4/ip_gre.c +5 -4
@@ -520,7 +520,8 @@ static struct rtable *gre_get_rt(structreturn ip_route_output_key (net, fl); } -static void gre_fb_xmit (struct sk_buff *skb,struct net_device *dev) +static void gre_fb_xmit (struct sk_buff *skb,struct net_device *dev, +__be16 proto) {struct ip_tunnel_info *tun_info;const struct ip_tunnel_key *key;
@@ -563,7 +564,7 @@ static void gre_fb_xmit(struct sk_buff * } flags = tun_info->key.tun_flags & (TUNNEL_CSUM |TUNNEL_KEY ); -build_header (skb, tunnel_hlen, flags,htons (ETH_P_TEB ), +build_header (skb, tunnel_hlen, flags, proto,tunnel_id_to_key (tun_info->key.tun_id),0 ); df = key->tun_flags &TUNNEL_DONT_FRAGMENT ?htons (IP_DF ) :0 ;
@@ -605,7 +606,7 @@ static netdev_tx_t ipgre_xmit(struct sk_const struct iphdr *tnl_params;if (tunnel->collect_md) { -gre_fb_xmit (skb, dev); +gre_fb_xmit (skb, dev, skb->protocol);return NETDEV_TX_OK ; }
@@ -649,7 +650,7 @@ static netdev_tx_t gre_tap_xmit(struct sstruct ip_tunnel *tunnel =netdev_priv (dev);if (tunnel->collect_md) { -gre_fb_xmit (skb, dev); +gre_fb_xmit (skb, dev,htons (ETH_P_TEB ));return NETDEV_TX_OK ; }
Greg Kroah-Hartman
maintainerPATCH 256/312 gre: reject GUE and FOU in collect metadata mode
In reply to Greg Kroah-HartmanFrom: Jiri Benc <[email protected]>
commit 946b636f1730c64e05ff7fe8cf7136422fa8ea70 upstream.
The collect metadata mode does not support GUE nor FOU. This might be implemented later; until then, we should reject such config.
I think this is okay to be changed. It's unlikely anyone has such configuration (as it doesn't work anyway) and we may need a way to distinguish whether it's supported or not by the kernel later.
For backwards compatibility with iproute2, it's not possible to just check the attribute presence (iproute2 always includes the attribute), the actual value has to be checked, too.
--- net/ipv4/ip_gre.c | 5 +++++ 1 file changed, 5 insertions(+)
net/ipv4/ip_gre.c +5 -0
@@ -937,6 +937,11 @@ static int ipgre_tunnel_validate(structif (flags & (GRE_VERSION |GRE_ROUTING ))return -EINVAL ; +if (data[IFLA_GRE_COLLECT_METADATA ] && + data[IFLA_GRE_ENCAP_TYPE ] && +nla_get_u16 (data[IFLA_GRE_ENCAP_TYPE ]) !=TUNNEL_ENCAP_NONE ) +return -EINVAL ; +return 0 ; }
Greg Kroah-Hartman
maintainerPATCH 257/312 sfc: fix potential stack corruption from running past stat bitmask
In reply to Greg Kroah-HartmanFrom: Andrew Rybchenko <[email protected]>
commit e70c70c38d7a5ced76fc8b1c4a7ccee76e9c2911 upstream.
On 32-bit systems, mask is only an array of 3 longs, not 4, so don't try to write to mask[3].
Also include build-time checks in case the size of the bitmask changes.
--- drivers/net/ethernet/sfc/ef10.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/ethernet/sfc/ef10.c +2 -1
@@ -1304,13 +1304,14 @@ static void efx_ef10_get_stat_mask(struc }#if BITS_PER_LONG ==64 +BUILD_BUG_ON (BITS_TO_LONGS (EF10_STAT_COUNT ) !=2 ); mask[0 ] = raw_mask[0 ]; mask[1 ] = raw_mask[1 ];#else +BUILD_BUG_ON (BITS_TO_LONGS (EF10_STAT_COUNT ) !=3 ); mask[0 ] = raw_mask[0 ] &0xffffffff ; mask[1 ] = raw_mask[0 ] >>32 ; mask[2 ] = raw_mask[1 ] &0xffffffff ; - mask[3 ] = raw_mask[1 ] >>32 ;#endif }
Greg Kroah-Hartman
maintainerPATCH 258/312 sfc: clear napi_hash state when copying channels
In reply to Greg Kroah-HartmanFrom: Bert Kenward <[email protected]>
commit 46d054f8f540612f09987a53154aa39ae15f2e4c upstream.
efx_copy_channel() doesn't correctly clear the napi_hash related state. This means that when napi_hash_add is called for that channel nothing is done, and we are left with a copy of the napi_hash_node from the old channel. When we later call napi_hash_del() on this channel we have a stale napi_hash_node.
Corruption is only seen when there are multiple entries in one of the napi_hash lists. This is made more likely by having a very large number of channels. Testing was carried out with 512 channels - 32 channels on each of 16 ports.
This failure typically appears as protection faults within napi_by_id() or napi_hash_add(). efx_copy_channel() is only used when tx or rx ring sizes are changed (ethtool -G).
--- drivers/net/ethernet/sfc/efx.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/net/ethernet/sfc/efx.c +3 -0
@@ -479,6 +479,9 @@ efx_copy_channel(const struct efx_channe *channel = *old_channel; channel->napi_dev =NULL ; +INIT_HLIST_NODE (&channel->napi_str.napi_hash_node); + channel->napi_str.napi_id =0 ; + channel->napi_str.state =0 ;memset (&channel->eventq,0 ,sizeof (channel->eventq));for (j =0 ; j <EFX_TXQ_TYPES ; j++) {
Greg Kroah-Hartman
maintainerPATCH 259/312 net: bcmsysport: Device stats are unsigned long
In reply to Greg Kroah-HartmanFrom: Florian Fainelli <[email protected]>
commit 016eb55157166132b094e53434748cae35e18455 upstream.
On 64bits kernels, device stats are 64bits wide, not 32bits.
--- drivers/net/ethernet/broadcom/bcmsysport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/broadcom/bcmsysport.c +1 -1
@@ -396,7 +396,7 @@ static void bcm_sysport_get_stats(structelse p = (char *)priv; p += s->stat_offset; - data[i] = *(u32 *)p; + data[i] = *(unsigned long *)p; } }
Greg Kroah-Hartman
maintainerPATCH 260/312 cxgbi: fix uninitialized flowi6
In reply to Greg Kroah-HartmanFrom: Jiri Benc <[email protected]>
commit 3d6d30d60abb19ba9a20e53ce65b18a9c148fcd1 upstream.
ip6_route_output looks into different fields in the passed flowi6 structure, yet cxgbi passes garbage in nearly all those fields. Zero the structure out first.
--- drivers/scsi/cxgbi/libcxgbi.c | 1 + 1 file changed, 1 insertion(+)
drivers/scsi/cxgbi/libcxgbi.c +1 -0
@@ -692,6 +692,7 @@ static struct rt6_info *find_route_ipv6( {struct flowi6 fl; +memset (&fl,0 ,sizeof (fl));if (saddr)memcpy (&fl.saddr, saddr,sizeof (struct in6_addr ));if (daddr)
Greg Kroah-Hartman
maintainerPATCH 261/312 net: macb: add missing free_netdev() on error in macb_probe()
In reply to Greg Kroah-HartmanFrom: Wei Yongjun <[email protected]>
commit b22ae0b4d9669495158a7fa0fd027bd0fcd8896e upstream.
Add the missing free_netdev() before return from function macb_probe() in the platform_get_irq() error handling case.
--- drivers/net/ethernet/cadence/macb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/cadence/macb.c +1 -1
@@ -2904,7 +2904,7 @@ static int macb_probe(struct platform_de dev->irq =platform_get_irq (pdev,0 );if (dev->irq <0 ) { err = dev->irq; - goto err_disable_clocks; + goto err_out_free_netdev; } mac =of_get_mac_address (np);
Greg Kroah-Hartman
maintainerPATCH 262/312 macvtap: segmented packet is consumed
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit be0bd3160165e42783d8215f426e41c07179c08a upstream.
If GSO packet is segmented and its segments are properly queued, we call consume_skb() instead of kfree_skb() to be drop monitor friendly.
--- drivers/net/macvtap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/macvtap.c +1 -1
@@ -373,7 +373,7 @@ static rx_handler_result_t macvtap_handl goto wake_up; } -kfree_skb (skb); +consume_skb (skb);while (segs) {struct sk_buff *nskb = segs->next;
Greg Kroah-Hartman
maintainerPATCH 263/312 tipc: fix the error handling in tipc_udp_enable()
In reply to Greg Kroah-HartmanFrom: Wei Yongjun <[email protected]>
commit a5de125dd46c851fc962806135953c1bd0a0f0df upstream.
Fix to return a negative error code in enable_mcast() error handling case, and release udp socket when necessary.
--- net/tipc/udp_media.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
net/tipc/udp_media.c +4 -1
@@ -405,10 +405,13 @@ static int tipc_udp_enable(struct net *n tuncfg.encap_destroy =NULL ;setup_udp_tunnel_sock (net, ub->ubsock, &tuncfg); -if (enable_mcast (ub, remote)) + err =enable_mcast (ub, remote); +if (err) goto err;return 0 ; err: +if (ub->ubsock) +udp_tunnel_sock_release (ub->ubsock);kfree (ub);return err; }
Greg Kroah-Hartman
maintainerPATCH 264/312 net: icmp6_send should use dst dev to determine L3 domain
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit 5d41ce29e3b91ef305f88d23f72b3359de329cec upstream.
icmp6_send is called in response to some event. The skb may not have the device set (skb->dev is NULL), but it is expected to have a dst set. Update icmp6_send to use the dst on the skb to determine L3 domain.
--- net/ipv6/icmp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/ipv6/icmp.c +1 -1
@@ -446,7 +446,7 @@ static void icmp6_send(struct sk_buff *sif (__ipv6_addr_needs_scope_id (addr_type)) iif = skb->dev->ifindex;else - iif =l3mdev_master_ifindex (skb->dev); + iif =l3mdev_master_ifindex (skb_dst (skb)->dev);/* * Must not send error if the source does not uniquely
Greg Kroah-Hartman
maintainerPATCH 265/312 et131x: Fix logical vs bitwise check in et131x_tx_timeout()
In reply to Greg Kroah-HartmanFrom: Florian Fainelli <[email protected]>
commit de702da7a823ab0c4a1e53ed79a2695f0d453855 upstream.
We should be using a logical check here instead of a bitwise operation to check if the device is closed already in et131x_tx_timeout().
--- drivers/net/ethernet/agere/et131x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/agere/et131x.c +1 -1
@@ -3854,7 +3854,7 @@ static void et131x_tx_timeout(struct netunsigned long flags;/* If the device is closed, ignore the timeout */ -if (~(adapter->flags &FMP_ADAPTER_INTERRUPT_IN_USE )) +if (!(adapter->flags &FMP_ADAPTER_INTERRUPT_IN_USE ))return ;/* Any nonrecoverable hardware error?
Greg Kroah-Hartman
maintainerPATCH 266/312 net: ethernet: stmmac: dwmac-sti: fix probe error path
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 0a9e22715ee384cf2a714c28f24ce8881b9fd815 upstream.
Make sure to disable clocks before returning on late probe errors.
probe function")
--- drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)
drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c +10 -1
@@ -365,7 +365,16 @@ static int sti_dwmac_probe(struct platfoif (ret)return ret; -return stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); + ret =stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); +if (ret) + goto err_dwmac_exit; + +return 0 ; + +err_dwmac_exit: +sti_dwmac_exit (pdev, plat_dat->bsp_priv); + +return ret; }static const struct sti_dwmac_of_data stih4xx_dwmac_data = {
Greg Kroah-Hartman
maintainerPATCH 267/312 rtnl: reset calcit fptr in rtnl_unregister()
In reply to Greg Kroah-HartmanFrom: Mathias Krause <[email protected]>
commit f567e950bf51290755a2539ff2aaef4c26f735d3 upstream.
To avoid having dangling function pointers left behind, reset calcit in rtnl_unregister(), too.
This is no issue so far, as only the rtnl core registers a netlink handler with a calcit hook which won't be unregistered, but may become one if new code makes use of the calcit hook.
--- net/core/rtnetlink.c | 1 + 1 file changed, 1 insertion(+)
net/core/rtnetlink.c +1 -0
@@ -253,6 +253,7 @@ int rtnl_unregister(int protocol, int ms rtnl_msg_handlers[protocol][msgindex].doit =NULL ; rtnl_msg_handlers[protocol][msgindex].dumpit =NULL ; + rtnl_msg_handlers[protocol][msgindex].calcit =NULL ;return 0 ; }
Greg Kroah-Hartman
maintainerPATCH 268/312 net: ethernet: stmmac: dwmac-rk: fix probe error path
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 2d222656db08b8eef3b53b56cf1ce4a90fe8cd78 upstream.
Make sure to disable runtime PM, power down the PHY, and disable clocks before returning on late probe errors.
--- drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)
drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c +10 -1
@@ -600,7 +600,16 @@ static int rk_gmac_probe(struct platformif (ret)return ret; -return stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); + ret =stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); +if (ret) + goto err_gmac_exit; + +return 0 ; + +err_gmac_exit: +rk_gmac_exit (pdev, plat_dat->bsp_priv); + +return ret; }static const struct of_device_id rk_gmac_dwmac_match[] = {
Greg Kroah-Hartman
maintainerPATCH 269/312 fq_codel: return non zero qlen in class dumps
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit aafddbf0cffeb790f919436285328c762279b5d4 upstream.
We properly scan the flow list to count number of packets, but John passed 0 to gnet_stats_copy_queue() so we report a zero value to user space instead of the result.
--- net/sched/sch_fq_codel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
net/sched/sch_fq_codel.c +1 -1
@@ -588,7 +588,7 @@ static int fq_codel_dump_class_stats(str qs.backlog = q->backlogs[idx]; qs.drops = flow->dropped; } -if (gnet_stats_copy_queue (d,NULL , &qs,0 ) <0 ) +if (gnet_stats_copy_queue (d,NULL , &qs, qs.qlen) <0 )return -1 ;if (idx < q->flows_cnt)return gnet_stats_copy_app (d, &xstats,sizeof (xstats));
Greg Kroah-Hartman
maintainerPATCH 270/312 net: ethernet: stmmac: dwmac-generic: fix probe error path
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 939b20022765bc338b0f72cbf1eed60a907398d7 upstream.
Make sure to call any exit() callback to undo the effect of init() before returning on late probe errors.
--- drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-)
drivers/net/ethernet/stmicro/stmmac/dwmac-generic.c +11 -1
@@ -53,7 +53,17 @@ static int dwmac_generic_probe(struct plreturn ret; } -return stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); + ret =stmmac_dvr_probe (&pdev->dev, plat_dat, &stmmac_res); +if (ret) + goto err_exit; + +return 0 ; + +err_exit: +if (plat_dat->exit) + plat_dat->exit (pdev, plat_dat->bsp_priv); + +return ret; }static const struct of_device_id dwmac_generic_match[] = {
Greg Kroah-Hartman
maintainerPATCH 271/312 ovs/geneve: fix rtnl notifications on iface deletion
In reply to Greg Kroah-HartmanFrom: Nicolas Dichtel <[email protected]>
commit 41009481b690493c169ce85f591b9d32c6fd9422 upstream.
The function geneve_dev_create_fb() (only used by ovs) never calls rtnl_configure_link(). The consequence is that dev->rtnl_link_state is never set to RTNL_LINK_INITIALIZED.
During the deletion phase, the function rollback_registered_many() sends a RTM_DELLINK only if dev->rtnl_link_state is set to RTNL_LINK_INITIALIZED.
--- drivers/net/geneve.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/net/geneve.c +4 -0
@@ -1363,6 +1363,10 @@ struct net_device *geneve_dev_create_fb(if (err) goto err; + err =rtnl_configure_link (dev,NULL ); +if (err <0 ) + goto err; +return dev; err:
Greg Kroah-Hartman
maintainerPATCH 272/312 bnxt: add a missing rcu synchronization
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit e5f6f564fd191d365fcd775c06a732a488205588 upstream.
Add a missing synchronize_net() call to avoid potential use after free, since we explicitly call napi_hash_del() to factorize the RCU grace period.
--- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/net/ethernet/broadcom/bnxt/bnxt.c +4 -0
@@ -4250,6 +4250,10 @@ static void bnxt_del_napi(struct bnxt *bnapi_hash_del (&bnapi->napi);netif_napi_del (&bnapi->napi); } +/* We called napi_hash_del() before netif_napi_del(), we need +* to respect an RCU grace period before freeing napi structures. +*/ +synchronize_net (); }static void bnxt_init_napi (struct bnxt *bp)
Greg Kroah-Hartman
maintainerPATCH 273/312 qdisc: fix a module refcount leak in qdisc_create_dflt()
In reply to Greg Kroah-HartmanFrom: Eric Dumazet <[email protected]>
commit 166ee5b87866de07a3e56c1b757f2b5cabba72a5 upstream.
Should qdisc_alloc() fail, we must release the module refcount we got right before.
--- net/sched/sch_generic.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)
net/sched/sch_generic.c +5 -4
@@ -626,18 +626,19 @@ struct Qdisc *qdisc_create_dflt(struct nstruct Qdisc *sch;if (!try_module_get (ops->owner)) - goto errout; +return NULL ; sch =qdisc_alloc (dev_queue, ops); -if (IS_ERR (sch)) - goto errout; +if (IS_ERR (sch)) { +module_put (ops->owner); +return NULL ; + } sch->parent = parentid;if (!ops->init || ops->init (sch,NULL ) ==0 )return sch;qdisc_destroy (sch); -errout:return NULL ; }EXPORT_SYMBOL (qdisc_create_dflt);
Greg Kroah-Hartman
maintainerPATCH 274/312 net: axienet: Fix return value check in axienet_probe()
In reply to Greg Kroah-HartmanFrom: Wei Yongjun <[email protected]>
commit 3ad7b1477ef9b01988ac052b02be9cd410c95157 upstream.
In case of error, the function of_parse_phandle() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test.
--- drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/xilinx/xilinx_axienet_main.c +2 -2
@@ -1602,9 +1602,9 @@ static int axienet_probe(struct platform/* Find the DMA node, map the DMA registers, and decode the DMA IRQs */ np =of_parse_phandle (pdev->dev.of_node,"axistream-connected" ,0 ); -if (IS_ERR (np)) { +if (!np) {dev_err (&pdev->dev,"could not find DMA node\n" ); - ret =PTR_ERR (np); + ret = -ENODEV ; goto free_netdev; } ret =of_address_to_resource (np,0 , &dmares);
Greg Kroah-Hartman
maintainerPATCH 275/312 bnxt_en: Remove locking around txr->dev_state
In reply to Greg Kroah-HartmanFrom: Florian Fainelli <[email protected]>
commit cbce91cad4ee39070bf3c7873767194e4be88e16 upstream.
txr->dev_state was not consistently manipulated with the acquisition of the per-queue lock, after further inspection the lock does not seem necessary, either the value is read as BNXT_DEV_STATE_CLOSING or 0.
--- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 2 -- 1 file changed, 2 deletions(-)
drivers/net/ethernet/broadcom/bnxt/bnxt.c +0 -2
@@ -4310,9 +4310,7 @@ static void bnxt_tx_disable(struct bnxt bnapi = bp->bnapi[i]; txr = &bnapi->tx_ring; txq =netdev_get_tx_queue (bp->dev, i); -__netif_tx_lock (txq,smp_processor_id ()); txr->dev_state =BNXT_DEV_STATE_CLOSING ; -__netif_tx_unlock (txq); } }/* Stop all TX queues */
Greg Kroah-Hartman
maintainerPATCH 276/312 net: ethernet: davinci_emac: Fix devioctl while in fixed link
In reply to Greg Kroah-HartmanFrom: Neil Armstrong <[email protected]>
commit 62522ef3c399996f6c8120bfd14b94280bc9f490 upstream.
When configured in fixed link, the DaVinci emac driver sets the priv->phydev to NULL and further ioctl calls to the phy_mii_ioctl() causes the kernel to crash.
--- drivers/net/ethernet/ti/davinci_emac.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
drivers/net/ethernet/ti/davinci_emac.c +4 -1
@@ -1512,7 +1512,10 @@ static int emac_devioctl(struct net_devi/* TODO: Add phy read and write and private statistics get feature */ -return phy_mii_ioctl (priv->phydev, ifrq, cmd); +if (priv->phydev) +return phy_mii_ioctl (priv->phydev, ifrq, cmd); +else +return -EOPNOTSUPP ; }static int match_first_device (struct device *dev,void *data)
Greg Kroah-Hartman
maintainerPATCH 277/312 net: ethernet: mvneta: Remove IFF_UNICAST_FLT which is not implemented
In reply to Greg Kroah-HartmanFrom: Andrew Lunn <[email protected]>
commit 97db8afa2ab919fc400fe982f5054060868bdf07 upstream.
The mvneta driver advertises it supports IFF_UNICAST_FLT. However, it actually does not. The hardware probably does support it, but there is no code to configure the filter. As a quick and simple fix, remove the flag. This will cause the core to fall back to promiscuous mode.
--- drivers/net/ethernet/marvell/mvneta.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/marvell/mvneta.c +1 -1
@@ -3406,7 +3406,7 @@ static int mvneta_probe(struct platform_ dev->features =NETIF_F_SG |NETIF_F_IP_CSUM |NETIF_F_TSO ; dev->hw_features |= dev->features; dev->vlan_features |= dev->features; - dev->priv_flags |=IFF_UNICAST_FLT |IFF_LIVE_ADDR_CHANGE ; + dev->priv_flags |=IFF_LIVE_ADDR_CHANGE ; dev->gso_max_segs =MVNETA_MAX_TSO_SEGS ; err =register_netdev (dev);
Greg Kroah-Hartman
maintainerPATCH 278/312 net: ethernet: ti: cpsw: fix device and of_node leaks
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit c7262aaace1b17a650598063e3b9ee1785fde377 upstream.
Make sure to drop the references taken by of_get_child_by_name() and bus_find_device() before returning from cpsw_phy_sel().
Note that holding a reference to the cpsw-phy-sel device does not prevent the devres-managed private data from going away.
--- drivers/net/ethernet/ti/cpsw-phy-sel.c | 3 +++ 1 file changed, 3 insertions(+)
drivers/net/ethernet/ti/cpsw-phy-sel.c +3 -0
@@ -154,9 +154,12 @@ void cpsw_phy_sel(struct device *dev, ph } dev =bus_find_device (&platform_bus_type,NULL , node, match); +of_node_put (node); priv =dev_get_drvdata (dev); priv->cpsw_phy_sel (priv, phy_mode, slave); + +put_device (dev); }EXPORT_SYMBOL_GPL (cpsw_phy_sel);
Greg Kroah-Hartman
maintainerPATCH 279/312 net: ethernet: ti: cpsw: fix secondary-emac probe error path
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit a7fe9d466f6a33558a38c7ca9d58bcc83512d577 upstream.
Make sure to deregister the primary device in case the secondary emac fails to probe.
kernel BUG at /home/johan/work/omicron/src/linux/net/core/dev.c:7743!
...
[<c05b3dec>] (free_netdev) from [<c04fe6c0>] (cpsw_probe+0x9cc/0xe50) [<c04fe6c0>] (cpsw_probe) from [<c047b28c>] (platform_drv_probe+0x5c/0xc0)
implementation")
--- drivers/net/ethernet/ti/cpsw.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
drivers/net/ethernet/ti/cpsw.c +3 -1
@@ -2532,12 +2532,14 @@ static int cpsw_probe(struct platform_de ret =cpsw_probe_dual_emac (pdev, priv);if (ret) {cpsw_err (priv, probe,"error probe slave 2 emac interface\n" ); - goto clean_ale_ret; + goto clean_unregister_netdev_ret; } }return 0 ; +clean_unregister_netdev_ret: +unregister_netdev (ndev); clean_ale_ret:cpsw_ale_destroy (priv->ale); clean_dma_ret:
Greg Kroah-Hartman
maintainerPATCH 280/312 net: hns: fix device reference leaks
In reply to Greg Kroah-HartmanFrom: Johan Hovold <[email protected]>
commit 2271150bfb814b72ec57ae2fdf66e39da2eafafd upstream.
Make sure to drop the reference taken by class_find_device() in hnae_get_handle() on errors and when later releasing the handle.
--- drivers/net/ethernet/hisilicon/hns/hnae.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-)
drivers/net/ethernet/hisilicon/hns/hnae.c +7 -1
@@ -331,8 +331,10 @@ struct hnae_handle *hnae_get_handle(strureturn ERR_PTR (-ENODEV ); handle = dev->ops->get_handle (dev, port_id); -if (IS_ERR (handle)) +if (IS_ERR (handle)) { +put_device (&dev->cls_dev);return handle; + } handle->dev = dev; handle->owner_dev = owner_dev;
@@ -355,6 +357,8 @@ out_when_init_queue:for (j = i -1 ; j >=0 ; j--)hnae_fini_queue (handle->qs[j]); +put_device (&dev->cls_dev); +return ERR_PTR (-ENOMEM ); }EXPORT_SYMBOL (hnae_get_handle);
@@ -376,6 +380,8 @@ void hnae_put_handle(struct hnae_handle dev->ops->put_handle (h);module_put (dev->owner); + +put_device (&dev->cls_dev); }EXPORT_SYMBOL (hnae_put_handle);
Greg Kroah-Hartman
maintainerPATCH 281/312 net: bridge: dont increment tx_dropped in br_do_proxy_arp
In reply to Greg Kroah-HartmanFrom: Nikolay Aleksandrov <[email protected]>
commit 85a3d4a9356b595d5440c3f1bf07ee7cecca1567 upstream.
pskb_may_pull may fail due to various reasons (e.g. alloc failure), but the skb isn't changed/dropped and processing continues so we shouldn't increment tx_dropped.
--- net/bridge/br_input.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
net/bridge/br_input.c +2 -5
@@ -78,13 +78,10 @@ static void br_do_proxy_arp(struct sk_buBR_INPUT_SKB_CB (skb)->proxyarp_replied = false; -if (dev->flags &IFF_NOARP ) +if ((dev->flags &IFF_NOARP ) || + !pskb_may_pull (skb,arp_hdr_len (dev)))return ; -if (!pskb_may_pull (skb,arp_hdr_len (dev))) { - dev->stats.tx_dropped++; -return ; - } parp =arp_hdr (skb); if (parp->ar_pro != htons(ETH_P_IP) ||
Greg Kroah-Hartman
maintainerPATCH 282/312 net: dsa: mv88e6xxx: enable SA learning on DSA ports
In reply to Greg Kroah-HartmanFrom: Vivien Didelot <[email protected]>
commit 996ecb8246676100af2a4dc1cfed747712a3c85f upstream.
In multi-chip systems, DSA Tag ports must learn SA addresses in order to correctly switch frames between interconnected chips.
This fixes cross-chip hardware bridging in a VLAN filtering aware system, because a bridge group gets implemented as an hardware 802.1Q VLAN and thus DSA and user ports share the same FDB.
--- drivers/net/dsa/mv88e6xxx.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/dsa/mv88e6xxx.c +2 -2
@@ -2064,8 +2064,8 @@ static int mv88e6xxx_setup_port(struct d* the other bits clear. */ reg =1 << port; -/* Disable learning for DSA and CPU ports */ -if (dsa_is_cpu_port (ds, port) ||dsa_is_dsa_port (ds, port)) +/* Disable learning for CPU port */ +if (dsa_is_cpu_port (ds, port)) reg =0 ; ret =_mv88e6xxx_reg_write (ds,REG_PORT (port),PORT_ASSOC_VECTOR , reg);
Greg Kroah-Hartman
maintainerPATCH 283/312 net: ehea: avoid null pointer dereference
In reply to Greg Kroah-HartmanFrom: [email protected] <[email protected]>
commit 1740c29a46b30a2f157afc473156f157e599d4c2 upstream.
ehea_get_port may return NULL. Do not dereference NULL value.
--- drivers/net/ethernet/ibm/ehea/ehea_main.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)
drivers/net/ethernet/ibm/ehea/ehea_main.c +4 -5
@@ -1169,16 +1169,15 @@ static void ehea_parse_eqe(struct ehea_a ec =EHEA_BMASK_GET (NEQE_EVENT_CODE , eqe); portnum =EHEA_BMASK_GET (NEQE_PORTNUM , eqe); port =ehea_get_port (adapter, portnum); +if (!port) { +netdev_err (NULL ,"unknown portnum %x\n" , portnum); +return ; + } dev = port->netdev;switch (ec) {case EHEA_EC_PORTSTATE_CHG :/* port state change */ -if (!port) { -netdev_err (dev,"unknown portnum %x\n" , portnum); -break ; - } -if (EHEA_BMASK_GET (NEQE_PORT_UP , eqe)) {if (!netif_carrier_ok (dev)) { ret =ehea_sense_port_attr (port);
Greg Kroah-Hartman
maintainerPATCH 284/312 l2tp: fix use-after-free during module unload
In reply to Greg Kroah-HartmanFrom: Sabrina Dubroca <[email protected]>
commit 2f86953e7436c9b9a4690909c5e2db24799e173b upstream.
Tunnel deletion is delayed by both a workqueue (l2tp_tunnel_delete -> wq
-> l2tp_tunnel_del_work) and RCU (sk_destruct -> RCU ->l2tp_tunnel_destruct).
By the time l2tp_tunnel_destruct() runs to destroy the tunnel and finish destroying the socket, the private data reserved via the net_generic mechanism has already been freed, but l2tp_tunnel_destruct() actually uses this data.
Make sure tunnel deletion for the netns has completed before returning from l2tp_exit_net() by first flushing the tunnel removal workqueue, and then waiting for RCU callbacks to complete.
--- net/l2tp/l2tp_core.c | 3 +++ 1 file changed, 3 insertions(+)
net/l2tp/l2tp_core.c +3 -0
@@ -1953,6 +1953,9 @@ static __net_exit void l2tp_exit_net(strl2tp_tunnel_delete (tunnel); }rcu_read_unlock_bh (); + +flush_workqueue (l2tp_wq); +rcu_barrier (); }static struct pernet_operations l2tp_net_ops = {
Greg Kroah-Hartman
maintainerPATCH 285/312 hwrng: exynos - Disable runtime PM on driver unbind
In reply to Greg Kroah-HartmanFrom: Krzysztof Kozlowski <[email protected]>
commit 27d80fa8bccf8d28bef4f89709638efc624fef9a upstream.
Driver enabled runtime PM but did not revert this on removal. Re-binding of a device triggered warning:
exynos-rng 10830400.rng: Unbalanced pm_runtime_enable!
--- drivers/char/hw_random/exynos-rng.c | 9 +++++++++ 1 file changed, 9 insertions(+)
drivers/char/hw_random/exynos-rng.c +9 -0
@@ -155,6 +155,14 @@ static int exynos_rng_probe(struct platfreturn ret; } +static int exynos_rng_remove (struct platform_device *pdev) +{ +pm_runtime_dont_use_autosuspend (&pdev->dev); +pm_runtime_disable (&pdev->dev); + +return 0 ; +} +static int __maybe_unused exynos_rng_runtime_suspend (struct device *dev) {struct platform_device *pdev =to_platform_device (dev);
@@ -212,6 +220,7 @@ static struct platform_driver exynos_rng .of_match_table = exynos_rng_dt_match, }, .probe = exynos_rng_probe, + .remove = exynos_rng_remove, };module_platform_driver (exynos_rng_driver);
Greg Kroah-Hartman
maintainerPATCH 286/312 net: icmp_route_lookup should use rt dev to determine L3 domain
In reply to Greg Kroah-HartmanFrom: David Ahern <[email protected]>
commit 9d1a6c4ea43e48c7880c85971c17939b56832d8a upstream.
icmp_send is called in response to some event. The skb may not have the device set (skb->dev is NULL), but it is expected to have an rt. Update icmp_route_lookup to use the rt on the skb to determine L3 domain.
--- net/ipv4/icmp.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
net/ipv4/icmp.c +2 -2
@@ -478,7 +478,7 @@ static struct rtable *icmp_route_lookup( fl4->flowi4_proto =IPPROTO_ICMP ; fl4->fl4_icmp_type = type; fl4->fl4_icmp_code = code; - fl4->flowi4_oif =l3mdev_master_ifindex (skb_in->dev); + fl4->flowi4_oif =l3mdev_master_ifindex (skb_dst (skb_in)->dev);security_skb_classify_flow (skb_in,flowi4_to_flowi (fl4)); rt = __ip_route_output_key_hash(net, fl4,
@@ -503,7 +503,7 @@ static struct rtable *icmp_route_lookup(if (err) goto relookup_failed; -if (inet_addr_type_dev_table (net, skb_in->dev, +if (inet_addr_type_dev_table (net,skb_dst (skb_in)->dev, fl4_dec.saddr) ==RTN_LOCAL ) { rt2 =__ip_route_output_key (net, &fl4_dec);if (IS_ERR (rt2))
Greg Kroah-Hartman
maintainerPATCH 287/312 net: mvneta: fix trivial cut-off issue in mvneta_ethtool_update_stats
In reply to Greg Kroah-HartmanFrom: Jisheng Zhang <[email protected]>
commit 2c832293e09be2f998ea916650927c8ccd5b4b3b upstream.
When s->type is T_REG_64, the high 32bits are lost in val. This patch fixes this trivial issue.
--- drivers/net/ethernet/marvell/mvneta.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)
drivers/net/ethernet/marvell/mvneta.c +4 -5
@@ -3058,26 +3058,25 @@ static void mvneta_ethtool_update_stats(const struct mvneta_statistic *s;void __iomem *base = pp->base;u32 high, low, val; +u64 val64;int i;for (i =0 , s = mvneta_statistics; s < mvneta_statistics +ARRAY_SIZE (mvneta_statistics); s++, i++) { - val =0 ; -switch (s->type) {case T_REG_32 : val =readl_relaxed (base + s->offset); + pp->ethtool_stats[i] += val;break ;case T_REG_64 :/* Docs say to read low 32-bit then high */ low =readl_relaxed (base + s->offset); high =readl_relaxed (base + s->offset +4 ); - val = (u64 )high <<32 | low; + val64 = (u64 )high <<32 | low; + pp->ethtool_stats[i] += val64;break ; } - - pp->ethtool_stats[i] += val; } }
Greg Kroah-Hartman
maintainerPATCH 288/312 net: macb: replace macb_writel() call by queue_writel() to update queue ISR
In reply to Greg Kroah-HartmanFrom: Cyrille Pitchen <[email protected]>
commit ba5049945421b8d2f3e2af786a15d13b82316503 upstream.
macb_interrupt() should not use macb_writel(bp, ISR, <value>) but only queue_writel(queue, ISR, <value>).
There is one IRQ and one set of {ISR, IER, IDR, IMR} [1] registers per queue on gem hardware, though only queue0 is actually used for now to receive frames: other queues can already be used to transmit frames.
The queue_readl() and queue_writel() helper macros are designed to access the relevant IRQ registers.
[1]
ISR: Interrupt Status Register
IER: Interrupt Enable Register
IDR: Interrupt Disable Register
IMR: Interrupt Mask Register
--- drivers/net/ethernet/cadence/macb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/cadence/macb.c +1 -1
@@ -1104,7 +1104,7 @@ static irqreturn_t macb_interrupt(int irmacb_writel (bp,NCR , ctrl |MACB_BIT (RE ));if (bp->caps &MACB_CAPS_ISR_CLEAR_ON_WRITE ) -macb_writel (bp,ISR ,MACB_BIT (RXUBR )); +queue_writel (queue,ISR ,MACB_BIT (RXUBR )); }if (status &MACB_BIT (ISR_ROVR )) {
Greg Kroah-Hartman
maintainerPATCH 289/312 ravb: Add missing free_irq() call to ravb_close()
In reply to Greg Kroah-HartmanFrom: Geert Uytterhoeven <[email protected]>
commit 7fa816b92c52e2c304f2ff6401e0d51e1d229ca5 upstream.
When reopening the network device on ra7795/salvator-x, e.g. after a DHCP timeout:
IP-Config: Reopening network devices...
genirq: Flags mismatch irq 139. 00000000 (eth0:ch24:emac) vs. 00000000 (eth0:ch24:emac) ravb e6800000.ethernet eth0: cannot request IRQ eth0:ch24:emac
IP-Config: Failed to open eth0
IP-Config: No network devices available
The "mismatch" is due to requesting an IRQ that is already in use, while IRQF_PROBE_SHARED wasn't set.
However, the real cause is that ravb_close() doesn't release the R-Car Gen3-specific secondary IRQ.
Add the missing free_irq() call to fix this.
--- drivers/net/ethernet/renesas/ravb_main.c | 2 ++ 1 file changed, 2 insertions(+)
drivers/net/ethernet/renesas/ravb_main.c +2 -0
@@ -1528,6 +1528,8 @@ static int ravb_close(struct net_device priv->phydev =NULL ; } +if (priv->chip_id ==RCAR_GEN3 ) +free_irq (priv->emac_irq, ndev);free_irq (ndev->irq, ndev);napi_disable (&priv->napi[RAVB_NC ]);
Greg Kroah-Hartman
maintainerPATCH 290/312 mvpp2: use correct size for memset
In reply to Greg Kroah-HartmanFrom: Arnd Bergmann <[email protected]>
commit e8f967c3d88489fc1562a31d4e44d905ac1d3aff upstream.
gcc-7 detects a short memset in mvpp2, introduced in the original merge of the driver:
drivers/net/ethernet/marvell/mvpp2.c: In function 'mvpp2_cls_init':drivers/net/ethernet/marvell/mvpp2.c:3296:2: error: 'memset' used with length equal to number of elements without multiplication by element size [-Werror=memset-elt-size]
The result seems to be that we write uninitialized data into the flow table registers, although we did not get any warning about that uninitialized data usage.
Using sizeof() lets us initialize then entire array instead.
--- drivers/net/ethernet/marvell/mvpp2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/ethernet/marvell/mvpp2.c +1 -1
@@ -3305,7 +3305,7 @@ static void mvpp2_cls_init(struct mvpp2mvpp2_write (priv,MVPP2_CLS_MODE_REG ,MVPP2_CLS_MODE_ACTIVE_MASK );/* Clear classifier flow table */ -memset (&fe.data,0 ,MVPP2_CLS_FLOWS_TBL_DATA_WORDS ); +memset (&fe.data,0 ,sizeof (fe.data));for (index =0 ; index <MVPP2_CLS_FLOWS_TBL_SIZE ; index++) { fe.index = index;mvpp2_cls_flow_write (priv, &fe);
Greg Kroah-Hartman
maintainerPATCH 291/312 net: vxlan: lwt: Fix vxlan local traffic.
In reply to Greg Kroah-HartmanFrom: pravin shelar <[email protected]>
commit bbec7802c6948c8626b71a4fe31283cb4691c358 upstream.
vxlan driver has bypass for local vxlan traffic, but that depends on information about all VNIs on local system in vxlan driver. This is not available in case of LWT. Therefore following patch disable encap bypass for LWT vxlan traffic.
--- drivers/net/vxlan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/vxlan.c +2 -2
@@ -2054,7 +2054,7 @@ static void vxlan_xmit_one(struct sk_buf }/* Bypass encapsulation if the destination is local */ -if (rt->rt_flags &RTCF_LOCAL && +if (!info && rt->rt_flags &RTCF_LOCAL && !(rt->rt_flags & (RTCF_BROADCAST |RTCF_MULTICAST ))) {struct vxlan_dev *dst_vxlan;
@@ -2112,7 +2112,7 @@ static void vxlan_xmit_one(struct sk_buf/* Bypass encapsulation if the destination is local */ rt6i_flags = ((struct rt6_info *)ndst)->rt6i_flags; -if (rt6i_flags &RTF_LOCAL && +if (!info && rt6i_flags &RTF_LOCAL && !(rt6i_flags & (RTCF_BROADCAST |RTCF_MULTICAST ))) {struct vxlan_dev *dst_vxlan;
Greg Kroah-Hartman
maintainerPATCH 292/312 net: ethoc: Fix early error paths
In reply to Greg Kroah-HartmanFrom: Florian Fainelli <[email protected]>
commit 386512d18b268c6182903239f9f3390f03ce4c7b upstream.
In case any operation fails before we can successfully go the point where we would register a MDIO bus, we would be going to an error label which involves unregistering then freeing this yet to be created MDIO bus. Update all error paths to go to label free which is the only one valid until either the clock is enabled, or the MDIO bus is allocated and registered. This fixes kernel oops observed while trying to dereference the MDIO bus structure which is not yet allocated.
--- drivers/net/ethernet/ethoc.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)
drivers/net/ethernet/ethoc.c +5 -5
@@ -1088,7 +1088,7 @@ static int ethoc_probe(struct platform_dif (!priv->iobase) {dev_err (&pdev->dev,"cannot remap I/O memory space\n" ); ret = -ENXIO ; - goto error; + goto free; }if (netdev->mem_end) {
@@ -1097,7 +1097,7 @@ static int ethoc_probe(struct platform_dif (!priv->membase) {dev_err (&pdev->dev,"cannot remap memory space\n" ); ret = -ENXIO ; - goto error; + goto free; } }else {/* Allocate buffer memory */
@@ -1108,7 +1108,7 @@ static int ethoc_probe(struct platform_ddev_err (&pdev->dev,"cannot allocate %dB buffer\n" , buffer_size); ret = -ENOMEM ; - goto error; + goto free; } netdev->mem_end = netdev->mem_start + buffer_size; priv->dma_alloc = buffer_size;
@@ -1122,7 +1122,7 @@ static int ethoc_probe(struct platform_d 128, (netdev->mem_end - netdev->mem_start + 1) / ETHOC_BUFSIZ);if (num_bd <4 ) { ret = -ENODEV ; - goto error; + goto free; } priv->num_bd = num_bd;/* num_tx must be a power of two */
@@ -1135,7 +1135,7 @@ static int ethoc_probe(struct platform_d priv->vma =devm_kzalloc (&pdev->dev, num_bd*sizeof (void *),GFP_KERNEL );if (!priv->vma) { ret = -ENOMEM ; - goto error; + goto free; }/* Allow the platform setup code to pass in a MAC address. */
Greg Kroah-Hartman
maintainerPATCH 293/312 ovs/vxlan: fix rtnl notifications on iface deletion
In reply to Greg Kroah-HartmanFrom: Nicolas Dichtel <[email protected]>
commit cf5da330bbdd0c06b05c525a3d1d58ccd82c87a6 upstream.
The function vxlan_dev_create() (only used by ovs) never calls rtnl_configure_link(). The consequence is that dev->rtnl_link_state is never set to RTNL_LINK_INITIALIZED.
During the deletion phase, the function rollback_registered_many() sends a RTM_DELLINK only if dev->rtnl_link_state is set to RTNL_LINK_INITIALIZED.
Note that the function vxlan_dev_create() is moved after the rtnl stuff so that vxlan_dellink() can be called in this function.
--- drivers/net/vxlan.c | 58 ++++++++++++++++++++++++++++++---------------------- 1 file changed, 34 insertions(+), 24 deletions(-)
drivers/net/vxlan.c +34 -24
@@ -2927,30 +2927,6 @@ static int vxlan_dev_configure(struct nereturn 0 ; } -struct net_device *vxlan_dev_create (struct net *net,const char *name, -u8 name_assign_type,struct vxlan_config *conf) -{ -struct nlattr *tb[IFLA_MAX +1 ]; -struct net_device *dev; -int err; - -memset (&tb,0 ,sizeof (tb)); - - dev =rtnl_create_link (net, name, name_assign_type, - &vxlan_link_ops, tb); -if (IS_ERR (dev)) -return dev; - - err =vxlan_dev_configure (net, dev, conf); -if (err <0 ) { -free_netdev (dev); -return ERR_PTR (err); - } - -return dev; -} -EXPORT_SYMBOL_GPL (vxlan_dev_create); -static int vxlan_newlink (struct net *src_net,struct net_device *dev,struct nlattr *tb[],struct nlattr *data[]) {
@@ -3218,6 +3194,40 @@ static struct rtnl_link_ops vxlan_link_o .get_link_net = vxlan_get_link_net, }; +struct net_device *vxlan_dev_create (struct net *net,const char *name, +u8 name_assign_type, +struct vxlan_config *conf) +{ +struct nlattr *tb[IFLA_MAX +1 ]; +struct net_device *dev; +int err; + +memset (&tb,0 ,sizeof (tb)); + + dev =rtnl_create_link (net, name, name_assign_type, + &vxlan_link_ops, tb); +if (IS_ERR (dev)) +return dev; + + err =vxlan_dev_configure (net, dev, conf); +if (err <0 ) { +free_netdev (dev); +return ERR_PTR (err); + } + + err =rtnl_configure_link (dev,NULL ); +if (err <0 ) { +LIST_HEAD (list_kill); + +vxlan_dellink (dev, &list_kill); +unregister_netdevice_many (&list_kill); +return ERR_PTR (err); + } + +return dev; +} +EXPORT_SYMBOL_GPL (vxlan_dev_create); +static void vxlan_handle_lowerdev_unregister (struct vxlan_net *vn,struct net_device *dev) {
Greg Kroah-Hartman
maintainerPATCH 294/312 net: mv643xx_eth: fix packet corruption with TSO and tiny unaligned packets.
In reply to Greg Kroah-HartmanFrom: Nicolas Schichan <[email protected]>
commit 3b89624ab54b9dc2d92fc08ce2670e5f19ad8ec8 upstream.
The code in txq_put_data() would use txq->tx_curr_desc to index the tso_hdrs/tso_hdrs_dma buffers, for less than 8 bytes unaligned fragments, which is already moved to the next descriptor at the beginning of the function.
If that fragment was the last of the the skb, the next skb would use that same space to place the ip headers, overwritting that small fragment data.
--- drivers/net/ethernet/marvell/mv643xx_eth.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/net/ethernet/marvell/mv643xx_eth.c +2 -2
@@ -762,10 +762,10 @@ txq_put_data_tso(struct net_device *dev,if (length <=8 && (uintptr_t )data &0x7 ) {/* Copy unaligned small data fragment to TSO header data area */ -memcpy (txq->tso_hdrs + txq->tx_curr_desc *TSO_HEADER_SIZE , +memcpy (txq->tso_hdrs + tx_index *TSO_HEADER_SIZE , data, length); desc->buf_ptr = txq->tso_hdrs_dma - + txq->tx_curr_desc *TSO_HEADER_SIZE ; + + tx_index *TSO_HEADER_SIZE ; }else {/* Alignment is okay, map buffer and hand off to hardware */ txq->tx_desc_mapping[tx_index] =DESC_DMA_MAP_SINGLE ;
Greg Kroah-Hartman
maintainerPATCH 295/312 regulator: core: Rely on regulator_dev_release to free constraints
In reply to Greg Kroah-HartmanFrom: Charles Keepax <[email protected]>
commit 6333ef46bbe514a8ece6c432aab6bcf8637b2d7c upstream.
As we now free the constraints in regulator_dev_release we will still call free on the constraints pointer even if we went down an error path in regulator_register, because it is only allocated after the device_register. As such we no longer need to free rdev->constraints on the error paths, so this patch removes said frees.
--- drivers/regulator/core.c | 29 ++++++++++++----------------- 1 file changed, 12 insertions(+), 17 deletions(-)
drivers/regulator/core.c +12 -17
@@ -1057,18 +1057,18 @@ static int set_machine_constraints(struc ret =machine_constraints_voltage (rdev, rdev->constraints);if (ret !=0 ) - goto out; +return ret; ret =machine_constraints_current (rdev, rdev->constraints);if (ret !=0 ) - goto out; +return ret;if (rdev->constraints->ilim_uA && ops->set_input_current_limit) { ret = ops->set_input_current_limit (rdev, rdev->constraints->ilim_uA);if (ret <0 ) {rdev_err (rdev,"failed to set input limit\n" ); - goto out; +return ret; } }
@@ -1077,21 +1077,20 @@ static int set_machine_constraints(struc ret =suspend_prepare (rdev, rdev->constraints->initial_state);if (ret <0 ) {rdev_err (rdev,"failed to set suspend state\n" ); - goto out; +return ret; } }if (rdev->constraints->initial_mode) {if (!ops->set_mode) {rdev_err (rdev,"no set_mode operation\n" ); - ret = -EINVAL ; - goto out; +return -EINVAL ; } ret = ops->set_mode (rdev, rdev->constraints->initial_mode);if (ret <0 ) {rdev_err (rdev,"failed to set initial mode: %d\n" , ret); - goto out; +return ret; } }
@@ -1102,7 +1101,7 @@ static int set_machine_constraints(struc ret =_regulator_do_enable (rdev);if (ret <0 && ret != -EINVAL ) {rdev_err (rdev,"failed to enable\n" ); - goto out; +return ret; } }
@@ -1111,7 +1110,7 @@ static int set_machine_constraints(struc ret = ops->set_ramp_delay (rdev, rdev->constraints->ramp_delay);if (ret <0 ) {rdev_err (rdev,"failed to set ramp_delay\n" ); - goto out; +return ret; } }
@@ -1119,7 +1118,7 @@ static int set_machine_constraints(struc ret = ops->set_pull_down (rdev);if (ret <0 ) {rdev_err (rdev,"failed to set pull down\n" ); - goto out; +return ret; } }
@@ -1127,7 +1126,7 @@ static int set_machine_constraints(struc ret = ops->set_soft_start (rdev);if (ret <0 ) {rdev_err (rdev,"failed to set soft start\n" ); - goto out; +return ret; } }
@@ -1136,16 +1135,12 @@ static int set_machine_constraints(struc ret = ops->set_over_current_protection (rdev);if (ret <0 ) {rdev_err (rdev,"failed to set over current protection\n" ); - goto out; +return ret; } }print_constraints (rdev);return 0 ; -out: -kfree (rdev->constraints); - rdev->constraints =NULL ; -return ret; }/**
@@ -3983,7 +3978,7 @@ unset_supplies: scrub:regulator_ena_gpio_free (rdev); -kfree (rdev->constraints); + wash:device_unregister (&rdev->dev);/* device core frees rdev */
Greg Kroah-Hartman
maintainerPATCH 296/312 net: dsa: mv88e6xxx: fix port VLAN maps
In reply to Greg Kroah-HartmanFrom: Vivien Didelot <[email protected]>
commit be1faa92e83b1252d9200c59d8c98aab44463f1e upstream.
Currently the port based VLAN maps should be configured to allow every port to egress frames on all other ports, except themselves.
The debugfs interface shows that they are misconfigured. For instance, a 7-port switch has the following content in the related register 0x06:
GLOBAL GLOBAL2 SERDES 0 1 2 3 4 5 6
...
6: 1fa4 1f0f 4 7f 7e 7d 7c 7b 7a 79
...This means that port 3 is allowed to talk to port 2-6, but cannot talk to ports 0 and 1. With this fix, port 3 can correctly talk to all ports except 3 itself:
GLOBAL GLOBAL2 SERDES 0 1 2 3 4 5 6
...
6: 1fa4 1f0f 4 7e 7d 7b 77 6f 5f 3f
...--- drivers/net/dsa/mv88e6xxx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/net/dsa/mv88e6xxx.c +2 -1
@@ -2150,7 +2150,8 @@ static int mv88e6xxx_setup_port(struct d* database, and allow every port to egress frames on all other ports. */ reg =BIT (ps->num_ports) -1 ;/* all ports */ - ret =_mv88e6xxx_port_vlan_map_set (ds, port, reg & ~port); + reg &= ~BIT (port);/* except itself */ + ret =_mv88e6xxx_port_vlan_map_set (ds, port, reg);if (ret) goto abort;
Greg Kroah-Hartman
maintainerPATCH 297/312 at803x: fix reset handling
In reply to Greg Kroah-HartmanFrom: Sergei Shtylyov <[email protected]>
commit d57019d1858a6f9b3ca05d76d793466ae428cfa3 upstream.
The driver of course "knows" that the chip's reset signal is active low, so it drives the GPIO to 0 to reset the PHY and to 1 otherwise; however all this will only work iff the GPIO is specified as active-high in the device tree! I think both the driver and the device trees (if there are any -- I was unable to find them) need to be fixed in this case...
--- drivers/net/phy/at803x.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)
drivers/net/phy/at803x.c +3 -3
@@ -198,7 +198,7 @@ static int at803x_probe(struct phy_devicif (!priv)return -ENOMEM ; - gpiod_reset =devm_gpiod_get_optional (dev,"reset" ,GPIOD_OUT_HIGH ); + gpiod_reset =devm_gpiod_get_optional (dev,"reset" ,GPIOD_OUT_LOW );if (IS_ERR (gpiod_reset))return PTR_ERR (gpiod_reset);
@@ -274,10 +274,10 @@ static void at803x_link_change_notify(stat803x_context_save (phydev, &context); -gpiod_set_value (priv->gpiod_reset,0 ); -msleep (1 );gpiod_set_value (priv->gpiod_reset,1 );msleep (1 ); +gpiod_set_value (priv->gpiod_reset,0 ); +msleep (1 );at803x_context_restore (phydev, &context);
Greg Kroah-Hartman
maintainerPATCH 298/312 regulator: Try to resolve regulators supplies on registration
In reply to Greg Kroah-HartmanFrom: Javier Martinez Canillas <[email protected]>
commit 5e3ca2b349b1e2c80b060b51bbf2af37448fad85 upstream.
Commit 6261b06de565 ("regulator: Defer lookup of supply to regulator_get") moved the regulator supplies lookup logic from the regulators registration to the regulators get time.
Unfortunately, that changed the behavior of the regulator core since now a parent supply with a child regulator marked as always-on, won't be enabled unless a client driver attempts to get the child regulator during boot.
This patch tries to resolve the parent supply for the already registered regulators each time that a new regulator is registered. So the regulators that have child regulators marked as always on will be enabled regardless if a driver gets the child regulator or not.
That was the behavior before the mentioned commit, since parent supplies were looked up at regulator registration time instead of during child get.
Since regulator_resolve_supply() checks for rdev->supply, most of the times it will be a no-op. Errors aren't checked to keep the possible out of order dependencies which was the motivation for the mentioned commit.
Also, the supply being available will be enforced on regulator get anyways in case the resolve fails on regulators registration.
--- drivers/regulator/core.c | 9 +++++++++ 1 file changed, 9 insertions(+)
drivers/regulator/core.c +9 -0
@@ -3822,6 +3822,11 @@ static void rdev_init_debugfs(struct reg &rdev->bypass_count); } +static int regulator_register_resolve_supply (struct device *dev,void *data) +{ +return regulator_resolve_supply (dev_to_rdev (dev)); +} +/** * regulator_register - register regulator * @regulator_desc: regulator to register
@@ -3968,6 +3973,10 @@ regulator_register(const struct regulato }rdev_init_debugfs (rdev); + +/* try to resolve regulators supply since a new one was registered */ +class_for_each_device (®ulator_class,NULL ,NULL , + regulator_register_resolve_supply); out:mutex_unlock (®ulator_list_mutex);kfree (config);
Greg Kroah-Hartman
maintainerPATCH 299/312 cxl: Fix DAR check & use REGION_ID instead of opencoding
In reply to Greg Kroah-HartmanFrom: Aneesh Kumar K.V <[email protected]>
commit 3b1dbfa14f97188ec33fdfc7acb66bea59a3bb21 upstream.
The current code will set _PAGE_USER to the access flags for any fault address, because the ~ operation will be true for all address we take a fault on. But setting _PAGE_USER also means that the fault will be handled only if the page table have _PAGE_USER set. Hence there is no security hole with the current code.
Now if it is an user space access, then the change in this patch really don't have an impact because we have (!ctx->kernel) set true and we take the if condition true.
Now kernel context created fault on an address in the kernel range will result in a fault loop because we will not insert the hash pte due to access and pte permission mismatch. This patch fix the above issue.
--- drivers/misc/cxl/fault.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/misc/cxl/fault.c +1 -1
@@ -152,7 +152,7 @@ static void cxl_handle_page_fault(struct access = _PAGE_PRESENT;if (dsisr & CXL_PSL_DSISR_An_S) access |= _PAGE_RW; -if ((!ctx->kernel) || ~(dar & (1ULL <<63 ))) +if ((!ctx->kernel) || (REGION_ID (dar) ==USER_REGION_ID )) access |= _PAGE_USER;if (dsisr &DSISR_NOHPTE )
Greg Kroah-Hartman
maintainerPATCH 300/312 net: ethernet: davinci_emac: Fix platform_data overwrite
In reply to Greg Kroah-HartmanFrom: Neil Armstrong <[email protected]>
commit 210990b05a1247886539078e857cd038881bb2d6 upstream.
When the DaVinci emac driver is removed and re-probed, the actual pdev->dev.platform_data is populated with an unwanted valid pointer saved by the previous davinci_emac_of_get_pdata() call, causing a kernel crash when calling priv->int_disable() in emac_int_disable().
Unable to handle kernel paging request at virtual address c8622a80
...
[<c0426fb4>] (emac_int_disable) from [<c0427700>] (emac_dev_open+0x290/0x5f8) [<c0427700>] (emac_dev_open) from [<c04c00ec>] (__dev_open+0xb8/0x120) [<c04c00ec>] (__dev_open) from [<c04c0370>] (__dev_change_flags+0x88/0x14c) [<c04c0370>] (__dev_change_flags) from [<c04c044c>] (dev_change_flags+0x18/0x48) [<c04c044c>] (dev_change_flags) from [<c052bafc>] (devinet_ioctl+0x6b4/0x7ac) [<c052bafc>] (devinet_ioctl) from [<c04a1428>] (sock_ioctl+0x1d8/0x2c0) [<c04a1428>] (sock_ioctl) from [<c014f054>] (do_vfs_ioctl+0x41c/0x600) [<c014f054>] (do_vfs_ioctl) from [<c014f2a4>] (SyS_ioctl+0x6c/0x7c) [<c014f2a4>] (SyS_ioctl) from [<c000ff60>] (ret_fast_syscall+0x0/0x1c)
--- drivers/net/ethernet/ti/davinci_emac.c | 2 -- 1 file changed, 2 deletions(-)
drivers/net/ethernet/ti/davinci_emac.c +0 -2
@@ -1888,8 +1888,6 @@ davinci_emac_of_get_pdata(struct platfor pdata->hw_ram_addr = auxdata->hw_ram_addr; } - pdev->dev.platform_data = pdata; -return pdata; }
Greg Kroah-Hartman
maintainerPATCH 301/312 ata: sata_dwc_460ex: remove incorrect locking
In reply to Greg Kroah-HartmanFrom: Mans Rullgard <[email protected]>
commit 55e610cdd28c0ad3dce0652030c0296d549673f3 upstream.
This lock is already taken in ata_scsi_queuecmd() a few levels up the call stack so attempting to take it here is an error. Moreover, it is pointless in the first place since it only protects a single, atomic assignment.
Enabling lock debugging gives the following output:
=============================================
[ INFO: possible recursive locking detected ]
4.4.0-rc5+ #189 Not tainted
---------------------------------------------
kworker/u2:3/37 is trying to acquire lock:
(&(&host->lock)->rlock){-.-...}, at: [<90283294>] sata_dwc_exec_command_by_tag.constprop.14+0x44/0x8c
but task is already holding lock:
(&(&host->lock)->rlock){-.-...}, at: [<902761ac>] ata_scsi_queuecmd+0x2c/0x330
other info that might help us debug this:
Possible unsafe locking scenario:
CPU0
---- lock(&(&host->lock)->rlock);
lock(&(&host->lock)->rlock);
*** DEADLOCK ***
May be due to missing lock nesting notation
4 locks held by kworker/u2:3/37:
#0: ("events_unbound"){.+.+.+}, at: [<9003a0a4>] process_one_work+0x12c/0x430
#1: ((&entry->work)){+.+.+.}, at: [<9003a0a4>] process_one_work+0x12c/0x430
#2: (&bdev->bd_mutex){+.+.+.}, at: [<9011fd54>] __blkdev_get+0x50/0x380
#3: (&(&host->lock)->rlock){-.-...}, at: [<902761ac>] ata_scsi_queuecmd+0x2c/0x330stack backtrace:
CPU: 0 PID: 37 Comm: kworker/u2:3 Not tainted 4.4.0-rc5+ #189
Workqueue: events_unbound async_run_entry_fn
Stack : 90b38e30 00000021 00000003 9b2a6040 00000000 9005f3f0 904fc8dc 00000025
906b96e4 00000000 90528648 9b3336c4 904fc8dc 9009bf18 00000002 00000004
00000000 00000000 9b3336c4 9b3336e4 904fc8dc 9003d074 00000000 90500000
9005e738 00000000 00000000 00000000 00000000 00000000 00000000 00000000
6e657665 755f7374 756f626e 0000646e 00000000 00000000 9b00ca00 9b025000
...
Call Trace:
[<90009d6c>] show_stack+0x88/0xa4
[<90057744>] __lock_acquire+0x1ce8/0x2154
[<900583e4>] lock_acquire+0x64/0x8c
[<9045ff10>] _raw_spin_lock_irqsave+0x54/0x78
[<90283294>] sata_dwc_exec_command_by_tag.constprop.14+0x44/0x8c
[<90283484>] sata_dwc_qc_issue+0x1a8/0x24c
[<9026b39c>] ata_qc_issue+0x1f0/0x410
[<90273c6c>] ata_scsi_translate+0xb4/0x200
[<90276234>] ata_scsi_queuecmd+0xb4/0x330
[<9025800c>] scsi_dispatch_cmd+0xd0/0x128
[<90259934>] scsi_request_fn+0x58c/0x638
[<901a3e50>] __blk_run_queue+0x40/0x5c
[<901a83d4>] blk_queue_bio+0x27c/0x28c
[<901a5914>] generic_make_request+0xf0/0x188
[<901a5a54>] submit_bio+0xa8/0x194
[<9011adcc>] submit_bh_wbc.isra.23+0x15c/0x17c
[<9011c908>] block_read_full_page+0x3e4/0x428
[<9009e2e0>] do_read_cache_page+0xac/0x210
[<9009fd90>] read_cache_page+0x18/0x24
[<901bbd18>] read_dev_sector+0x38/0xb0
[<901bd174>] msdos_partition+0xb4/0x5c0
[<901bcb8c>] check_partition+0x140/0x274
[<901bba60>] rescan_partitions+0xa0/0x2b0
[<9011ff68>] __blkdev_get+0x264/0x380
[<901201ac>] blkdev_get+0x128/0x36c
[<901b9378>] add_disk+0x3c0/0x4bc
[<90268268>] sd_probe_async+0x100/0x224
[<90043a44>] async_run_entry_fn+0x50/0x124
[<9003a11c>] process_one_work+0x1a4/0x430
[<9003a4f4>] worker_thread+0x14c/0x4fc
[<900408f4>] kthread+0xd0/0xe8
[<90004338>] ret_from_kernel_thread+0x14/0x1c--- drivers/ata/sata_dwc_460ex.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
drivers/ata/sata_dwc_460ex.c +1 -3
@@ -924,15 +924,13 @@ static void sata_dwc_exec_command_by_tagstruct ata_taskfile *tf, u8 tag, u32 cmd_issued) { - unsigned long flags; struct sata_dwc_device_port *hsdevp = HSDEVP_FROM_AP(ap);dev_dbg (ap->dev,"%s cmd(0x%02x): %s tag=%d\n" , __func__, tf->command,ata_get_cmd_descript (tf->command), tag); -spin_lock_irqsave (&ap->host->lock, flags); hsdevp->cmd_issued[tag] = cmd_issued; -spin_unlock_irqrestore (&ap->host->lock, flags); +/* * Clear SError before executing a new command. * sata_dwc_scr_write and read can not be used here. Clearing the PM
Greg Kroah-Hartman
maintainerPATCH 302/312 pinctrl: tegra: Correctly check the supported configuration
In reply to Greg Kroah-HartmanFrom: Laxman Dewangan <[email protected]>
commit b22ef2a0979f2b91cfeeabb086e4d665183a93a1 upstream.
The pincontrol registers of Tegra chips has multiple filed per registers. There is two type of registers mux and drive. All configurations belongs to one of these registers.
If any configurations are supported then <config>_bit is set to bit position of these registers otherwise -1 to not support it. The member is defined as
s32 <config>_bit:6;
So if config is not supported ifor given SoC then it is set to -1 in soc pinmmux table.
In common driver code, to find out that given config is supported or not, it is checked as:
s8 bit = <config>_bit;
if (bit > 31) {
/* Not supported config */
}But in this case, bit is s8 and hence for non supporting it is -1.
Correct the check as:
if (bit < 0) {
/* Not supported config */
}--- drivers/pinctrl/pinctrl-tegra.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pinctrl/pinctrl-tegra.c +1 -1
@@ -418,7 +418,7 @@ static int tegra_pinconf_reg(struct tegrreturn -ENOTSUPP ; } -if (*reg <0 || *bit >31 ) { +if (*reg <0 || *bit <0 ) {if (report_err) {const char *prop ="unknown" ;int i;
Greg Kroah-Hartman
maintainerPATCH 303/312 brcmfmac: add fallback for devices that do not report per-chain values
In reply to Greg Kroah-HartmanFrom: Jaap Jan Meijer <[email protected]>
commit 94abd778a7bb00ed5d00f56d9fbfcbf5b7c02a5c upstream.
If brcmf_cfg80211_get_station fails to determine the RSSI from the per-chain values get the value individually as a fallback.
--- drivers/net/wireless/brcm80211/brcmfmac/cfg80211.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)
drivers/net/wireless/brcm80211/brcmfmac/cfg80211.c +16 -0
@@ -2419,12 +2419,14 @@ brcmf_cfg80211_get_station(struct wiphy const u8 *mac, struct station_info *sinfo) { struct brcmf_if *ifp = netdev_priv(ndev); +struct brcmf_scb_val_le scb_val;s32 err =0 ;struct brcmf_sta_info_le sta_info_le;u32 sta_flags;u32 is_tdls_peer;s32 total_rssi;s32 count_rssi; +int rssi;u32 i;brcmf_dbg (TRACE ,"Enter, MAC %pM\n" , mac);
@@ -2505,6 +2507,20 @@ brcmf_cfg80211_get_station(struct wiphy sinfo->filled |=BIT (NL80211_STA_INFO_SIGNAL ); total_rssi /= count_rssi; sinfo->signal = total_rssi; + }else if (test_bit (BRCMF_VIF_STATUS_CONNECTED , + &ifp->vif->sme_state)) { +memset (&scb_val,0 ,sizeof (scb_val)); + err =brcmf_fil_cmd_data_get (ifp,BRCMF_C_GET_RSSI , + &scb_val,sizeof (scb_val)); +if (err) { +brcmf_err ("Could not get rssi (%d)\n" , err); + goto done; + }else { + rssi =le32_to_cpu (scb_val.val); + sinfo->filled |=BIT (NL80211_STA_INFO_SIGNAL ); + sinfo->signal = rssi; +brcmf_dbg (CONN ,"RSSI %d dBm\n" , rssi); + } } } done:
Greg Kroah-Hartman
maintainerPATCH 304/312 brcmfmac: restore stopping netdev queue when bus clogs up
In reply to Greg Kroah-HartmanFrom: Arend Van Spriel <[email protected]>
commit 82bc9ab6a8f577d2174a736c33f3d4ecf7d9ef47 upstream.
When the host-interface bus has hard time handling transmit packets it informs higher layer about this and it would stop the netdev queue when needed. However, since commit 9cd18359d31e ("brcmfmac:
Make FWS queueing configurable.") this was broken. With this patch the behaviour is restored.
--- drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c | 22 ++++++++++++++++----- 1 file changed, 17 insertions(+), 5 deletions(-)
drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c +17 -5
@@ -2262,10 +2262,22 @@ void brcmf_fws_bustxfail(struct brcmf_fwvoid brcmf_fws_bus_blocked (struct brcmf_pub *drvr,bool flow_blocked) {struct brcmf_fws_info *fws = drvr->fws; +struct brcmf_if *ifp; +int i; - fws->bus_flow_blocked = flow_blocked; -if (!flow_blocked) -brcmf_fws_schedule_deq (fws); -else - fws->stats.bus_flow_block++; +if (fws->avoid_queueing) { +for (i =0 ; i <BRCMF_MAX_IFS ; i++) { + ifp = drvr->iflist[i]; +if (!ifp || !ifp->ndev) +continue ; +brcmf_txflowblock_if (ifp,BRCMF_NETIF_STOP_REASON_FLOW , + flow_blocked); + } + }else { + fws->bus_flow_blocked = flow_blocked; +if (!flow_blocked) +brcmf_fws_schedule_deq (fws); +else + fws->stats.bus_flow_block++; + } }
Greg Kroah-Hartman
maintainerPATCH 305/312 bridge: Fix problems around fdb entries pointing to the bridge device
In reply to Greg Kroah-HartmanFrom: Toshiaki Makita <[email protected]>
commit 7bb90c3715a496c650b2e879225030f9dd9cfafb upstream.
Adding fdb entries pointing to the bridge device uses fdb_insert(), which lacks various checks and does not respect added_by_user flag.
As a result, some inconsistent behavior can happen:
* Adding temporary entries succeeds but results in permanent entries.
* Same goes for "dynamic" and "use".
* Changing mac address of the bridge device causes deletion of user-added entries.
* Replacing existing entries looks successful from userspace but actually not, regardless of NLM_F_EXCL flag.
Use the same logic as other entries and fix them.
--- net/bridge/br_fdb.c | 52 +++++++++++++++++++++++++++------------------------- 1 file changed, 27 insertions(+), 25 deletions(-)
net/bridge/br_fdb.c +27 -25
@@ -266,7 +266,7 @@ void br_fdb_change_mac_address(struct ne/* If old entry was unassociated with any port, then delete it. */ f =__br_fdb_get (br, br->dev->dev_addr,0 ); -if (f && f->is_local && !f->dst) +if (f && f->is_local && !f->dst && !f->added_by_user)fdb_delete_local (br,NULL , f);fdb_insert (br,NULL , newaddr,0 );
@@ -281,7 +281,7 @@ void br_fdb_change_mac_address(struct neif (!br_vlan_should_use (v))continue ; f =__br_fdb_get (br, br->dev->dev_addr, v->vid); -if (f && f->is_local && !f->dst) +if (f && f->is_local && !f->dst && !f->added_by_user)fdb_delete_local (br,NULL , f);fdb_insert (br,NULL , newaddr, v->vid); }
@@ -758,20 +758,25 @@ out: }/* Update (create or replace) forwarding database entry */ -static int fdb_add_entry (struct net_bridge_port *source,const __u8 *addr, -__u16 state,__u16 flags,__u16 vid) +static int fdb_add_entry (struct net_bridge *br,struct net_bridge_port *source, +const __u8 *addr,__u16 state,__u16 flags,__u16 vid) { -struct net_bridge *br = source->br;struct hlist_head *head = &br->hash[br_mac_hash (addr, vid)];struct net_bridge_fdb_entry *fdb;bool modified = false;/* If the port cannot learn allow only local and static entries */ -if (!(state &NUD_PERMANENT ) && !(state &NUD_NOARP ) && +if (source && !(state &NUD_PERMANENT ) && !(state &NUD_NOARP ) && !(source->state ==BR_STATE_LEARNING || source->state ==BR_STATE_FORWARDING ))return -EPERM ; +if (!source && !(state &NUD_PERMANENT )) { +pr_info ("bridge: RTM_NEWNEIGH %s without NUD_PERMANENT\n" , + br->dev->name); +return -EINVAL ; + } + fdb =fdb_find (head, addr, vid);if (fdb ==NULL ) {if (!(flags &NLM_F_CREATE ))
@@ -826,22 +831,28 @@ static int fdb_add_entry(struct net_bridreturn 0 ; } -static int __br_fdb_add (struct ndmsg *ndm,struct net_bridge_port *p, -const unsigned char *addr,u16 nlh_flags,u16 vid) +static int __br_fdb_add (struct ndmsg *ndm,struct net_bridge *br, +struct net_bridge_port *p,const unsigned char *addr, +u16 nlh_flags,u16 vid) {int err =0 ;if (ndm->ndm_flags &NTF_USE ) { +if (!p) { +pr_info ("bridge: RTM_NEWNEIGH %s with NTF_USE is not supported\n" , + br->dev->name); +return -EINVAL ; + }local_bh_disable ();rcu_read_lock (); -br_fdb_update (p->br, p, addr, vid, true); +br_fdb_update (br, p, addr, vid, true);rcu_read_unlock ();local_bh_enable (); }else { -spin_lock_bh (&p->br->hash_lock); - err =fdb_add_entry (p, addr, ndm->ndm_state, +spin_lock_bh (&br->hash_lock); + err =fdb_add_entry (br, p, addr, ndm->ndm_state, nlh_flags, vid); -spin_unlock_bh (&p->br->hash_lock); +spin_unlock_bh (&br->hash_lock); }return err;
@@ -878,6 +889,7 @@ int br_fdb_add(struct ndmsg *ndm, struct dev->name);return -EINVAL ; } + br = p->br; vg =nbp_vlan_group (p); }
@@ -889,15 +901,9 @@ int br_fdb_add(struct ndmsg *ndm, struct }/* VID was specified, so use it. */ -if (dev->priv_flags &IFF_EBRIDGE ) - err =br_fdb_insert (br,NULL , addr, vid); -else - err =__br_fdb_add (ndm, p, addr, nlh_flags, vid); + err =__br_fdb_add (ndm, br, p, addr, nlh_flags, vid); }else { -if (dev->priv_flags &IFF_EBRIDGE ) - err =br_fdb_insert (br,NULL , addr,0 ); -else - err =__br_fdb_add (ndm, p, addr, nlh_flags,0 ); + err =__br_fdb_add (ndm, br, p, addr, nlh_flags,0 );if (err || !vg || !vg->num_vlans) goto out;
@@ -908,11 +914,7 @@ int br_fdb_add(struct ndmsg *ndm, structlist_for_each_entry (v, &vg->vlan_list, vlist) {if (!br_vlan_should_use (v))continue ; -if (dev->priv_flags &IFF_EBRIDGE ) - err =br_fdb_insert (br,NULL , addr, v->vid); -else - err =__br_fdb_add (ndm, p, addr, nlh_flags, - v->vid); + err =__br_fdb_add (ndm, br, p, addr, nlh_flags, v->vid);if (err) goto out; }
Greg Kroah-Hartman
maintainerPATCH 306/312 bna: add missing per queue ethtool stat
In reply to Greg Kroah-HartmanFrom: Ivan Vecera <[email protected]>
commit 2835d2d9e366a2985b24051d228333bfba82f3a7 upstream.
Commit ba5ca784 "bna: check for dma mapping errors" added besides other things a statistic that counts number of DMA buffer mapping failures per each Rx queue. This counter is not included in ethtool stats output.
--- drivers/net/ethernet/brocade/bna/bnad_ethtool.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
drivers/net/ethernet/brocade/bna/bnad_ethtool.c +6 -1
@@ -31,7 +31,7 @@#define BNAD_NUM_TXF_COUNTERS 12#define BNAD_NUM_RXF_COUNTERS 10#define BNAD_NUM_CQ_COUNTERS (3 + 5) -#define BNAD_NUM_RXQ_COUNTERS 6 +#define BNAD_NUM_RXQ_COUNTERS 7#define BNAD_NUM_TXQ_COUNTERS 5#define BNAD_ETHTOOL_STATS_NUM \
@@ -658,6 +658,8 @@ bnad_get_strings(struct net_device *netd string +=ETH_GSTRING_LEN ;sprintf (string,"rxq%d_allocbuf_failed" , q_num); string +=ETH_GSTRING_LEN ; +sprintf (string,"rxq%d_mapbuf_failed" , q_num); + string +=ETH_GSTRING_LEN ;sprintf (string,"rxq%d_producer_index" , q_num); string +=ETH_GSTRING_LEN ;sprintf (string,"rxq%d_consumer_index" , q_num);
@@ -678,6 +680,9 @@ bnad_get_strings(struct net_device *netdsprintf (string,"rxq%d_allocbuf_failed" , q_num); string +=ETH_GSTRING_LEN ; +sprintf (string,"rxq%d_mapbuf_failed" , + q_num); + string +=ETH_GSTRING_LEN ;sprintf (string,"rxq%d_producer_index" , q_num); string +=ETH_GSTRING_LEN ;
Greg Kroah-Hartman
maintainerPATCH 307/312 net: skbuff: Remove errornous length validation in skb_vlan_pop()
In reply to Greg Kroah-HartmanFrom: Shmulik Ladkani <[email protected]>
commit 636c2628086e40c86dac7ddc84a1c4b4fcccc6e3 upstream.
In 93515d53b1
"net: move vlan pop/push functions into common code"skb_vlan_pop was moved from its private location in openvswitch to skbuff common code.
In case skb has non hw-accel vlan tag, the original 'pop_vlan()' assured that skb->len is sufficient (if skb->len < VLAN_ETH_HLEN then pop was considered a no-op).
This validation was moved as is into the new common 'skb_vlan_pop'.
Alas, in its original location (openvswitch), there was a guarantee that 'data' points to the mac_header, therefore the 'skb->len < VLAN_ETH_HLEN' condition made sense.
However there's no such guarantee in the generic 'skb_vlan_pop'.
For short packets received in rx path going through 'skb_vlan_pop', this causes 'skb_vlan_pop' to fail pop-ing a valid vlan hdr (in the non hw-accel case) or to fail moving next tag into hw-accel tag.
Remove the 'skb->len < VLAN_ETH_HLEN' condition entirely:
It is superfluous since inner '__skb_vlan_pop' already verifies there are VLAN_ETH_HLEN writable bytes at the mac_header.
Note this presents a slight change to skb_vlan_pop() users:
In case total length is smaller than VLAN_ETH_HLEN, skb_vlan_pop() now returns an error, as opposed to previous "no-op" behavior. Existing callers (e.g. tc act vlan, ovs) usually drop the packet if 'skb_vlan_pop' fails.
--- net/core/skbuff.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-)
net/core/skbuff.c +4 -6
@@ -4451,9 +4451,8 @@ int skb_vlan_pop(struct sk_buff *skb)if (likely (skb_vlan_tag_present (skb))) { skb->vlan_tci =0 ; }else { -if (unlikely ((skb->protocol !=htons (ETH_P_8021Q ) && - skb->protocol !=htons (ETH_P_8021AD )) || - skb->len <VLAN_ETH_HLEN )) +if (unlikely (skb->protocol !=htons (ETH_P_8021Q ) && + skb->protocol !=htons (ETH_P_8021AD )))return 0 ; err =__skb_vlan_pop (skb, &vlan_tci);
@@ -4461,9 +4460,8 @@ int skb_vlan_pop(struct sk_buff *skb)return err; }/* move next vlan tag to hw accel tag */ -if (likely ((skb->protocol !=htons (ETH_P_8021Q ) && - skb->protocol !=htons (ETH_P_8021AD )) || - skb->len <VLAN_ETH_HLEN )) +if (likely (skb->protocol !=htons (ETH_P_8021Q ) && + skb->protocol !=htons (ETH_P_8021AD )))return 0 ; vlan_proto = skb->protocol;
Greg Kroah-Hartman
maintainerPATCH 308/312 net: ep93xx_eth: Do not crash unloading module
In reply to Greg Kroah-HartmanFrom: Florian Fainelli <[email protected]>
commit c823abac17926767fb50175e098f087a6ac684c3 upstream.
When we unload the ep93xx_eth, whether we have opened the network interface or not, we will either hit a kernel paging request error, or a simple NULL pointer de-reference because:
- if ep93xx_open has been called, we have created a valid DMA mapping for ep->descs, when we call ep93xx_stop, we also call ep93xx_free_buffers, ep->descs now has a stale value
- if ep93xx_open has not been called, we have a NULL pointer for ep->descs, so performing any operation against that address just won't work
Fix this by adding a NULL pointer check for ep->descs which means that ep93xx_free_buffers() was able to successfully tear down the descriptors and free the DMA cookie as well.
--- drivers/net/ethernet/cirrus/ep93xx_eth.c | 4 ++++ 1 file changed, 4 insertions(+)
drivers/net/ethernet/cirrus/ep93xx_eth.c +4 -0
@@ -468,6 +468,9 @@ static void ep93xx_free_buffers(struct estruct device *dev = ep->dev->dev.parent;int i; +if (!ep->descs) +return ; +for (i =0 ; i <RX_QUEUE_ENTRIES ; i++) {dma_addr_t d;
@@ -490,6 +493,7 @@ static void ep93xx_free_buffers(struct edma_free_coherent (dev,sizeof (struct ep93xx_descs ), ep->descs, ep->descs_dma_addr); + ep->descs =NULL ; }static int ep93xx_alloc_buffers (struct ep93xx_priv *ep)
Greg Kroah-Hartman
maintainerPATCH 309/312 macvlan: Fix potential use-after free for broadcasts
In reply to Greg Kroah-HartmanFrom: Herbert Xu <[email protected]>
commit 260916dfb48c374f7840f3b86e69afd3afdb6e96 upstream.
When we postpone a broadcast packet we save the source port in the skb if it is local. However, the source port can disappear before we get a chance to process the packet.
This patch fixes this by holding a ref count on the netdev.
It also delays the skb->cb modification until after we allocate the new skb as you should not modify shared skbs.
--- drivers/net/macvlan.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-)
drivers/net/macvlan.c +8 -2
@@ -305,6 +305,8 @@ static void macvlan_process_broadcast(strcu_read_unlock (); +if (src) +dev_put (src->dev);kfree_skb (skb);cond_resched ();
@@ -312,6 +314,7 @@ static void macvlan_process_broadcast(st }static void macvlan_broadcast_enqueue (struct macvlan_port *port, +const struct macvlan_dev *src,struct sk_buff *skb) {struct sk_buff *nskb;
@@ -321,8 +324,12 @@ static void macvlan_broadcast_enqueue(stif (!nskb) goto err; +MACVLAN_SKB_CB (nskb)->src = src; +spin_lock (&port->bc_queue.lock);if (skb_queue_len (&port->bc_queue) <MACVLAN_BC_QUEUE_LEN ) { +if (src) +dev_hold (src->dev);__skb_queue_tail (&port->bc_queue, nskb); err =0 ; }
@@ -432,8 +439,7 @@ static rx_handler_result_t macvlan_handl goto out; } -MACVLAN_SKB_CB (skb)->src = src; -macvlan_broadcast_enqueue (port, skb); +macvlan_broadcast_enqueue (port, src, skb);return RX_HANDLER_PASS ; }
Greg Kroah-Hartman
maintainerPATCH 310/312 sctp: Fix SHUTDOWN CTSN Ack in the peer restart case
In reply to Greg Kroah-HartmanFrom: Jere Leppänen <[email protected]>
commit 12dfd78e3a74825e6f0bc8df7ef9f938fbc6bfe3 upstream.
When starting shutdown in sctp_sf_do_dupcook_a(), get the value for SHUTDOWN Cumulative TSN Ack from the new association, which is reconstructed from the cookie, instead of the old association, which the peer doesn't have anymore.
Otherwise the SHUTDOWN is either ignored or replied to with an ABORT by the peer because CTSN Ack doesn't match the peer's Initial TSN.
--- net/sctp/sm_make_chunk.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
net/sctp/sm_make_chunk.c +5 -1
@@ -857,7 +857,11 @@ struct sctp_chunk *sctp_make_shutdown(cosctp_shutdownhdr_t shut;__u32 ctsn; - ctsn =sctp_tsnmap_get_ctsn (&asoc->peer.tsn_map); +if (chunk && chunk->asoc) + ctsn =sctp_tsnmap_get_ctsn (&chunk->asoc->peer.tsn_map); +else + ctsn =sctp_tsnmap_get_ctsn (&asoc->peer.tsn_map); + shut.cum_tsn_ack =htonl (ctsn); retval = sctp_make_control(asoc, SCTP_CID_SHUTDOWN, 0,
Greg Kroah-Hartman
maintainerPATCH 311/312 ALSA: hda: Match both PCI ID and SSID for driver blacklist
In reply to Greg Kroah-HartmanFrom: Takashi Iwai <[email protected]>
commit 977dfef40c8996b69afe23a9094d184049efb7bb upstream.
The commit 3c6fd1f07ed0 ("ALSA: hda: Add driver blacklist") added a new blacklist for the devices that are known to have empty codecs, and one of the entries was ASUS ROG Zenith II (PCI SSID 1043:874f). However, it turned out that the very same PCI SSID is used for the previous model that does have the valid HD-audio codecs and the change broke the sound on it.
Since the empty codec problem appear on the certain AMD platform (PCI ID 1022:1487), this patch changes the blacklist matching to both PCI ID and SSID using pci_match_id(). Also, the entry that was removed by the previous fix for ASUS ROG Zenigh II is re-added.
--- sound/pci/hda/hda_intel.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)
sound/pci/hda/hda_intel.c +5 -4
@@ -1977,9 +1977,10 @@ static const struct hdac_io_ops pci_hda_* some HD-audio PCI entries are exposed without any codecs, and such devices * should be ignored from the beginning. */ -static const struct snd_pci_quirk driver_blacklist[] = { -SND_PCI_QUIRK (0x1462 ,0xcb59 ,"MSI TRX40 Creator" ,0 ), -SND_PCI_QUIRK (0x1462 ,0xcb60 ,"MSI TRX40" ,0 ), +static const struct pci_device_id driver_blacklist[] = { + {PCI_DEVICE_SUB (0x1022 ,0x1487 ,0x1043 ,0x874f ) },/* ASUS ROG Zenith II / Strix */ + {PCI_DEVICE_SUB (0x1022 ,0x1487 ,0x1462 ,0xcb59 ) },/* MSI TRX40 Creator */ + {PCI_DEVICE_SUB (0x1022 ,0x1487 ,0x1462 ,0xcb60 ) },/* MSI TRX40 */ {} };
@@ -2002,7 +2003,7 @@ static int azx_probe(struct pci_dev *pcibool schedule_probe;int err; -if (snd_pci_quirk_lookup (pci, driver_blacklist)) { +if (pci_match_id (driver_blacklist, pci)) {dev_info (&pci->dev,"Skipping the blacklisted device\n" );return -ENODEV ; }
Greg Kroah-Hartman
maintainerPATCH 312/312 mac80211: add ieee80211_is_any_nullfunc()
In reply to Greg Kroah-HartmanFrom: Thomas Pedersen <[email protected]>
commit 30b2f0be23fb40e58d0ad2caf8702c2a44cda2e1 upstream.
commit 08a5bdde3812 ("mac80211: consider QoS Null frames for STA_NULLFUNC_ACKED") Fixed a bug where we failed to take into account a
nullfunc frame can be either non-QoS or QoS. It turns out
there is at least one more bug inieee80211_sta_tx_notify(), introduced in
commit 7b6ddeaf27ec ("mac80211: use QoS NDP for AP probing"),
where we forgot to check for the QoS variant and so
assumed the QoS nullfunc frame never went out
Fix this by adding a helper ieee80211_is_any_nullfunc() which consolidates the check for non-QoS and QoS nullfunc frames. Replace existing compound conditionals and add a couple more missing checks for QoS variant.
--- include/linux/ieee80211.h | 9 +++++++++ net/mac80211/mlme.c | 2 +- net/mac80211/rx.c | 8 +++----- net/mac80211/status.c | 5 ++--- net/mac80211/tx.c | 2 +- 5 files changed, 16 insertions(+), 10 deletions(-)
include/linux/ieee80211.h +9 -0
@@ -607,6 +607,15 @@ static inline bool ieee80211_is_qos_null }/** +* ieee80211_is_any_nullfunc - check if frame is regular or QoS nullfunc frame +* @fc: frame control bytes in little-endian byteorder +*/ +static inline bool ieee80211_is_any_nullfunc (__le16 fc) +{ +return (ieee80211_is_nullfunc (fc) ||ieee80211_is_qos_nullfunc (fc)); +} + +/** * ieee80211_is_bufferable_mmpdu - check if frame is bufferable MMPDU * @fc: frame control field in little-endian byteorder */
net/mac80211/mlme.c +1 -1
@@ -2283,7 +2283,7 @@ void ieee80211_sta_tx_notify(struct ieeeif (!ieee80211_is_data (hdr->frame_control))return ; -if (ieee80211_is_nullfunc (hdr->frame_control) && +if (ieee80211_is_any_nullfunc (hdr->frame_control) && sdata->u.mgd.probe_send_count >0 ) {if (ack)ieee80211_sta_reset_conn_monitor (sdata);
net/mac80211/rx.c +3 -5
@@ -1110,8 +1110,7 @@ ieee80211_rx_h_check_dup(struct ieee8021return RX_CONTINUE ;if (ieee80211_is_ctl (hdr->frame_control) || -ieee80211_is_nullfunc (hdr->frame_control) || -ieee80211_is_qos_nullfunc (hdr->frame_control) || +ieee80211_is_any_nullfunc (hdr->frame_control) ||is_multicast_ether_addr (hdr->addr1))return RX_CONTINUE ;
@@ -1487,8 +1486,7 @@ ieee80211_rx_h_sta_process(struct ieee80* Drop (qos-)data::nullfunc frames silently, since they * are used only to control station power saving mode. */ -if (ieee80211_is_nullfunc (hdr->frame_control) || -ieee80211_is_qos_nullfunc (hdr->frame_control)) { +if (ieee80211_is_any_nullfunc (hdr->frame_control)) {I802_DEBUG_INC (rx->local->rx_handlers_drop_nullfunc);/*
@@ -1977,7 +1975,7 @@ static int ieee80211_drop_unencrypted(st/* Drop unencrypted frames if key is set. */ if (unlikely (!ieee80211_has_protected (fc) && - !ieee80211_is_nullfunc (fc) && + !ieee80211_is_any_nullfunc (fc) &&ieee80211_is_data (fc) && rx->key))return -EACCES ;
net/mac80211/status.c +2 -3
@@ -474,8 +474,7 @@ static void ieee80211_report_ack_skb(strrcu_read_lock (); sdata =ieee80211_sdata_from_skb (local, skb);if (sdata) { -if (ieee80211_is_nullfunc (hdr->frame_control) || -ieee80211_is_qos_nullfunc (hdr->frame_control)) +if (ieee80211_is_any_nullfunc (hdr->frame_control))cfg80211_probe_status (sdata->dev, hdr->addr1, cookie, acked,GFP_ATOMIC );
@@ -905,7 +904,7 @@ void ieee80211_tx_status(struct ieee8021I802_DEBUG_INC (local->dot11FailedCount); } -if (ieee80211_is_nullfunc (fc) &&ieee80211_has_pm (fc) && +if (ieee80211_is_any_nullfunc (fc) &&ieee80211_has_pm (fc) &&ieee80211_hw_check (&local->hw,REPORTS_TX_ACK_STATUS ) && !(info->flags &IEEE80211_TX_CTL_INJECTED ) && local->ps_sdata && !(local->scanning)) {
net/mac80211/tx.c +1 -1
@@ -291,7 +291,7 @@ ieee80211_tx_h_check_assoc(struct ieee80if (unlikely (test_bit (SCAN_SW_SCANNING , &tx->local->scanning)) &&test_bit (SDATA_STATE_OFFCHANNEL , &tx->sdata->state) && !ieee80211_is_probe_req (hdr->frame_control) && - !ieee80211_is_nullfunc (hdr->frame_control)) + !ieee80211_is_any_nullfunc (hdr->frame_control))/* * When software scanning only nullfunc frames (to notify * the sleep state to the AP) and probe requests (for the
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
This is not a complete list of errors.
arm64:allmodconfig
drivers/spi/spi-rockchip.c: In function 'rockchip_spi_prepare_dma':drivers/spi/spi-rockchip.c:461:19: error: 'struct dma_slave_caps' has no member named 'max_burst'
arm:allmodconfig
drivers/mtd/nand/pxa3xx_nand.c: In function 'alloc_nand_resource':drivers/mtd/nand/pxa3xx_nand.c:1753:7: error: 'np' undeclared
mips:defconfig
Lots of
In file included from ./arch/mips/include/asm/fpu.h:24,
from arch/mips/kernel/branch.c:16:./arch/mips/include/asm/msa.h: In function 'read_msa_ir':./arch/mips/include/asm/msa.h:204:2: error: expected ':' or ')' before '_ASM_INSN_IF_MIPS'
and similar errors.
arch/mips/mm/tlbex.c: In function 'config_htw_params':arch/mips/mm/tlbex.c:2334:13: error: 'MIPS_PWSIZE_PS_MASK' undeclared
All mips builds are badly broken.
Guenter
Greg Kroah-Hartman
maintainerQuoted 11 lines · Guenter Roeck
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
This is not a complete list of errors.
Yeah, I knew this was going to be a rough one. I was hoping the "early warning" messages from Linaro would have caught most of these, oh well :(
Quoted 5 lines
arm64:allmodconfig
drivers/spi/spi-rockchip.c: In function 'rockchip_spi_prepare_dma':drivers/spi/spi-rockchip.c:461:19: error: 'structdma_slave_caps' has no member named 'max_burst'
Now fixed.
Quoted 4 lines
arm:allmodconfig
drivers/mtd/nand/pxa3xx_nand.c: In function 'alloc_nand_resource':drivers/mtd/nand/pxa3xx_nand.c:1753:7: error: 'np' undeclared
Now fixed.
Quoted 9 lines
mips:defconfig
Lots of
In file included from
./arch/mips/include/asm/fpu.h:24,
fromarch/mips/kernel/branch.c:16:./arch/mips/include/asm/msa.h: In function 'read_msa_ir':./arch/mips/include/asm/msa.h:204:2: error: expected ':' or ')' before '_ASM_INSN_IF_MIPS'
Quoted 6 lines
and similar errors.
arch/mips/mm/tlbex.c: In function 'config_htw_params':arch/mips/mm/tlbex.c:2334:13: error: 'MIPS_PWSIZE_PS_MASK' undeclaredAll mips builds are badly broken.
Odd, I thought I had tested mips locally, I guess I got it completly wrong... Let me go through those again...
I'll push out a -rc2 after digging into the mips mess...
greg k-h
Quoted 11 lines · Greg Kroah-Hartman
Quoted 10 lines · Guenter Roeck
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
Quoted 6 lines
Quoted 1 line
This is not a complete list of errors.
Yeah, I knew this was going to be a rough one. I was hoping the "early warning" messages from Linaro would have caught most of these, oh well :(
To be fair, I had noticed the errors yesterday. I just thought this was so bad that it looked like a stray (bad) push to me, and I didn't send feedback. Maybe I should always send feedback if I see errors prior to an -rc release. I don't want to spam people with noise, so I am not sure. Any thoughts on that ?
Guenter
Greg Kroah-Hartman
maintainerQuoted 19 lines · Guenter Roeck
Quoted 11 lines · Greg Kroah-Hartman
Quoted 10 lines · Guenter Roeck
Quoted 8 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
Quoted 6 lines
Quoted 1 line
This is not a complete list of errors.
Yeah, I knew this was going to be a rough one. I was hoping the "early warning" messages from Linaro would have caught most of these, oh well :(
Quoted 5 lines
To be fair, I had noticed the errors yesterday. I just thought this was so bad that it looked like a stray (bad) push to me, and I didn't send feedback. Maybe I should always send feedback if I see errors prior to an -rc release. I don't want to spam people with noise, so I am not sure. Any thoughts on that ?
If the tree is broken, I'd like to know about it. Rarely do I know that it is broken and leave it alone. One exception would be last night when it was too late for me to fix the build issues, but Sasha was kind enough to do so.
So it's not noise to me, if it's easy enough for you to do so.
I think I fixed up the MIPS stuff, turns out gcc was just crashing on a filesystem driver with an internal error. I think I need a different version of gcc if I want to build more arches...
I'll go push out a -rc with this fixed up now, thanks.
greg k-h
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.223-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.4.y
and the diffstat can be found below.thanks,
greg k-h
<trim>
Quoted 2 lines
Addy Ke <[email protected]>
spi: rockchip: modify DMA max burst to 1
While building kernel Image for arm architecture the following error noticed on stable-rc 4.4 kernel branch
# make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm
CROSS_COMPILE=arm-linux-gnueabihf- HOSTCC=gcc CC="sccache
arm-linux-gnueabihf-gcc" O=build zImage
#
#
# make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm
CROSS_COMPILE=arm-linux-gnueabihf- HOSTCC=gcc CC="sccache
arm-linux-gnueabihf-gcc" O=build modules
#
../drivers/spi/spi-rockchip.c: In function ‘rockchip_spi_prepare_dma’:
../drivers/spi/spi-rockchip.c:461:19: error: ‘struct dma_slave_caps’
has no member named ‘max_burst’
461 | if (rs->dma_caps.max_burst > 4)
| ^
../drivers/spi/spi-rockchip.c:481:19: error: ‘struct dma_slave_caps’
has no member named ‘max_burst’
481 | if (rs->dma_caps.max_burst > 4)
| ^full build log,
https://gitlab.com/Linaro/lkft/kernel-runs/-/jobs/544289003
Signature · 2 lines
Linaro LKFT https://lkft.linaro.org
Greg Kroah-Hartman
maintainerQuoted 37 lines · Naresh Kamboju
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 4.4.223 release. There are 312 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Sun, 10 May 2020 12:29:44 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.4.223-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-4.4.y
and the diffstat can be found below.thanks,
greg k-h
<trim>
Quoted 2 lines
Addy Ke <[email protected]>
spi: rockchip: modify DMA max burst to 1While building kernel Image for arm architecture the following error noticed on stable-rc 4.4 kernel branch
# make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- HOSTCC=gcc CC="sccache arm-linux-gnueabihf-gcc" O=build zImage # # # make -sk KBUILD_BUILD_USER=TuxBuild -C/linux -j16 ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- HOSTCC=gcc CC="sccache arm-linux-gnueabihf-gcc" O=build modules # ../drivers/spi/spi-rockchip.c: In function ‘rockchip_spi_prepare_dma’:
Quoted 8 lines
../drivers/spi/spi-rockchip.c:461:19: error: ‘struct dma_slave_caps’
has no member named ‘max_burst’
461 | if (rs->dma_caps.max_burst > 4)
| ^
../drivers/spi/spi-rockchip.c:481:19: error: ‘struct dma_slave_caps’
has no member named ‘max_burst’
481 | if (rs->dma_caps.max_burst > 4)
| ^I just dropped the offending patch, Guenter beat you to it :)
greg k-h