Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1253780
| From | "Michael S. Tsirkin" <mst@redhat.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package |
| Date | 2015-10-22 15:00 +0200 |
| Message-ID | <qmnDk-2Sv-15@gated-at.bofh.it> (permalink) |
| References | <qm4Km-nt-11@gated-at.bofh.it> <qm4U4-zk-33@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Thu, Oct 22, 2015 at 12:37:40AM +0800, Lan Tianyu wrote:
> When transmit a package, the end transmit desc of package
> indicates whether package is sent already. Current code records
> the end desc's pointer in the next_to_watch of struct tx buffer.
> This code will be broken if shifting desc ring after migration.
> The pointer will be invalid. This patch is to replace recording
> pointer with recording the desc number of the package and find
> the end decs via the first desc and desc number.
>
> Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
Do you really need to play the shifting games?
Can't you just reset everything and re-initialize the rings?
It's slower but way less intrusive.
Also removes the need to track writes into rings.
> ---
> drivers/net/ethernet/intel/ixgbevf/ixgbevf.h | 1 +
> drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 19 ++++++++++++++++---
> 2 files changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h b/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
> index 775d089..c823616 100644
> --- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
> +++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf.h
> @@ -54,6 +54,7 @@
> */
> struct ixgbevf_tx_buffer {
> union ixgbe_adv_tx_desc *next_to_watch;
> + u16 desc_num;
> unsigned long time_stamp;
> struct sk_buff *skb;
> unsigned int bytecount;
> diff --git a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
> index 4446916..056841c 100644
> --- a/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
> +++ b/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
> @@ -210,6 +210,7 @@ static void ixgbevf_unmap_and_free_tx_resource(struct ixgbevf_ring *tx_ring,
> DMA_TO_DEVICE);
> }
> tx_buffer->next_to_watch = NULL;
> + tx_buffer->desc_num = 0;
> tx_buffer->skb = NULL;
> dma_unmap_len_set(tx_buffer, len, 0);
> /* tx_buffer must be completely set up in the transmit path */
> @@ -295,7 +296,7 @@ static bool ixgbevf_clean_tx_irq(struct ixgbevf_q_vector *q_vector,
> union ixgbe_adv_tx_desc *tx_desc;
> unsigned int total_bytes = 0, total_packets = 0;
> unsigned int budget = tx_ring->count / 2;
> - unsigned int i = tx_ring->next_to_clean;
> + int i, watch_index;
>
> if (test_bit(__IXGBEVF_DOWN, &adapter->state))
> return true;
> @@ -305,9 +306,17 @@ static bool ixgbevf_clean_tx_irq(struct ixgbevf_q_vector *q_vector,
> i -= tx_ring->count;
>
> do {
> - union ixgbe_adv_tx_desc *eop_desc = tx_buffer->next_to_watch;
> + union ixgbe_adv_tx_desc *eop_desc;
> +
> + if (!tx_buffer->desc_num)
> + break;
> +
> + if (i + tx_buffer->desc_num >= 0)
> + watch_index = i + tx_buffer->desc_num;
> + else
> + watch_index = i + tx_ring->count + tx_buffer->desc_num;
>
> - /* if next_to_watch is not set then there is no work pending */
> + eop_desc = IXGBEVF_TX_DESC(tx_ring, watch_index);
> if (!eop_desc)
> break;
>
> @@ -320,6 +329,7 @@ static bool ixgbevf_clean_tx_irq(struct ixgbevf_q_vector *q_vector,
>
> /* clear next_to_watch to prevent false hangs */
> tx_buffer->next_to_watch = NULL;
> + tx_buffer->desc_num = 0;
>
> /* update the statistics for this packet */
> total_bytes += tx_buffer->bytecount;
> @@ -3457,6 +3467,7 @@ static void ixgbevf_tx_map(struct ixgbevf_ring *tx_ring,
> u32 tx_flags = first->tx_flags;
> __le32 cmd_type;
> u16 i = tx_ring->next_to_use;
> + u16 start;
>
> tx_desc = IXGBEVF_TX_DESC(tx_ring, i);
>
> @@ -3540,6 +3551,8 @@ static void ixgbevf_tx_map(struct ixgbevf_ring *tx_ring,
>
> /* set next_to_watch value indicating a packet is present */
> first->next_to_watch = tx_desc;
> + start = first - tx_ring->tx_buffer_info;
> + first->desc_num = (i - start >= 0) ? i - start: i + tx_ring->count - start;
>
> i++;
> if (i == tx_ring->count)
> --
> 1.8.4.rc0.1.g8f6a3e5.dirty
>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Lan Tianyu <tianyu.lan@intel.com> - 2015-10-21 19:00 +0200
[RFC Patch 10/12] IXGBEVF: Add lock to protect tx/rx ring operation Lan Tianyu <tianyu.lan@intel.com> - 2015-10-21 19:00 +0200
Re: [RFC Patch 10/12] IXGBEVF: Add lock to protect tx/rx ring operation Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-22 00:00 +0200
Re: [RFC Patch 10/12] IXGBEVF: Add lock to protect tx/rx ring operation "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 14:50 +0200
[RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package Lan Tianyu <tianyu.lan@intel.com> - 2015-10-21 19:00 +0200
Re: [RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-21 23:20 +0200
Re: [RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package "Lan, Tianyu" <tianyu.lan@intel.com> - 2015-10-24 18:20 +0200
Re: [RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 15:00 +0200
Re: [RFC Patch 08/12] IXGBEVF: Rework code of finding the end transmit desc of package "Lan, Tianyu" <tianyu.lan@intel.com> - 2015-10-24 18:10 +0200
[RFC Patch 09/12] IXGBEVF: Add live migration support for VF driver Lan Tianyu <tianyu.lan@intel.com> - 2015-10-21 19:00 +0200
Re: [RFC Patch 09/12] IXGBEVF: Add live migration support for VF driver Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-21 23:50 +0200
Re: [RFC Patch 09/12] IXGBEVF: Add live migration support for VF driver "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 14:50 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Or Gerlitz <gerlitz.or@gmail.com> - 2015-10-21 20:50 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alex Williamson <alex.williamson@redhat.com> - 2015-10-21 21:30 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-22 01:30 +0200
Re: [Qemu-devel] [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 14:40 +0200
Re: [Qemu-devel] [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alex Williamson <alex.williamson@redhat.com> - 2015-10-22 15:10 +0200
Re: [Qemu-devel] [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 15:10 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Or Gerlitz <gerlitz.or@gmail.com> - 2015-10-22 18:00 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alex Williamson <alex.williamson@redhat.com> - 2015-10-22 18:20 +0200
Re: [Qemu-devel] [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC "Michael S. Tsirkin" <mst@redhat.com> - 2015-10-22 15:00 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-23 20:40 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alex Williamson <alex.williamson@redhat.com> - 2015-10-23 21:10 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-23 22:10 +0200
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Lan Tianyu <tianyu.lan@intel.com> - 2015-10-26 06:50 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-26 16:10 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Lan Tianyu <tianyu.lan@intel.com> - 2015-10-29 07:30 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-29 08:00 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Lan Tianyu <tianyu.lan@intel.com> - 2015-10-29 09:50 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-29 17:20 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Lan Tianyu <tianyu.lan@intel.com> - 2015-10-30 04:00 +0100
Re: [RFC Patch 00/12] IXGBE: Add live migration support for SRIOV NIC Alexander Duyck <alexander.duyck@gmail.com> - 2015-10-30 19:10 +0100
csiph-web