<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/include/linux/avf, branch v4.19.159</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.19.159</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v4.19.159'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2019-12-13T07:51:18Z</updated>
<entry>
<title>virtchnl: Fix off by one error</title>
<updated>2019-12-13T07:51:18Z</updated>
<author>
<name>Alice Michael</name>
<email>alice.michael@intel.com</email>
</author>
<published>2018-10-26T21:33:31Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=d5be9df0679440bdca7412e2c303aca1f561696a'/>
<id>urn:sha1:d5be9df0679440bdca7412e2c303aca1f561696a</id>
<content type='text'>
[ Upstream commit 843faff87af261bf55eda719a06087af0486a168 ]

When calculating the valid length for a VIRTCHNL_OP_ENABLE_CHANNELS
message, we accidentally allowed messages with one extra
virtchnl_channel_info structure on the end. This happened due
to an off by one error, because we forgot that valid_len already
accounted for one virtchnl_channel_info structure, so we need to
subtract one from the num_tc value.

Signed-off-by: Alice Michael &lt;alice.michael@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;
Signed-off-by: Sasha Levin &lt;sashal@kernel.org&gt;
</content>
</entry>
<entry>
<title>virtchnl: Whitespace and parenthesis cleanup</title>
<updated>2018-05-14T14:05:16Z</updated>
<author>
<name>Bruce Allan</name>
<email>bruce.w.allan@intel.com</email>
</author>
<published>2018-05-10T12:59:39Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=0fccb85ad21d65c6ee1ea912acc3f0ae4df31876'/>
<id>urn:sha1:0fccb85ad21d65c6ee1ea912acc3f0ae4df31876</id>
<content type='text'>
Clean up existing instances of unnecessary parentheses in if
statement and change order of conditionals to make it easier to read

The opening /* should be followed by a single space and the closing */
should be preceded with a single space.

Signed-off-by: Bruce Allan &lt;bruce.w.allan@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>virtchnl: Add filter data structures</title>
<updated>2018-02-14T17:43:22Z</updated>
<author>
<name>Harshitha Ramamurthy</name>
<email>harshitha.ramamurthy@intel.com</email>
</author>
<published>2018-01-23T16:51:04Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=3872c8d44c2e489bcce0c743e808a4135e8da228'/>
<id>urn:sha1:3872c8d44c2e489bcce0c743e808a4135e8da228</id>
<content type='text'>
This patch adds infrastructure to send virtchnl messages to the
PF to configure filters on the VF. The patch adds a struct
called virtchnl_filter which contains information about the fields
in the user-specified tc filter.

Signed-off-by: Harshitha Ramamurthy &lt;harshitha.ramamurthy@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>virtchnl: Add a macro to check the size of a union</title>
<updated>2018-02-14T17:43:22Z</updated>
<author>
<name>Harshitha Ramamurthy</name>
<email>harshitha.ramamurthy@intel.com</email>
</author>
<published>2018-01-23T16:51:03Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=0718e560a330599d15fddc37651d693c7a09e49e'/>
<id>urn:sha1:0718e560a330599d15fddc37651d693c7a09e49e</id>
<content type='text'>
This patch adds a macro to check if the size of a union is correct.
It throws a divide by zero error if the union is not of the correct
size.

Signed-off-by: Harshitha Ramamurthy &lt;harshitha.ramamurthy@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>virtchnl: Add virtchl structures to support queue channels</title>
<updated>2018-02-14T17:43:22Z</updated>
<author>
<name>Harshitha Ramamurthy</name>
<email>harshitha.ramamurthy@intel.com</email>
</author>
<published>2018-01-23T16:50:56Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=eb09f1feb8e5999390a6f149307cb88354232680'/>
<id>urn:sha1:eb09f1feb8e5999390a6f149307cb88354232680</id>
<content type='text'>
This patch defines new structs in support of the virtchannel message
that the VF sends to the PF to create a queue channel specified by the
user via tc tool.

Signed-off-by: Harshitha Ramamurthy &lt;harshitha.ramamurthy@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/i40evf: don't trust VF to reset itself</title>
<updated>2017-10-13T21:27:13Z</updated>
<author>
<name>Alan Brady</name>
<email>alan.brady@intel.com</email>
</author>
<published>2017-10-11T21:49:43Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=17a9422de78c3a59b490b400f555635c477f1476'/>
<id>urn:sha1:17a9422de78c3a59b490b400f555635c477f1476</id>
<content type='text'>
When using 'ethtool -L' on a VF to change number of requested queues
from PF, we shouldn't trust the VF to reset itself after making the
request.  Doing it that way opens the door for a potentially malicious
VF to do nasty things to the PF which should never be the case.

