Netdev List
 help / color / mirror / Atom feed
* [PATCH v2 12/19] irda: sh_irda: Enable the driver on all ARM platforms
From: Laurent Pinchart @ 2013-10-29 22:37 UTC (permalink / raw)
  To: linux-sh; +Cc: linux-arm-kernel, Samuel Ortiz, netdev
In-Reply-To: <1383086274-11049-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>

Renesas ARM platforms are transitioning from single-platform to
multi-platform kernels using the new ARCH_SHMOBILE_MULTI. Make the
driver available on all ARM platforms to enable it on both ARCH_SHMOBILE
and ARCH_SHMOBILE_MULTI, and increase build testing coverage with
COMPILE_TEST.

Cc: Samuel Ortiz <samuel@sortiz.org>
Cc: netdev@vger.kernel.org
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 drivers/net/irda/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/irda/Kconfig b/drivers/net/irda/Kconfig
index 2a30193..e04b907 100644
--- a/drivers/net/irda/Kconfig
+++ b/drivers/net/irda/Kconfig
@@ -403,7 +403,7 @@ config MCS_FIR
 
 config SH_IRDA
 	tristate "SuperH IrDA driver"
-	depends on IRDA && ARCH_SHMOBILE
+	depends on IRDA && (ARM || COMPILE_TEST)
 	help
 	  Say Y here if your want to enable SuperH IrDA devices.
 
-- 
1.8.1.5


^ permalink raw reply related

* [PATCH v2 11/19] net: sh_eth: Set receive alignment correctly on all ARM platforms
From: Laurent Pinchart @ 2013-10-29 22:37 UTC (permalink / raw)
  To: linux-sh; +Cc: linux-arm-kernel, David S. Miller, Sergei Shtylyov, netdev
In-Reply-To: <1383086274-11049-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>

Renesas ARM platforms are transitioning from single-platform to
multi-platform kernels using the new ARCH_SHMOBILE_MULTI. Configure the
receive alignement correctly on all ARM platforms to enable the driver
on both ARCH_SHMOBILE and ARCH_SHMOBILE_MULTI.

Cc: "David S. Miller" <davem@davemloft.net>
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Cc: netdev@vger.kernel.org
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 drivers/net/ethernet/renesas/sh_eth.c | 2 +-
 drivers/net/ethernet/renesas/sh_eth.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c
index b57c278..990fd5b 100644
--- a/drivers/net/ethernet/renesas/sh_eth.c
+++ b/drivers/net/ethernet/renesas/sh_eth.c
@@ -809,7 +809,7 @@ out:
 	return ret;
 }
 
