Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-11 Thread Jonathan Maxwell
> I personally think that drivers need to synchronize such things > internally. They are the only entity which knows when it's "OK" > to do whatever the netpoll method does, and they are also the only > entity which can properly synchronize such checks. Thanks agreed. I am testing the following

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-11 Thread David Miller
From: Jonathan Maxwell Date: Tue, 11 Aug 2015 15:53:18 +1000 >> What if the carrier check passes, and then the chip reset starts on >> another cpu? You'll have the same problem. > > Okay, let me see if I can come up with a better way to mitigate this. I personally think that drivers need to

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-11 Thread David Miller
From: Jonathan Maxwell jmaxwel...@gmail.com Date: Tue, 11 Aug 2015 15:53:18 +1000 What if the carrier check passes, and then the chip reset starts on another cpu? You'll have the same problem. Okay, let me see if I can come up with a better way to mitigate this. I personally think that

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-11 Thread Jonathan Maxwell
I personally think that drivers need to synchronize such things internally. They are the only entity which knows when it's OK to do whatever the netpoll method does, and they are also the only entity which can properly synchronize such checks. Thanks agreed. I am testing the following ixgbe

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread Jonathan Maxwell
> What if the carrier check passes, and then the chip reset starts on > another cpu? You'll have the same problem. Okay, let me see if I can come up with a better way to mitigate this. On Tue, Aug 11, 2015 at 2:22 PM, David Miller wrote: > From: Jon Maxwell > Date: Tue, 11 Aug 2015 11:32:26

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread David Miller
From: Jon Maxwell Date: Tue, 11 Aug 2015 11:32:26 +1000 > We have seen a few crashes recently where a NIC is getting > reset for some reason and then the driver or another module calls > printk() which invokes netconsole. Netconsole then calls the > adapter specific poll routine via netpoll

[PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread Jon Maxwell
We have seen a few crashes recently where a NIC is getting reset for some reason and then the driver or another module calls printk() which invokes netconsole. Netconsole then calls the adapter specific poll routine via netpoll which crashes because the adapter is resetting and its structures are

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread Jonathan Maxwell
What if the carrier check passes, and then the chip reset starts on another cpu? You'll have the same problem. Okay, let me see if I can come up with a better way to mitigate this. On Tue, Aug 11, 2015 at 2:22 PM, David Miller da...@davemloft.net wrote: From: Jon Maxwell jmaxwel...@gmail.com

Re: [PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread David Miller
From: Jon Maxwell jmaxwel...@gmail.com Date: Tue, 11 Aug 2015 11:32:26 +1000 We have seen a few crashes recently where a NIC is getting reset for some reason and then the driver or another module calls printk() which invokes netconsole. Netconsole then calls the adapter specific poll routine

[PATCH net] netconsole: Check for carrier before calling netpoll_send_udp()

2015-08-10 Thread Jon Maxwell
We have seen a few crashes recently where a NIC is getting reset for some reason and then the driver or another module calls printk() which invokes netconsole. Netconsole then calls the adapter specific poll routine via netpoll which crashes because the adapter is resetting and its structures are