This makes it such that after VF makes a successful request, PF will
then reset the VF to institute required changes.  Only if the request
fails will PF send a message back to VF letting it know the request was
unsuccessful.

Testing-hints:
There should be no real functional changes.  This is simply hardening
against a potentially malicious VF.

Signed-off-by: Alan Brady &lt;alan.brady@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: Enable VF to negotiate number of allocated queues</title>
<updated>2017-09-29T19:51:01Z</updated>
<author>
<name>Alan Brady</name>
<email>alan.brady@intel.com</email>
</author>
<published>2017-07-14T13:27:08Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=a3f5aa907340b5d7b54223ddbaa90410f168864d'/>
<id>urn:sha1:a3f5aa907340b5d7b54223ddbaa90410f168864d</id>
<content type='text'>
Currently the PF allocates a default number of queues for each VF and
cannot be changed.  This patch enables the VF to request a different
number of queues allocated to it.  This patch also adds a new virtchnl
op and capability flag to facilitate this negotiation.

After the PF receives a request message, it will set a requested number
of queues for that VF.  Then when the VF resets, its VSI will get a new
number of queues allocated to it.

This is a best effort request and since we only allocate a guaranteed
default number, if the VF tries to ask for more than the guaranteed
number, there may not be enough in HW to accommodate it unless other
queues for other VFs are freed. It should also be noted decreasing the
number queues allocated to a VF to below the default will NOT enable the
allocation of more than 32 VFs per PF and will not free queues guaranteed
to each VF by default.

Signed-off-by: Alan Brady &lt;alan.brady@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/i40evf: support for VF VLAN tag stripping control</title>
<updated>2017-08-27T22:47:43Z</updated>
<author>
<name>Mariusz Stachura</name>
<email>mariusz.stachura@intel.com</email>
</author>
<published>2017-07-18T05:09:45Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=8774370d268f2f43d8487d230e0d4fa1647759b3'/>
<id>urn:sha1:8774370d268f2f43d8487d230e0d4fa1647759b3</id>
<content type='text'>
This patch gives VF capability to control VLAN tag stripping via
ethtool. As rx-vlan-offload was fixed before, now the VF is able to
change it using "ethtool --offload &lt;IF&gt; rxvlan on/off" settings.

Signed-off-by: Mariusz Stachura &lt;mariusz.stachura@intel.com&gt;
Signed-off-by: Jeff Kirsher &lt;jeffrey.t.kirsher@intel.com&gt;
</content>
</entry>
<entry>
<title>i40e/i40evf: rename vf_offload_flags to vf_cap_flags in struct virtchnl_vf_resource</title>
<updated>2017-08-25T21:52:29Z</updated>
<author>
<name>Stefan Assmann</name>
<email>sassmann@kpanic.de</email>
</author>
<published>2017-06-29T13:12:24Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=fbb113f773482496d601e0bd934e680b35876016'/>
<id>urn:sha1:fbb113f773482496d601e0bd934e680b35876016</id>
<content type='text'>
The current name of vf_offload_flags indicates that the bitmap is
limited to offload related features. Make this more generic by renaming
it to vf_cap_flags, which allows for other capabilities besides
offloading to be added.

Signed-off-by: Stefan Assmann &lt;sassmann@kpanic.de&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>virtchnl: Add compile time static asserts to validate structure sizes</title>
<updated>2017-06-01T21:26:23Z</updated>
<author>
<name>Sridhar Samudrala</name>
<email>sridhar.samudrala@intel.com</email>
</author>
<published>2017-05-11T18:23:19Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=73556269aab30c39cba9cf8efafc402d0deb87b2'/>
<id>urn:sha1:73556269aab30c39cba9cf8efafc402d0deb87b2</id>
<content type='text'>
This uses preprocessor tricks to make sure that a divide by zero
occurs if a struct changes size outside the expected number of
bytes.

Signed-off-by: Sridhar Samudrala &lt;sridhar.samudrala@intel.com&gt;
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>
</feed>