-#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARCH_SHMOBILE)
+#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARM)
 static void sh_eth_set_receive_align(struct sk_buff *skb)
 {
 	int reserve;
diff --git a/drivers/net/ethernet/renesas/sh_eth.h b/drivers/net/ethernet/renesas/sh_eth.h
index a0db02c..41509f7 100644
--- a/drivers/net/ethernet/renesas/sh_eth.h
+++ b/drivers/net/ethernet/renesas/sh_eth.h
@@ -165,7 +165,7 @@ enum {
 };
 
 /* Driver's parameters */
-#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARCH_SHMOBILE)
+#if defined(CONFIG_CPU_SH4) || defined(CONFIG_ARM)
 #define SH4_SKB_RX_ALIGN	32
 #else
 #define SH2_SH3_SKB_RX_ALIGN	2
-- 
1.8.1.5


^ permalink raw reply related

* [PATCH v2 00/19] Enable various Renesas drivers on all ARM platforms
From: Laurent Pinchart @ 2013-10-29 22:37 UTC (permalink / raw)
  To: linux-sh-u79uwXL29TY76Z2rM5mHXA
  Cc: linux-fbdev-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang, Linus Walleij,
	Guennadi Liakhovetski, Thierry Reding,
	linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA, Vinod Koul, Magnus Damm,
	Eduardo Valentin, Tomi Valkeinen,
	linux-serial-u79uwXL29TY76Z2rM5mHXA,
	linux-input-u79uwXL29TY76Z2rM5mHXA, Zhang Rui, Chris Ball,
	Jean-Christophe Plagniol-Villard,
	linux-media-u79uwXL29TY76Z2rM5mHXA,
	linux-pwm-u79uwXL29TY76Z2rM5mHXA, Samuel Ortiz,
	linux-pm-u79uwXL29TY76Z2rM5mHXA, Ian Molton, Mark Brown,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Sergei Shtylyov, Greg 

Hello,

This patch series, based on v3.12-rc7, prepares various Renesas drivers
for migration to multiplatform kernels by enabling their compilation or
otherwise fixing them on all ARM platforms. The patches are pretty
straightforward and are described in their commit message.

Changes since v1:

- The drivers can also be selected when COMPILE_TEST is enabled, regardless of
  the architecture. This should provide a good compromise between wide build
  test coverage and not clobbering configuration with drivers useless on
  non-SuperH, non-ARM platforms.

- DMA configuration is now unconditional in patch 08/19

I'd like to get all these patches merged in v3.14. As they will need to go
through their respective subsystems' trees, I would appreciate if all
maintainers involved could notify me when they merge patches from this series
in their tree to help me tracking the merge status. I don't plan to send pull
requests individually for these patches, and I will repost patches
individually if changes are requested during review.

If you believe the issue should be solved in a different way please reply to
the cover letter to let other maintainers chime in.

Cc: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org>
Cc: "David S. Miller" <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
Cc: David Woodhouse <dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
Cc: Dmitry Torokhov <dmitry.torokhov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Eduardo Valentin <eduardo.valentin-l0cyMroinI0@public.gmane.org>
Cc: Greg Kroah-Hartman <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
Cc: Guennadi Liakhovetski <g.liakhovetski+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Ian Molton <ian-zdned+2MO1+9FHfhHBbuYA@public.gmane.org>
Cc: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org
Cc: Jean-Christophe Plagniol-Villard <plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>
Cc: Joerg Roedel <joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
Cc: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Magnus Damm <magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Mauro Carvalho Chehab <m.chehab-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
Cc: Samuel Ortiz <samuel-jcdQHdrhKHMdnm+yROfE0A@public.gmane.org>
Cc: Sergei Shtylyov <sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org>
Cc: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Tomi Valkeinen <tomi.valkeinen-l0cyMroinI0@public.gmane.org>
Cc: Vinod Koul <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org>
Cc: Zhang Rui <rui.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-fbdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Cc: linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

Laurent Pinchart (19):
  serial: sh-sci: Enable the driver on all ARM platforms
  DMA: shdma: Enable the driver on all ARM platforms
  i2c: sh_mobile: Enable the driver on all ARM platforms
  input: sh_keysc: Enable the driver on all ARM platforms
  iommu: shmobile: Enable the driver on all ARM platforms
  i2c: rcar: Enable the driver on all ARM platforms
  v4l: sh_vou: Enable the driver on all ARM platforms
  mmc: sdhi: Enable the driver on all ARM platforms
  mmc: sh_mmcif: Enable the driver on all ARM platforms
  mtd: sh_flctl: Enable the driver on all ARM platforms
  net: sh_eth: Set receive alignment correctly on all ARM platforms
  irda: sh_irda: Enable the driver on all ARM platforms
  pinctrl: sh-pfc: Enable the driver on all ARM platforms
  pwm: pwm-renesas-tpu: Enable the driver on all ARM platforms
  sh: intc: Enable the driver on all ARM platforms
  spi: sh_msiof: Enable the driver on all ARM platforms
  spi: sh_hspi: Enable the driver on all ARM platforms
  thermal: rcar-thermal: Enable the driver on all ARM platforms
  fbdev: sh-mobile-lcdcfb: Enable the driver on all ARM platforms

 drivers/dma/sh/Kconfig                | 2 +-
 drivers/dma/sh/shdmac.c               | 6 +++---
 drivers/i2c/busses/Kconfig            | 4 ++--
 drivers/input/keyboard/Kconfig        | 2 +-
 drivers/iommu/Kconfig                 | 2 +-
 drivers/media/platform/Kconfig        | 2 +-
 drivers/mmc/host/Kconfig              | 4 ++--
 drivers/mmc/host/tmio_mmc_dma.c       | 4 +---
 drivers/mtd/nand/Kconfig              | 2 +-
 drivers/net/ethernet/renesas/sh_eth.c | 2 +-
 drivers/net/ethernet/renesas/sh_eth.h | 2 +-
 drivers/net/irda/Kconfig              | 2 +-
 drivers/pinctrl/Makefile              | 2 +-
 drivers/pinctrl/sh-pfc/Kconfig        | 2 +-
 drivers/pwm/Kconfig                   | 2 +-
 drivers/sh/intc/Kconfig               | 2 +-
 drivers/spi/Kconfig                   | 4 ++--
 drivers/thermal/Kconfig               | 2 +-
 drivers/tty/serial/Kconfig            | 2 +-
 drivers/video/Kconfig                 | 6 +++---
 20 files changed, 27 insertions(+), 29 deletions(-)

-- 
Regards,

Laurent Pinchart

^ permalink raw reply

* [PATCH] bnx2: Use dev_kfree_skb_any() in bnx2_tx_int()
From: David Mackey @ 2013-10-29 22:16 UTC (permalink / raw)
  To: mchan, netdev; +Cc: linux-kernel, David Mackey

Using dev_kfree_skb_any() will resolve the below issue when a
netconsole message is transmitted in an irq.

 ------------[ cut here ]------------
 WARNING: at net/core/skbuff.c:451 skb_release_head_state+0x7b/0xe1()
 ...
 Pid: 0, comm: swapper/2 Not tainted 3.4.55 #1
 Call Trace:
  <IRQ>  [<ffffffff8104934c>] warn_slowpath_common+0x85/0x9d
  [<ffffffff8104937e>] warn_slowpath_null+0x1a/0x1c
  [<ffffffff81429aa7>] skb_release_head_state+0x7b/0xe1
  [<ffffffff814297e1>] __kfree_skb+0x16/0x81
  [<ffffffff814298a0>] consume_skb+0x54/0x69
  [<ffffffffa015925b>] bnx2_tx_int.clone.6+0x1b0/0x33e [bnx2]
  [<ffffffff8129c54d>] ? unmask_msi_irq+0x10/0x12
  [<ffffffffa015aa06>] bnx2_poll_work+0x3a/0x73 [bnx2]
  [<ffffffffa015aa73>] bnx2_poll_msix+0x34/0xb4 [bnx2]
  [<ffffffff814466a2>] netpoll_poll_dev+0xb9/0x1b7
  [<ffffffff814467d7>] ? find_skb+0x37/0x82
  [<ffffffff814461ed>] netpoll_send_skb_on_dev+0x117/0x200
  [<ffffffff81446a52>] netpoll_send_udp+0x230/0x242
  [<ffffffffa0174296>] write_msg+0xa7/0xfb [netconsole]
  [<ffffffff814258a4>] ? sk_free+0x1c/0x1e
  [<ffffffff810495ad>] __call_console_drivers+0x7d/0x8f
  [<ffffffff81049674>] _call_console_drivers+0xb5/0xd0
  [<ffffffff8104a134>] console_unlock+0x131/0x219
  [<ffffffff8104a7f9>] vprintk+0x3bc/0x405
  [<ffffffff81460073>] ? NF_HOOK.clone.1+0x4c/0x53
  [<ffffffff81460308>] ? ip_rcv+0x23c/0x268
  [<ffffffff814ddd4f>] printk+0x68/0x71
  [<ffffffff813315b3>] __dev_printk+0x78/0x7a
  [<ffffffff813316b2>] dev_warn+0x53/0x55
  [<ffffffff8127f181>] ? swiotlb_unmap_sg_attrs+0x47/0x5c
  [<ffffffffa004f876>] complete_scsi_command+0x28a/0x4a0 [hpsa]
  [<ffffffffa004fadb>] finish_cmd+0x4f/0x66 [hpsa]
  [<ffffffffa004fd97>] process_indexed_cmd+0x48/0x54 [hpsa]
  [<ffffffffa004ff25>] do_hpsa_intr_msi+0x4e/0x77 [hpsa]
  [<ffffffff810baebb>] handle_irq_event_percpu+0x5e/0x1b6
  [<ffffffff81088a0b>] ? timekeeping_update+0x43/0x45
  [<ffffffff810bb04b>] handle_irq_event+0x38/0x54
  [<ffffffff8102bd1e>] ? ack_apic_edge+0x36/0x3a
  [<ffffffff810bd762>] handle_edge_irq+0xa5/0xc8
  [<ffffffff81010d56>] handle_irq+0x127/0x135
  [<ffffffff814e3426>] ? __atomic_notifier_call_chain+0x12/0x14
  [<ffffffff814e343c>] ? atomic_notifier_call_chain+0x14/0x16
  [<ffffffff814e897d>] do_IRQ+0x4d/0xb4
  [<ffffffff814dffea>] common_interrupt+0x6a/0x6a
  <EOI>  [<ffffffff812b7603>] ? intel_idle+0xd8/0x112
  [<ffffffff812b7603>] ? intel_idle+0xd8/0x112
  [<ffffffff812b75e9>] ? intel_idle+0xbe/0x112
  [<ffffffff814012fc>] cpuidle_enter+0x12/0x14
  [<ffffffff814019c2>] cpuidle_idle_call+0xd1/0x19b
  [<ffffffff81016551>] cpu_idle+0xb6/0xff
  [<ffffffff814d726b>] start_secondary+0xc8/0xca
 ---[ end trace 3f15cd66441c770d ]---

Signed-off-by: David Mackey <tdmackey@booleanhaiku.com>
---
 drivers/net/ethernet/broadcom/bnx2.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/broadcom/bnx2.c b/drivers/net/ethernet/broadcom/bnx2.c
index e838a3f..372cbb5 100644
--- a/drivers/net/ethernet/broadcom/bnx2.c
+++ b/drivers/net/ethernet/broadcom/bnx2.c
@@ -2868,7 +2868,7 @@ bnx2_tx_int(struct bnx2 *bp, struct bnx2_napi *bnapi, int budget)
 		sw_cons = BNX2_NEXT_TX_BD(sw_cons);
 
 		tx_bytes += skb->len;
-		dev_kfree_skb(skb);
+		dev_kfree_skb_any(skb);
 		tx_pkt++;
 		if (tx_pkt == budget)
 			break;
-- 
1.7.9.5

^ permalink raw reply related

* [PATCH NEXT] rtl8187: Increase RX queue depth
From: Larry Finger @ 2013-10-29 22:06 UTC (permalink / raw)
  To: linville-2XuSBdqkA4R54TAoqtyWWQ
  Cc: linux-wireless-u79uwXL29TY76Z2rM5mHXA, Larry Finger,
	netdev-u79uwXL29TY76Z2rM5mHXA

Under heavy load, the relatively small number of RX queue entries are
completely filled. With an increase from 16 to 32 entries, this condition
rarely happens.

Signed-off-by: Larry Finger <Larry.Finger-tQ5ms3gMjBLk1uMJSBkQmQ@public.gmane.org>
---
 drivers/net/wireless/rtl818x/rtl8187/dev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/rtl818x/rtl8187/dev.c b/drivers/net/wireless/rtl818x/rtl8187/dev.c
index 9a6edb0..ec9aa5b 100644
--- a/drivers/net/wireless/rtl818x/rtl8187/dev.c
+++ b/drivers/net/wireless/rtl818x/rtl8187/dev.c
@@ -416,7 +416,7 @@ static int rtl8187_init_urbs(struct ieee80211_hw *dev)
 	struct rtl8187_rx_info *info;
 	int ret = 0;
 
-	while (skb_queue_len(&priv->rx_queue) < 16) {
+	while (skb_queue_len(&priv->rx_queue) < 32) {
 		skb = __dev_alloc_skb(RTL8187_MAX_RX, GFP_KERNEL);
 		if (!skb) {
 			ret = -ENOMEM;
-- 
1.8.4

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related

* Re: [PATCH v2 2/2] net/benet: Make lancer_wait_ready() static
From: David Miller @ 2013-10-29 21:50 UTC (permalink / raw)
  To: shangw; +Cc: netdev, Sathya.Perla
In-Reply-To: <1383039057-28164-2-git-send-email-shangw@linux.vnet.ibm.com>

From: Gavin Shan <shangw@linux.vnet.ibm.com>
Date: Tue, 29 Oct 2013 17:30:57 +0800

> The function needn't to be public, so to make it as static.
> 
> Signed-off-by: Gavin Shan <shangw@linux.vnet.ibm.com>

Applied.

^ permalink raw reply

* Re: [PATCH v2 1/2] net/benet: Remove interface type
From: David Miller @ 2013-10-29 21:50 UTC (permalink / raw)
  To: shangw; +Cc: netdev, Sathya.Perla
In-Reply-To: <1383039057-28164-1-git-send-email-shangw@linux.vnet.ibm.com>

From: Gavin Shan <shangw@linux.vnet.ibm.com>
Date: Tue, 29 Oct 2013 17:30:56 +0800

> The interface type, which is being traced by "struct be_adapter::
> if_type", isn't used currently. So we can remove that safely
> according to Sathya's comments.
> 
> Signed-off-by: Gavin Shan <shangw@linux.vnet.ibm.com>

Applied.

^ permalink raw reply

* Re: [PATCH net 2/3] r8152: modify the tx flow
From: David Miller @ 2013-10-29 21:49 UTC (permalink / raw)
  To: hayeswang; +Cc: netdev, nic_swsd, linux-kernel, linux-usb
In-Reply-To: <1383033377-1178-3-git-send-email-hayeswang@realtek.com>

From: Hayes Wang <hayeswang@realtek.com>
Date: Tue, 29 Oct 2013 15:56:16 +0800

> Support stopping and waking tx queue. The maximum tx queue length
> is 60.

What is so special about the number 60?  It seems arbitrary, and if
it isn't arbitrary you haven't described why this value was choosen.

I've asked you politely last time around to significantly improve
the quality of your commit messages, and you haven't done this at
all.

I'm not applying any of these patches until your commit messages
properly describe your changes completely.

^ permalink raw reply

* Re: [PATCH 1/2] octeon_mgmt: remove double validation of mac address
From: David Daney @ 2013-10-29 21:45 UTC (permalink / raw)
  To: Laurent Navet, Luka Perkov
  Cc: davem, jiri, david.daney, wfp5p, gregkh, netdev, linux-kernel
In-Reply-To: <1383082053-12405-1-git-send-email-laurent.navet@gmail.com>

On 10/29/2013 02:27 PM, Laurent Navet wrote:
> Mac address validity is already checked in of_get_mac_address().
> No need to do it twice.
>
> Signed-off-by: Laurent Navet <laurent.navet@gmail.com>

Luka already sent this exact patch, and I acked his.

No need to do it twice.

David Daney


> ---
>   drivers/net/ethernet/octeon/octeon_mgmt.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/octeon/octeon_mgmt.c b/drivers/net/ethernet/octeon/octeon_mgmt.c
> index 622aa75..1b326cbc 100644
> --- a/drivers/net/ethernet/octeon/octeon_mgmt.c
> +++ b/drivers/net/ethernet/octeon/octeon_mgmt.c
> @@ -1545,7 +1545,7 @@ static int octeon_mgmt_probe(struct platform_device *pdev)
>
>   	mac = of_get_mac_address(pdev->dev.of_node);
>
> -	if (mac && is_valid_ether_addr(mac))
> +	if (mac)
>   		memcpy(netdev->dev_addr, mac, ETH_ALEN);
>   	else
>   		eth_hw_addr_random(netdev);
>

^ permalink raw reply

* Re: [PATCH net-next] netconsole: Convert to pr_<level>
From: David Miller @ 2013-10-29 21:43 UTC (permalink / raw)
  To: joe; +Cc: netdev, linux-kernel
In-Reply-To: <1382990001.8718.4.camel@joe-AO722>

From: Joe Perches <joe@perches.com>
Date: Mon, 28 Oct 2013 12:53:21 -0700

> Use a more current logging style.
> 
> Convert printks to pr_<level>.
> 
> Consolidate multiple printks into a single printk to avoid
> any possible dmesg interleaving.  Add a default "event" msg
> in case the listed types are ever expanded.
> 
> Signed-off-by: Joe Perches <joe@perches.com>

Applied, thanks Joe.

^ permalink raw reply

* Re: 3.12-rc7 regression - network panic from ipv6
From: David Miller @ 2013-10-29 21:42 UTC (permalink / raw)
  To: mroos; +Cc: hannes, linux-kernel, netdev, steffen.klassert
In-Reply-To: <alpine.SOC.1.00.1310292338010.22172@math.ut.ee>

From: Meelis Roos <mroos@linux.ee>
Date: Tue, 29 Oct 2013 23:38:28 +0200 (EET)

>> > Some bad news - in a system where 3.12-rc6 and earlier worked fine, 
>> > 3.12-rc7 panics or hangs repeatedly with network traffic (torrent being 
>> > good test). First there is BUG from ipv6 code, followed by panic.
>> 
>> Could you do a bisect on this? There seems to be one commit for this
>> particular function _decode_session6:
>> 
>> commit bafd4bd4dcfa13145db7f951251eef3e10f8c278
>> Author: Steffen Klassert <steffen.klassert@secunet.com>
>> Date:   Mon Sep 9 10:38:38 2013 +0200
>> 
>>     xfrm: Decode sessions with output interface.
>>     
>>     The output interface matching does not work on forward
>>     policy lookups, the output interface of the flowi is
>>     always 0. Fix this by setting the output interface when
>>     we decode the session.
>> 
>> Maybe try to just revert this change locally and try again?
> 
> Yes, just reverting this patch on top of rc7 gets rid of the problem for 
> me.

Steffen please fix this or I'll have to revert.

^ permalink raw reply

* Re: Realtek RTL8102E registers
From: Francois Romieu @ 2013-10-29 21:40 UTC (permalink / raw)
  To: Ivan Frederiks; +Cc: Linux r8169 crew
In-Reply-To: <526FCEAA.30102@speechpro.com>

Ivan Frederiks <frederiks@speechpro.com> :
[...]
> Maybe you have access to RTL8102E register description ?

No.

Realtek's hardware documentation policy isn't exactly opened. Your hardware
vendor may help.

[...]
> Issue description:
[snip]

kernel version and r81xx XID value as it should appear in dmesg would
be welcome.

-- 
Ueimor

^ permalink raw reply

* Re: [PATCH] bridge: pass correct vlan id to multicast code
From: David Miller @ 2013-10-29 21:40 UTC (permalink / raw)
  To: vyasevic; +Cc: netdev, shemminger, makita.toshiaki
In-Reply-To: <1382989507-23061-1-git-send-email-vyasevic@redhat.com>

From: Vlad Yasevich <vyasevic@redhat.com>
Date: Mon, 28 Oct 2013 15:45:07 -0400

> Currently multicast code attempts to extrace the vlan id from
> the skb even when vlan filtering is disabled.  This can lead
> to mdb entries being created with the wrong vlan id.
> Pass the already extracted vlan id to the multicast
> filtering code to make the correct id is used in
> creation as well as lookup.
> 
> Signed-off-by: Vlad Yasevich <vyasevic@redhat.com>

Applied, thanks Vlad and others.

^ permalink raw reply

* Re: 3.12-rc7 regression - network panic from ipv6
From: Meelis Roos @ 2013-10-29 21:38 UTC (permalink / raw)
  To: Hannes Frederic Sowa; +Cc: Linux Kernel list, netdev, steffen.klassert
In-Reply-To: <20131029210758.GA18323@order.stressinduktion.org>

> > Some bad news - in a system where 3.12-rc6 and earlier worked fine, 
> > 3.12-rc7 panics or hangs repeatedly with network traffic (torrent being 
> > good test). First there is BUG from ipv6 code, followed by panic.
> 
> Could you do a bisect on this? There seems to be one commit for this
> particular function _decode_session6:
> 
> commit bafd4bd4dcfa13145db7f951251eef3e10f8c278
> Author: Steffen Klassert <steffen.klassert@secunet.com>
> Date:   Mon Sep 9 10:38:38 2013 +0200
> 
>     xfrm: Decode sessions with output interface.
>     
>     The output interface matching does not work on forward
>     policy lookups, the output interface of the flowi is
>     always 0. Fix this by setting the output interface when
>     we decode the session.
> 
> Maybe try to just revert this change locally and try again?

Yes, just reverting this patch on top of rc7 gets rid of the problem for 
me.

-- 
Meelis Roos (mroos@linux.ee)

^ permalink raw reply

* Re: [GIT net] Open vSwitch
From: David Miller @ 2013-10-29 21:37 UTC (permalink / raw)
  To: jesse; +Cc: netdev, dev
In-Reply-To: <1382983491-24432-1-git-send-email-jesse@nicira.com>

From: Jesse Gross <jesse@nicira.com>
Date: Mon, 28 Oct 2013 11:04:50 -0700

> One patch for net/3.12 fixing an issue where devices could be in an
> invalid state they are removed while still attached to OVS.

Pulled, thanks Jesse.

^ permalink raw reply

* Re: [PATCH] net: x25: Fix dead URLs in Kconfig
From: David Miller @ 2013-10-29 21:35 UTC (permalink / raw)
  To: michael; +Cc: andrew.hendry, linux-x25, netdev, linux-kernel
In-Reply-To: <1382981592-7449-1-git-send-email-michael@drueing.de>

From: Michael Drüing <michael@drueing.de>
Date: Mon, 28 Oct 2013 18:33:12 +0100

> Update the URLs in the Kconfig file to the new pages at sangoma.com and cisco.com
> 
> Signed-off-by: Michael Drüing <michael@drueing.de>

Applied, thanks.

^ permalink raw reply

* Re: [PATCH net-next v2] net: sched: cls_bpf: add BPF-based classifier
From: David Miller @ 2013-10-29 21:34 UTC (permalink / raw)
  To: dborkman; +Cc: eric.dumazet, netdev, tgraf
In-Reply-To: <8b7eb31a8976ea086ff904e42514e4ab3d937712.1382974535.git.dborkman@redhat.com>

From: Daniel Borkmann <dborkman@redhat.com>
Date: Mon, 28 Oct 2013 16:43:02 +0100

> This work contains a lightweight BPF-based traffic classifier that can
> serve as a flexible alternative to ematch-based tree classification, i.e.
> now that BPF filter engine can also be JITed in the kernel. Naturally, tc
> actions and policies are supported as well with cls_bpf. Multiple BPF
> programs/filter can be attached for a class, or they can just as well be
> written within a single BPF program, that's really up to the user how he
> wishes to run/optimize the code, e.g. also for inversion of verdicts etc.
> The notion of a BPF program's return/exit codes is being kept as follows:

This looks great, although a lot of things this supports could be done
using u32 filters, but this is more flexible.

Applied, thanks.

^ permalink raw reply

* [PATCH 1/2] octeon_mgmt: remove double validation of mac address
From: Laurent Navet @ 2013-10-29 21:27 UTC (permalink / raw)
  To: davem; +Cc: jiri, david.daney, wfp5p, gregkh, netdev, linux-kernel,
	Laurent Navet

Mac address validity is already checked in of_get_mac_address().
No need to do it twice.

Signed-off-by: Laurent Navet <laurent.navet@gmail.com>
---
 drivers/net/ethernet/octeon/octeon_mgmt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/octeon/octeon_mgmt.c b/drivers/net/ethernet/octeon/octeon_mgmt.c
index 622aa75..1b326cbc 100644
--- a/drivers/net/ethernet/octeon/octeon_mgmt.c
+++ b/drivers/net/ethernet/octeon/octeon_mgmt.c
@@ -1545,7 +1545,7 @@ static int octeon_mgmt_probe(struct platform_device *pdev)
 
 	mac = of_get_mac_address(pdev->dev.of_node);
 
-	if (mac && is_valid_ether_addr(mac))
+	if (mac)
 		memcpy(netdev->dev_addr, mac, ETH_ALEN);
 	else
 		eth_hw_addr_random(netdev);
-- 
1.8.4.rc3

^ permalink raw reply related

* [PATCH] mvneta: remove double validation of mac address
From: Laurent Navet @ 2013-10-29 21:25 UTC (permalink / raw)
  To: thomas.petazzoni; +Cc: netdev, linux-kernel, Laurent Navet

Mac address validity is already checked in of_get_mac_address().
No need to do it twice.

Signed-off-by: Laurent Navet <laurent.navet@gmail.com>
---
 drivers/net/ethernet/marvell/mvneta.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/marvell/mvneta.c b/drivers/net/ethernet/marvell/mvneta.c
index e35bac7..7d99e695 100644
--- a/drivers/net/ethernet/marvell/mvneta.c
+++ b/drivers/net/ethernet/marvell/mvneta.c
@@ -2811,7 +2811,7 @@ static int mvneta_probe(struct platform_device *pdev)
 	}
 
 	dt_mac_addr = of_get_mac_address(dn);
-	if (dt_mac_addr && is_valid_ether_addr(dt_mac_addr)) {
+	if (dt_mac_addr) {
 		mac_from = "device tree";
 		memcpy(dev->dev_addr, dt_mac_addr, ETH_ALEN);
 	} else {
-- 
1.8.4.rc3

^ permalink raw reply related

* Re: 3.12-rc7 regression - network panic from ipv6
From: Julian Anastasov @ 2013-10-29 21:21 UTC (permalink / raw)
  To: Hannes Frederic Sowa
  Cc: Meelis Roos, Linux Kernel list, netdev, steffen.klassert
In-Reply-To: <20131029210758.GA18323@order.stressinduktion.org>


	Hello,

On Tue, 29 Oct 2013, Hannes Frederic Sowa wrote:

> On Tue, Oct 29, 2013 at 10:33:36PM +0200, Meelis Roos wrote:
> > Some bad news - in a system where 3.12-rc6 and earlier worked fine, 
> > 3.12-rc7 panics or hangs repeatedly with network traffic (torrent being 
> > good test). First there is BUG from ipv6 code, followed by panic.
> 
> Could you do a bisect on this? There seems to be one commit for this
> particular function _decode_session6:
> 
> commit bafd4bd4dcfa13145db7f951251eef3e10f8c278
> Author: Steffen Klassert <steffen.klassert@secunet.com>
> Date:   Mon Sep 9 10:38:38 2013 +0200
> 
>     xfrm: Decode sessions with output interface.
>     
>     The output interface matching does not work on forward
>     policy lookups, the output interface of the flowi is
>     always 0. Fix this by setting the output interface when
>     we decode the session.
> 
> Maybe try to just revert this change locally and try again?

	At first look this change can work only for
forwarding, for output the skb_dst is attached later
in icmpv6_push_pending_frames. May be a check for present
skb_dst is needed.

Regards

--
Julian Anastasov <ja@ssi.bg>

^ permalink raw reply

* Re: [PATCH] bgmac: separate RX descriptor setup code into a new function
From: David Miller @ 2013-10-29 21:21 UTC (permalink / raw)
  To: zajec5; +Cc: netdev
In-Reply-To: <1382967629-3794-1-git-send-email-zajec5@gmail.com>

From: Rafał Miłecki <zajec5@gmail.com>
Date: Mon, 28 Oct 2013 14:40:29 +0100

> This cleans code a bit and will be useful when allocating buffers in
> other places (like RX path, to avoid skb_copy_from_linear_data_offset).
> 
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>

Applied to net-next, thanks.

^ permalink raw reply

* Re:Answer back
From: Lee Hyuk @ 2013-10-29 14:34 UTC (permalink / raw)





I would like to discuss a very important crude oil project with
you,kindlyrevert back to me if this is your valid email address for
further information.

Regards,

Lee

^ permalink raw reply

* ipv6 fragmentation-related panic in netfilter
From: Tomas Hlavacek @ 2013-10-29 21:07 UTC (permalink / raw)
  To: netdev; +Cc: netfilter-devel

Hi!

I have encountered following condition on 3 distinct hosts in last few 
days. Hosts are failing several times a day (4 to 7 times) and it usually 
happens roughly at the same time. Affected hosts has almost exactly the 
same HW, but different kernel versions from Debian (Wheezy) default 3.2 up 
to 3.11.6.


      KERNEL: /usr/src/vmlinux                
    DUMPFILE: dump.201310291545  [PARTIAL DUMP]
        CPUS: 16
        DATE: Tue Oct 29 15:45:11 2013
      UPTIME: 06:04:17
LOAD AVERAGE: 0.04, 0.25, 0.32
       TASKS: 211
    NODENAME: fw03a
     RELEASE: 3.11.6
     VERSION: #2 SMP Mon Oct 28 20:29:03 CET 2013
     MACHINE: x86_64  (2393 Mhz)
      MEMORY: 12 GB
       PANIC: 
         PID: 0
     COMMAND: "swapper/1"
        TASK: ffff8801b90ac7b0  (1 of 16)  [THREAD_INFO: ffff8801b90b4000]
         CPU: 1
       STATE: TASK_RUNNING (PANIC)

crash> bt
PID: 0      TASK: ffff8801b90ac7b0  CPU: 1   COMMAND: "swapper/1"
 #0 [ffff8801bfc235d0] machine_kexec at ffffffff81032f68
 #1 [ffff8801bfc23610] crash_kexec at ffffffff8109e055
 #2 [ffff8801bfc236e0] oops_end at ffffffff81005e90
 #3 [ffff8801bfc23700] do_invalid_op at ffffffff81003004
 #4 [ffff8801bfc237a0] invalid_op at ffffffff8142b368
    [exception RIP: pskb_expand_head+596]
    RIP: ffffffff81333c74  RSP: ffff8801bfc23850  RFLAGS: 00010202
    RAX: 0000000000000003  RBX: ffff8801b6d99080  RCX: 0000000000000020
    RDX: 00000000000005f4  RSI: 0000000000000000  RDI: ffff8801b6d99080
    RBP: 0000000040115833   R8: 00000000000002c0   R9: ffff8801b8cf2c00
    R10: 000000000000ffff  R11: 00000000197033fe  R12: 0000000000000000
    R13: ffff880337b59a00  R14: ffffffffa03fb160  R15: ffff880337b59a00
    ORIG_RAX: ffffffffffffffff  CS: 0010  SS: 0018
 #5 [ffff8801bfc23858] __nf_conntrack_confirm at ffffffffa03ace16 
[nf_conntrack]
 #6 [ffff8801bfc238c8] vlan_netlink_fini at ffffffffa03fb160 [8021q]
 #7 [ffff8801bfc23928] dev_queue_xmit at ffffffff81342d79
 #8 [ffff8801bfc23978] ip6_finish_output2 at ffffffff813d26ee
 #9 [ffff8801bfc239c8] ip6_forward at ffffffff813d44be
#10 [ffff8801bfc23a48] __ipv6_conntrack_in at ffffffffa034f7b6 
[nf_conntrack_ipv6]
#11 [ffff8801bfc23a98] nf_iterate at ffffffff8136ba0d
#12 [ffff8801bfc23af8] nf_hook_slow at ffffffff8136baae
#13 [ffff8801bfc23b68] nf_ct_frag6_output at ffffffffa039decf 
[nf_defrag_ipv6]
#14 [ffff8801bfc23bd8] ipv6_defrag at ffffffffa039d0c1 [nf_defrag_ipv6]
#15 [ffff8801bfc23c18] nf_iterate at ffffffff8136ba0d
#16 [ffff8801bfc23c78] nf_hook_slow at ffffffff8136baae
#17 [ffff8801bfc23ce8] ipv6_rcv at ffffffff813d59f5
#18 [ffff8801bfc23d38] __netif_receive_skb_core at ffffffff813410db
#19 [ffff8801bfc23db8] napi_gro_receive at ffffffff81341d88
#20 [ffff8801bfc23dd8] igb_poll at ffffffffa0035867 [igb]
#21 [ffff8801bfc23e88] net_rx_action at ffffffff81341ac9
#22 [ffff8801bfc23ed8] __do_softirq at ffffffff81049fb6
#23 [ffff8801bfc23f38] call_softirq at ffffffff8142b4fc
#24 [ffff8801bfc23f50] do_softirq at ffffffff8100481d
#25 [ffff8801bfc23f80] do_IRQ at ffffffff810043bb
--- <IRQ stack> ---
#26 [ffff8801b90b5db8] ret_from_intr at ffffffff81429baa
    [exception RIP: cpuidle_enter_state+86]
    RIP: ffffffff813107a6  RSP: ffff8801b90b5e68  RFLAGS: 00000216
    RAX: 000000000007ff2b  RBX: 0000000140523c4c  RCX: 0000000000000018
    RDX: 0000000225c17d03  RSI: 0000000000000000  RDI: ffffffff81812600
    RBP: 0000000000000004   R8: 0000000000000018   R9: 00000000000006cf
    R10: 0000000000000001  R11: 0000000000000006  R12: 0000000100523c4e
    R13: 0000000000000000  R14: ffffffff81066415  R15: 0000000000000086
    ORIG_RAX: ffffffffffffff94  CS: 0010  SS: 0018
#27 [ffff8801b90b5eb0] cpuidle_idle_call at ffffffff813108ce
#28 [ffff8801b90b5ee0] arch_cpu_idle at ffffffff8100b769
#29 [ffff8801b90b5ef0] cpu_startup_entry at ffffffff81086b1d
#30 [ffff8801b90b5f30] start_secondary at ffffffff8102af40

I am investigating at the moment. All suggestions/help would be 
appreciated.

Tomas

^ permalink raw reply

* Re: 3.12-rc7 regression - network panic from ipv6
From: Hannes Frederic Sowa @ 2013-10-29 21:07 UTC (permalink / raw)
  To: Meelis Roos; +Cc: Linux Kernel list, netdev, steffen.klassert
In-Reply-To: <alpine.SOC.1.00.1310292221550.19416@math.ut.ee>

On Tue, Oct 29, 2013 at 10:33:36PM +0200, Meelis Roos wrote:
> Some bad news - in a system where 3.12-rc6 and earlier worked fine, 
> 3.12-rc7 panics or hangs repeatedly with network traffic (torrent being 
> good test). First there is BUG from ipv6 code, followed by panic.

Could you do a bisect on this? There seems to be one commit for this
particular function _decode_session6:

commit bafd4bd4dcfa13145db7f951251eef3e10f8c278
Author: Steffen Klassert <steffen.klassert@secunet.com>
Date:   Mon Sep 9 10:38:38 2013 +0200

    xfrm: Decode sessions with output interface.
    
    The output interface matching does not work on forward
    policy lookups, the output interface of the flowi is
    always 0. Fix this by setting the output interface when
    we decode the session.

Maybe try to just revert this change locally and try again?

Thanks,

  Hannes

^ permalink raw reply

* Re: [PATCH] netfilter fix for net
From: David Miller @ 2013-10-29 20:54 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel, netdev
In-Reply-To: <1383048796-7964-1-git-send-email-pablo@netfilter.org>

From: Pablo Neira Ayuso <pablo@netfilter.org>
Date: Tue, 29 Oct 2013 13:13:15 +0100

> This pull request contains the following netfilter fix:
> 
> * fix --queue-bypass in xt_NFQUEUE revision 3. While adding the
>   revision 3 of this target, the bypass flags were not correctly
>   handled anymore, thus, breaking packet bypassing if no application
>   is listening from userspace, patch from Holger Eitzenberger,
>   reported by Florian Westphal.
> 
> You can pull this change from:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf.git master

Pulled, thanks Pablo.

^ permalink raw reply


This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox