<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-dev/drivers/net/ethernet/intel/i40evf, branch master</title>
<subtitle>Linux kernel development work - see feature branches</subtitle>
<id>https://git.zx2c4.com/linux-dev/atom/drivers/net/ethernet/intel/i40evf?h=master</id>
<link rel='self' href='https://git.zx2c4.com/linux-dev/atom/drivers/net/ethernet/intel/i40evf?h=master'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/'/>
<updated>2018-09-18T15:43:03Z</updated>
<entry>
<title>intel-ethernet: rename i40evf to iavf</title>
<updated>2018-09-18T15:43:03Z</updated>
<author>
<name>Jesse Brandeburg</name>
<email>jesse.brandeburg@intel.com</email>
</author>
<published>2018-09-15T00:37:44Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=8062b2263a9fc294ddeb4024b113e8e26b82d5de'/>
<id>urn:sha1:8062b2263a9fc294ddeb4024b113e8e26b82d5de</id>
<content type='text'>
Rename the Intel Ethernet Adaptive Virtual Function driver
(i40evf) to a new name (iavf) that is more consistent with
the ongoing maintenance of the driver as the universal VF driver
for multiple product lines.

This first patch fixes up the directory names and the .ko name,
intentionally ignoring the function names inside the driver
for now.  Basically this is the simplest patch that gets
the rename done and will be followed by other patches that
rename the internal functions.

This patch also addresses a couple of string/name issues
and updates the Copyright year.

Also, made sure to add a MODULE_ALIAS to the old name.

