Re: [GIT] Networking

2015-09-09 Thread Corinna Vinschen
ault) > > > > On Thu, Sep 3, 2015 at 1:35 AM, David Miller wrote: > >> > >> > >> Another merge window, another set of networking changes. I've heard > >> rumblings that the lightweight tunnels infrastructure has been voted > >> networking

[PATCH net] r8169: Fix sleeping function called during get_stats64

2015-09-09 Thread Corinna Vinschen
changing the link state as well as removing/reloading the r8169 module at the same time. Signed-off-by: Corinna Vinschen --- drivers/net/ethernet/realtek/r8169.c | 140 ++- 1 file changed, 56 insertions(+), 84 deletions(-) diff --git a/drivers/net/ethernet

Re: [PATCH net] r8169: Fix sleeping function called during get_stats64

2015-09-09 Thread Corinna Vinschen
On Sep 9 22:23, Francois Romieu wrote: > Corinna Vinschen : > [...] > > diff --git a/drivers/net/ethernet/realtek/r8169.c > > b/drivers/net/ethernet/realtek/r8169.c > > index 24dcbe6..630811a 100644 > > --- a/drivers/net/ethernet/realtek/r8169.c > > +++ b/

[PATCH v2] r8169: Fix sleeping function called during get_stats64

2015-09-09 Thread Corinna Vinschen
changing the link state as well as removing/reloading the r8169 module. Signed-off-by: Corinna Vinschen --- drivers/net/ethernet/realtek/r8169.c | 139 ++- 1 file changed, 55 insertions(+), 84 deletions(-) diff --git a/drivers/net/ethernet/realtek/r8169.c b

Re: [PATCH v2] r8169: Fix sleeping function called during get_stats64

2015-09-10 Thread Corinna Vinschen
On Sep 9 20:31, David Miller wrote: > From: Corinna Vinschen > Date: Wed, 9 Sep 2015 23:16:40 +0200 > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=104031 > > Fixes: 6e85d5ad36a26debc23a9a865c029cbe242b2dc8 > > > > Based on the discussion start

[PATCH v3] r8169: Fix sleeping function called during get_stats64, v2

2015-09-10 Thread Corinna Vinschen
changing the link state as well as removing/reloading the r8169 module. Signed-off-by: Corinna Vinschen --- v2 -> v3: Rename CntPhysAddr -> counters_phys_addr Rename CntArray-> counters Fix whitespace issue Fix label names in rtl_init_one. drivers/net

Re: [Intel-wired-lan] [PATCH] igb: use igb_adapter->io_addr instead of e1000_hw->hw_addr

2016-11-08 Thread Corinna Vinschen
om a register failed. That makes me wonder if setting the hardware address to NULL in rd32/igb_rd32 is really such a good idea. It's performed in a function which return value is *never* tested for validity in the calling functions and leads to subsequent crashes since no tests for hw_addr == NULL

Re: [Intel-wired-lan] [PATCH] igb: use igb_adapter->io_addr instead of e1000_hw->hw_addr

2016-11-08 Thread Corinna Vinschen
On Nov 8 09:16, Hisashi T Fujinaka wrote: > On Tue, 8 Nov 2016, Corinna Vinschen wrote: > > On Nov 8 15:06, Cao jin wrote: > > > When running as guest, under certain condition, it will oops as following. > > > writel() in igb_configure_tx_ring() results in oops, becaus

Re: [Intel-wired-lan] [PATCH] igb: use igb_adapter->io_addr instead of e1000_hw->hw_addr

2016-11-10 Thread Corinna Vinschen
On Nov 8 11:33, Alexander Duyck wrote: > On Tue, Nov 8, 2016 at 10:37 AM, Corinna Vinschen wrote: > > On Nov 8 09:16, Hisashi T Fujinaka wrote: > >> On Tue, 8 Nov 2016, Corinna Vinschen wrote: > >> > On Nov 8 15:06, Cao jin wrote: > >> > > When

Re: [Intel-wired-lan] [PATCH] igb: use igb_adapter->io_addr instead of e1000_hw->hw_addr

2016-11-10 Thread Corinna Vinschen
On Nov 10 05:48, Hisashi T Fujinaka wrote: > On Thu, 10 Nov 2016, Corinna Vinschen wrote: > > On Nov 8 11:33, Alexander Duyck wrote: > ... > > > The question I would have is what is reading the device when it is in > > > this state. The watchdog and any other