<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/drivers/net/ethernet/pensando/ionic/ionic_ethtool.c, branch linux-5.12.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.12.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.12.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2020-12-09T00:22:54Z</updated>
<entry>
<title>drivers: net: ionic: simplify the return expression of ionic_set_rxfh()</title>
<updated>2020-12-09T00:22:54Z</updated>
<author>
<name>Zheng Yongjun</name>
</author>
<published>2020-12-08T13:53:53Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=10dd7b4fe504d4cf32fff4a0e3512fcf937305a8'/>
<id>urn:sha1:10dd7b4fe504d4cf32fff4a0e3512fcf937305a8</id>
<content type='text'>
Simplify the return expression.

Signed-off-by: Zheng Yongjun &lt;zhengyongjun3@huawei.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: check port ptr before use</title>
<updated>2020-11-05T17:58:25Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-11-04T19:56:06Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=2bcbf42add911ef63a6d90e92001dc2bcb053e68'/>
<id>urn:sha1:2bcbf42add911ef63a6d90e92001dc2bcb053e68</id>
<content type='text'>
Check for corner case of port_init failure before using
the port_info pointer.

Fixes: 4d03e00a2140 ("ionic: Add initial ethtool support")
Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Link: https://lore.kernel.org/r/20201104195606.61184-1-snelson@pensando.io
Signed-off-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
</content>
</entry>
<entry>
<title>ionic: dynamic interrupt moderation</title>
<updated>2020-09-17T00:35:47Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-09-15T23:59:03Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=04a834592bf51355383db6fc00cb09f776a6b8dc'/>
<id>urn:sha1:04a834592bf51355383db6fc00cb09f776a6b8dc</id>
<content type='text'>
Use the dim library to manage dynamic interrupt
moderation in ionic.

v3: rebase
v2: untangled declarations in ionic_dim_work()

Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Acked-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: clarify boolean precedence</title>
<updated>2020-09-02T22:47:01Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-09-01T18:20:24Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=2aaa05a1585f5a0b7d81f8261824872bd96377cb'/>
<id>urn:sha1:2aaa05a1585f5a0b7d81f8261824872bd96377cb</id>
<content type='text'>
Add parenthesis to clarify a boolean usage.

Pointed out in https://lore.kernel.org/lkml/202008060413.VgrMuqLJ%25lkp@intel.com/

Reported-by: kernel test robot &lt;lkp@intel.com&gt;
Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: change queue count with no reset</title>
<updated>2020-08-28T15:01:30Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-08-27T23:00:29Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=101b40a0171facc0349b6304f33392c01b96d8fd'/>
<id>urn:sha1:101b40a0171facc0349b6304f33392c01b96d8fd</id>
<content type='text'>
Add to our new ionic_reconfigure_queues() to also be able to change
the number of queues in use, and to change the queue interrupt layout
between split and combined.

Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: change the descriptor ring length without full reset</title>
<updated>2020-08-28T15:01:30Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-08-27T23:00:28Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=a34e25ab977ced1c3366fc69c0595bb3fde63fad'/>
<id>urn:sha1:a34e25ab977ced1c3366fc69c0595bb3fde63fad</id>
<content type='text'>
The original way of changing ring length was to completely
tear down the lif's queue structure and then rebuild it, while
running the risk of allocations that might fail in the middle
and leave us with a broken driver.

Instead, we can set up all the new queue and descriptor
allocations first, then swap them out and delete the old
allocations.  If the new allocations fail, we report the error,
stay with the old setup and continue running.  This gives us
a safer path, and a smaller window of time where we're not
processing traffic.

Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: rework and simplify handling of the queue stats block</title>
<updated>2020-08-28T15:01:30Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-08-27T23:00:23Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=34dec947b9ceff8a822ece467aac49b0ac992b8e'/>
<id>urn:sha1:34dec947b9ceff8a822ece467aac49b0ac992b8e</id>
<content type='text'>
Use a block of stats structs attached to the lif instead of
little ones attached to each qcq.  This simplifies our memory
management and gets rid of a lot of unnecessary indirection.

Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: separate interrupt for Tx and Rx</title>
<updated>2020-08-03T22:32:02Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-07-31T20:15:36Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=fe8c30b50835436152a4d2319f834acb36b3b659'/>
<id>urn:sha1:fe8c30b50835436152a4d2319f834acb36b3b659</id>
<content type='text'>
Add the capability to split the Tx queues onto their own
interrupts with their own napi contexts.  This gives the
opportunity for more direct control of Tx interrupt
handling, such as CPU affinity and interrupt coalescing,
useful for some traffic loads.

v2: use ethtool -L, not a vendor specific priv-flag
v3: simplify logging, drop unnecessary "no-change" tests

Reviewed-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: use offset for ethtool regs data</title>
<updated>2020-07-21T01:09:38Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-07-20T23:00:13Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=f85ae16f924f92a370b81b4e77862c1c59882fce'/>
<id>urn:sha1:f85ae16f924f92a370b81b4e77862c1c59882fce</id>
<content type='text'>
Use an offset to write the second half of the regs data into the
second half of the buffer instead of overwriting the first half.

Fixes: 4d03e00a2140 ("ionic: Add initial ethtool support")
Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>ionic: centralize queue reset code</title>
<updated>2020-07-07T22:50:31Z</updated>
<author>
<name>Shannon Nelson</name>
</author>
<published>2020-07-07T21:13:26Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=086c18f2452d0028f81e319f098bcb8e53133dbf'/>
<id>urn:sha1:086c18f2452d0028f81e319f098bcb8e53133dbf</id>
<content type='text'>
The queue reset pattern is used in a couple different places,
only slightly different from each other, and could cause
issues if one gets changed and the other didn't.  This puts
them together so that only one version is needed, yet each
can have slighty different effects by passing in a pointer
to a work function to do whatever configuration twiddling is
needed in the middle of the reset.

This specifically addresses issues seen where under loops
of changing ring size or queue count parameters we could
occasionally bump into the netdev watchdog.

v2: added more commit message commentary

Fixes: 4d03e00a2140 ("ionic: Add initial ethtool support")
Signed-off-by: Shannon Nelson &lt;snelson@pensando.io&gt;
Acked-by: Jakub Kicinski &lt;kuba@kernel.org&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
</feed>