Signed-off-by: Jesse Brandeburg &lt;jesse.brandeburg@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40e(vf): remove i40e_ethtool_stats.h header file</title>
<updated>2018-09-08T17:06:17Z</updated>
<author>
<name>Jacob Keller</name>
<email>jacob.e.keller@intel.com</email>
</author>
<published>2018-09-07T21:55:44Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=6ad96bdca8aae5002819280df2a850c05cb0a592'/>
<id>urn:sha1:6ad96bdca8aae5002819280df2a850c05cb0a592</id>
<content type='text'>
Essentially reverts commit 8fd75c58a09a ("i40e: move ethtool
stats boiler plate code to i40e_ethtool_stats.h", 2018-08-30), and
additionally moves the similar code in i40evf into i40evf_ethtool.c.

The code was intially moved from i40e_ethtool.c into i40e_ethtool_stats.h
as a way of better logically organizing the code. This has two problems.
First, we can't have an inline function with variadic arguments on all
platforms. Second, it gave the appearance that we had plans to share
code between the i40e and i40evf drivers, due to having a near copy of
the contents in the i40evf/i40e_ethtool_stats.h file.

Patches which actually attempt to combine or share code between the i40e
and i40evf drivers have not materialized, and are likely a ways off.

Rather than fixing the one function which causes build issues, just move
this code back into the i40e_ethtool.c and i40evf_ethtool.c files. Note
that we also change these functions back from static inlines to just
statics, since they're no longer in a header file.

We can revisit this if/when work is done to actually attempt to share
code between drivers. Alternatively, this stats code could be made more
generic so that it can be shared across drivers as part of ethtool
kernel work.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>i40evf: cancel workqueue sync for adminq when a VF is removed</title>
<updated>2018-08-30T20:53:04Z</updated>
<author>
<name>Lihong Yang</name>
<email>lihong.yang@intel.com</email>
</author>
<published>2018-08-28T17:16:08Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=babbcc60040abfb7a9e3caa1c58fe182ae73762a'/>
<id>urn:sha1:babbcc60040abfb7a9e3caa1c58fe182ae73762a</id>
<content type='text'>
If a VF is being removed, there is no need to continue with the
workqueue sync for the adminq task, thus cancel it. Without this call,
when VFs are created and removed right away, there might be a chance for
the driver to crash with events stuck in the adminq.

Signed-off-by: Lihong Yang &lt;lihong.yang@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40evf: Don't enable vlan stripping when rx offload is turned on</title>
<updated>2018-08-30T20:53:04Z</updated>
<author>
<name>Patryk Małek</name>
<email>patryk.malek@intel.com</email>
</author>
<published>2018-08-28T17:16:02Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=3bd77e2ae1477d6f87fc3f542c737119d5decf9f'/>
<id>urn:sha1:3bd77e2ae1477d6f87fc3f542c737119d5decf9f</id>
<content type='text'>
With current implementation of i40evf_set_features when user sets
any offload via ethtool we set I40EVF_FLAG_AQ_ENABLE_VLAN_STRIPPING
as a required aq which triggers driver to call
i40evf_enable_vlan_stripping. This shouldn't take place.
This patches fixes it by setting the flag only when VLAN offload
is turned on.

Signed-off-by: Patryk Małek &lt;patryk.malek@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40evf: set IFF_UNICAST_FLT flag for the VF</title>
<updated>2018-08-30T20:53:04Z</updated>
<author>
<name>Lihong Yang</name>
<email>lihong.yang@intel.com</email>
</author>
<published>2018-08-20T15:12:31Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=e65aae086330d0a6c6c9f874aef03c69cf98884b'/>
<id>urn:sha1:e65aae086330d0a6c6c9f874aef03c69cf98884b</id>
<content type='text'>
Set IFF_UNICAST_FLT flag for the VF to prevent it from entering
promiscuous mode when macvlan is added to the VF.

Signed-off-by: Lihong Yang &lt;lihong.yang@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40evf: Validate the number of queues a PF sends</title>
<updated>2018-08-30T20:53:04Z</updated>
<author>
<name>Paul M Stillwell Jr</name>
<email>paul.m.stillwell.jr@intel.com</email>
</author>
<published>2018-08-20T15:12:29Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=3c818910911c93bb5099c6637ec350f90c0e71fc'/>
<id>urn:sha1:3c818910911c93bb5099c6637ec350f90c0e71fc</id>
<content type='text'>
A PF can send any number of queues to the VF and the VF may not
be able to support that many. Check to see that the number of
queues is less than or equal to the max number of queues the
VF can have.

Signed-off-by: Paul M Stillwell Jr &lt;paul.m.stillwell.jr@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40evf: Change a VF mac without reloading the VF driver</title>
<updated>2018-08-30T20:53:03Z</updated>
<author>
<name>Paweł Jabłoński</name>
<email>pawel.jablonski@intel.com</email>
</author>
<published>2018-08-20T15:12:26Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=ae1e29f671b467f3e9e9aa2b82ee40e4300ea810'/>
<id>urn:sha1:ae1e29f671b467f3e9e9aa2b82ee40e4300ea810</id>
<content type='text'>
Add possibility to change a VF mac address from host side
without reloading the VF driver on the guest side. Without
this patch it is not possible to change the VF mac because
executing i40evf_virtchnl_completion function with
VIRTCHNL_OP_GET_VF_RESOURCES opcode resets the VF mac
address to previous value.

Signed-off-by: Paweł Jabłoński &lt;pawel.jablonski@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40evf: update ethtool stats code and use helper functions</title>
<updated>2018-08-30T20:53:03Z</updated>
<author>
<name>Jacob Keller</name>
<email>jacob.e.keller@intel.com</email>
</author>
<published>2018-08-20T15:12:25Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=6dba41cd02fc0c408ae9a99809e1960a691dfad0'/>
<id>urn:sha1:6dba41cd02fc0c408ae9a99809e1960a691dfad0</id>
<content type='text'>
Fix a bug in the way we handled VF queues, by always showing stats for
the maximum number of queues, even if they aren't allocated. It is not
safe to change the number of strings reported to ethtool, as grabbing
statistics occurs over multiple ethtool ops for which the rtnl_lock()
cannot be held the entire time.

Avoid this by always reporting queue stats for the maximum number of
queues in the netdevice. Share some of the helper functionality for
adding stats with the PF code in i40e_ethtool_stats.h

This should reduce the chance of potential future bugs, and make adding
new statistics easier.

Note for the queue stats, unlike the PF driver we do not keep an array
of queue pointers, but an array of queues, so care must be taken to
avoid accessing queue memory that hasn't yet been allocated.

Signed-off-by: Jacob Keller &lt;jacob.e.keller@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40e: Add AQ command for rearrange NVM structure</title>
<updated>2018-08-07T19:20:45Z</updated>
<author>
<name>Piotr Azarewicz</name>
<email>piotr.azarewicz@intel.com</email>
</author>
<published>2018-07-31T10:41:47Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=f05798b4ff82d919e94e5060d1e9993a3e025361'/>
<id>urn:sha1:f05798b4ff82d919e94e5060d1e9993a3e025361</id>
<content type='text'>
During switching between old NVM structure approach (called structured
NVM) to new one (called flat NVM) or backward flash needs to be
rearranged to required NVM structure. This is a part of transition from
one NVM structure to another. The function is introduced to command
firmware to start rearrangement process.

Signed-off-by: Piotr Azarewicz &lt;piotr.azarewicz@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40e: Add additional return code to i40e_asq_send_command</title>
<updated>2018-08-07T19:20:38Z</updated>
<author>
<name>Piotr Azarewicz</name>
<email>piotr.azarewicz@intel.com</email>
</author>
<published>2018-07-31T10:41:46Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=b2b57b29588c36920fe1364f3f24bebfbe8bf321'/>
<id>urn:sha1:b2b57b29588c36920fe1364f3f24bebfbe8bf321</id>
<content type='text'>
Firmware can return a busy state, so the function return
I40E_ERR_NOT_READY.

Signed-off-by: Piotr Azarewicz &lt;piotr.azarewicz@intel.com&gt;
Tested-by: Andrew Bowers &lt;andrewx.bowers@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
</feed>
