Dne 04. 08. 20 v 18:09 izabela.bakoll...@gmail.com napsala:
From: Izabela Bakollari
Dropwatch is a utility that monitors dropped frames by having userspace
record them over the dropwatch protocol over a file. This augument
allows live monitoring of dropped frames using tools like tcpdump.
With
Dne 11. 06. 20 v 6:07 Shannon Nelson napsal(a):
We no longer support the mgmt device in the ionic driver,
so remove the device id and related code.
...> @@ -252,8 +248,6 @@ static int ionic_probe(struct pci_dev *pdev,
const struct pci_device_id *ent)
}
pci_set_master(pdev);
-
When freeing VF's DMA mappings, an already NULLed pointer was checked
again due to an apparent copy&paste error. Consequently, the pf2vf
bulletin DMA mapping was not freed.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | 2 +-
1 file changed, 1
g to start on a system with sfc driver
after upgrading it to Linux 4.11, i.e. when sfc added support for
phys_port_name.
Signed-off-by: Michal Schmidt
---
net/core/rtnetlink.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/core/rtnetlink.c b/net/core/rtnetlink.c
index
're looking at the error paths in the function, also return
an error code from it when the PF responds with an error. Even though
the caller ignores it.
v2: Move the check before bnx2x_vfpf_prep() as suggested by Yuval.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2
interface is up.
Use -ENETDOWN as the error code when the interface is down.
-EFAULT in bnx2x_ptp_adjfreq() did not seem right.
Tested using phc_ctl get/set/adj/freq commands.
Signed-off-by: Michal Schmidt
While I have no objections to the patch contents, does it even make
sense to try
Dne 5.3.2017 v 11:13 Mintz, Yuval napsal(a):
On failure to configure a VF MAC/VLAN filter we should not attempt to
rollback filters that we failed to configure with -EEXIST.
Is this theoretical or did you actually manage to hit it?
If so, did it involve non-linux VFs?
Asking as linux VFs don't
VFs are currently missing the VLAN filtering feature, because we were
checking the PF's acquire response before actually performing the acquire.
Fix it by setting the feature flag later when we have the PF response.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/
When BNX2X_MSG_IOV is enabled, the driver produces too many VF statistics
messages. Lower the verbosity of the VF stats messages similarly as in
commit 76ca70fabbdaa3 ("bnx2x: [Debug] change verbosity of some prints").
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broa
Configuring VLANs from the VF side had no effect, because the PF ignored
filters of type VFPF_VLAN_FILTER in the VF-PF message.
Add the missing filter type to configure.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_vfpf.c | 17 +
1 file changed
code when the interface is down.
-EFAULT in bnx2x_ptp_adjfreq() did not seem right.
Tested using phc_ctl get/set/adj/freq commands.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff
On failure to configure a VF MAC/VLAN filter we should not attempt to
rollback filters that we failed to configure with -EEXIST.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | 8 +++-
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.h | 1 +
2 files
It is too late to check for the limit of the number of VF multicast
addresses after they have already been copied to the req->multicast[]
array, possibly overflowing it.
Do the check before copying.
Also fix the error path to not skip unlocking vf2pf_mutex.
Signed-off-by: Michal Schm
Hello,
here are fixes for a crash with PTP, a crash in setting of VF multicast
addresses, and non-working VLAN filters configuration from the VF side.
Michal Schmidt (7):
bnx2x: prevent crash when accessing PTP with interface down
bnx2x: lower verbosity of VF stats debug messages
bnx2x: fix
filters->count is the number of filters we were supposed to configure.
There is no reason to increase it by +1 when printing the count in an error
message.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
Use the same code for reconfiguring VLANs during NIC load.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 151 ++-
1 file changed, 62 insertions(+), 89 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c
b/drivers
iteration the check for the terminator
is an out of bounds read.
Add the missing terminator to rspq_map[].
Reported-by: Jan Stancek
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/chelsio/cxgb3/cxgb3_main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net
Fix a case of misleading code indentation.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c
b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c
There were no missing endianness conversions in this case, but the
fields of struct cfc_del_event_data should be defined as little-endian
to get rid of the ugly (__force __le32) casts.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h | 6 +++---
drivers/net
writeb(), not
writel(), so the attempt to define the struct with a reversed layout on
big-endian is counter-productive.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/bnx2x
For consistency with other event data structs and to lessen
the chance of a mistake should one of the reserved fields become
used in the future, define the reserved fields as little-endian.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h | 12 ++--
1
On ppc64 the PF did not receive messages from VFs correctly.
Fields of struct vf_pf_event_data are little-endian.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h | 6 +++---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_vfpf.c | 6 --
2 files changed, 7
It's not really a bug, but it was odd that bnx2x_eq_int() read the
message data as if it were a cfc_del_event regardless of the event type.
It's cleaner to access only the appropriate member of union event_data
after checking the event opcode.
Signed-off-by: Michal Schmidt
---
d
Hello,
this fixes a VLAN crash and some SRIOV bugs in bnx2x observed on ppc64.
Michal Schmidt (7):
bnx2x: fix crash on big-endian when adding VLAN
bnx2x: fix sending VF->PF messages on big-endian
bnx2x: fix receive of VF->PF mailbox messages by the PF on big-endian
bnx2x:
added where missing.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_hsi.h | 6 +++---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 13 +
drivers/net/ethernet/broadcom/bnx2x/bnx2x_sriov.c | 9 -
3 files changed, 12 insertions(+), 16
alloc_pages() already prints a warning when it fails. No need to emit
another message. Certainly not at KERN_ERR level, because it is no big
deal if this GFP_ATOMIC allocation fails occasionally.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 4 +---
1 file
It's supposed to be impossible for TPA to give us anything else
than IPv4 or IPv6 here. But in case there is a way to reach this error
by some strange received frames, we don't want to flood the kernel log.
WARN_ONCE is better for this purpose.
Signed-off-by: Michal Schmidt
---
d
Hello,
This removes one redundant error message in bnx2x and changes another one to
WARN_ONCE. The third patch is a small simplification in ethtool stats.
Michal Schmidt (3):
bnx2x: drop redundant error message about allocation failure
bnx2x: change FW GRO error message to WARN_ONCE
bnx2x
(). It was used only once and it's in fact just
a negation of IS_PORT_STAT().
Signed-off-by: Michal Schmidt
---
.../net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c| 117 ++---
1 file changed, 56 insertions(+), 61 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/bnx2
On 06/26/2015 05:50 PM, Michal Schmidt wrote:
> With CONFIG_DMA_API_DEBUG=y bnx2x triggers the error "DMA-API: device
> driver frees DMA memory with wrong function".
> On archs where PAGE_SIZE > SGE_PAGE_SIZE it also triggers "DMA-API:
> device driver frees
quot;bnx2x: Alloc 4k fragment for each rx ring buffer element")
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | 1 -
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 23 ++-
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h | 12 ++
quot;bnx2x: Alloc 4k fragment for each rx ring buffer element")
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | 1 -
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 23 ++-
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h | 12 ++
On 05/21/2015 03:20 PM, Gabriel Krisman Bertazi wrote:
> +#define SGE_PAGE_SHIFT 12
> +#define SGE_PAGE_SIZE(1 << SGE_PAGE_SHIFT)
...
> +struct bnx2x_alloc_pool {
> + struct page *page;
> + dma_addr_t dma;
> + u8 offset;
> + u8
I noticed some simplification possibilities while looking into the bug
fixed by "bnx2x: really disable TPA if 'disable_tpa' is set'.
Michal Schmidt (3):
bnx2x: mark LRO as a fixed disabled feature if disable_tpa is set
bnx2x: merge fp->disable_tpa with fp->mod
These flags are redundant with dev->features. Remove them.
Just make sure to set dev->features ourselves in bnx2x_set_features()
before performing the reload of the card.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | 2 --
drivers/net/ethernet/br
It is simpler to have the TPA mode as one three-state variable.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x.h | 2 +-
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 26
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.h | 2
n in bnx2x_fix_features() becomes unnecessary.
Signed-off-by: Michal Schmidt
---
drivers/net/ethernet/broadcom/bnx2x/bnx2x_cmn.c | 4
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 2 +-
2 files changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/broadcom/bnx2x
On 04/27/2015 08:15 PM, Yuval Mintz wrote:
> I agree that this fixes an incorrect logic, but its incomplete as the bnx2x
> 'bp->flags' no longer represent the correct logic. I.e., it might cause
> additional issues down the road, as the 'fp->mode' and 'fp->disable_tpa'
> are no longer in sync.
> W
ll observe tpa_* statistics increase and large packets being received
in tcpdump.
The bug was an unintended consequence of commit aebf6244cd39 "bnx2x: Be
more forgiving toward SW GRO".
Fix it by following the bp->disable_tpa flag when initializing fp's.
Signed-off-by: Michal S
e, I'll do the same for GRE and SIT tunnels.
Signed-off-by: Michal Schmidt <[EMAIL PROTECTED]>
diff --git a/net/ipv4/ipip.c b/net/ipv4/ipip.c
index 8c2b2b0..160535b 100644
--- a/net/ipv4/ipip.c
+++ b/net/ipv4/ipip.c
@@ -651,6 +651,40 @@ tx_error:
return 0;
}
+static vo
e new bound device's MTU.
If the change is acceptable, I'll do the same for GRE and SIT tunnels.
Signed-off-by: Michal Schmidt <[EMAIL PROTECTED]>
diff --git a/net/ipv4/ipip.c b/net/ipv4/ipip.c
index 8c2b2b0..05b267b 100644
--- a/net/ipv4/ipip.c
+++ b/net/ipv4/ipip.c
@@ -651,
l already keeps track of them. We can take advantage of it and
use the .remove callback as it was meant to be.
This makes /sys/bus/pci/drivers/airo/{,un}bind work.
Signed-off-by: Michal Schmidt <[EMAIL PROTECTED]>
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c
index 0d
The airo driver leaks memory if request_irq() fails.
Signed-off-by: Michal Schmidt <[EMAIL PROTECTED]>
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c
index 71ac1f1..0d5b433 100644
--- a/drivers/net/wireless/airo.c
+++ b/drivers/net/wireless/airo.c
@@ -2850,7 +
It's unnecessary to check for NULL before calling kfree().
Signed-off-by: Michal Schmidt <[EMAIL PROTECTED]>
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c
index a8c2bfe..71ac1f1 100644
--- a/drivers/net/wireless/airo.c
+++ b/drivers/net/wireless/airo.c
@@ -274
Kok, Auke wrote:
> uhh, sure... while we are at it then, are there any more patches like this?
> perhaps for e100? ixgb?
A quick grep through the current RT patch shows there are no other
changes to e100, e1000, ixgb.
Michal
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Kok, Auke wrote:
> Mark Huth wrote:
>> Well, I'm not an expert on the realtime patches - but most any patch set
>> from Ingo seems to set this off - we've run through a bunch all the way
>> since 2.6.10.
>
> that's bizarre, and would imply that Ingo
ed (only firmware versions 5.30.17 and greater
support WPA. Detected 5.30.17)
airo_test_wpa_capable assumes that the softSubVer part of the firmware
version number is coded in BCD. Apparently, that's not true.
I have firmware version 5.30.17 and cap_rid.softSubVer is 0x11==17.
Signed-
Rick Jones wrote:
lumber:~# cat /etc/udev/rules.d/010_netinterfaces.rules
KERNEL="eth*",SYSFS{address}=="00:30:6e:4c:27:3c", NAME="eth0"
KERNEL="eth*",SYSFS{address}=="00:30:6e:4c:27:3d", NAME="eth1"
KERNEL="eth*",SYSFS{address}=="00:12:79:9e:0e:d2", NAME="eth2"
KERNEL="eth*",SYSFS{address}=="00:
josh wrote:
I have two ethernet cards on my box, an onboard SiS 7012 and a pci ne-2k
compatible. When I boot the system, sometimes the SiS is detected as
eth0 and the pci as eth1, and sometimes the SiS is eth1 and the pci
eth0, *without my changing anything*. I just reboot and it's different.
Y
49 matches
Mail list logo
