On 11.08.2017 16:11, Bodireddy, Bhanuprakash wrote:
>> On 09.08.2017 15:35, Bodireddy, Bhanuprakash wrote:
>
> +static int
> +netdev_dpdk_vhost_tx_burst(struct netdev_dpdk *dev, int qid) {
> +struct dpdk_tx_queue *txq = &dev->tx_q[qid];
> +struct rte_mbuf **cur_pkts = (s
On 10.08.2017 21:52, Bodireddy, Bhanuprakash wrote:
>>>
> } else {
> +/* If the queue is disabled in the guest, the
> corresponding qid
> + * map shall be set to OVS_VHOST_QUEUE_DISABLED(-2).
> + *
> +
>On 09.08.2017 15:35, Bodireddy, Bhanuprakash wrote:
+static int
+netdev_dpdk_vhost_tx_burst(struct netdev_dpdk *dev, int qid) {
+struct dpdk_tx_queue *txq = &dev->tx_q[qid];
+struct rte_mbuf **cur_pkts = (struct rte_mbuf
+**)txq->vhost_burst_pkts;
+
>>
} else {
+/* If the queue is disabled in the guest, the
corresponding qid
+ * map shall be set to OVS_VHOST_QUEUE_DISABLED(-2).
+ *
+ * The packets that were queued in 'qid' could be
On 09.08.2017 15:35, Bodireddy, Bhanuprakash wrote:
>>>
>>> +static int
>>> +netdev_dpdk_vhost_tx_burst(struct netdev_dpdk *dev, int qid) {
>>> +struct dpdk_tx_queue *txq = &dev->tx_q[qid];
>>> +struct rte_mbuf **cur_pkts = (struct rte_mbuf
>>> +**)txq->vhost_burst_pkts;
>>> +
>>> +int
>enable)
if (enable) {
dev->tx_q[qid].map = qid;
>>
>> Here flushing required too because we're possibly enabling previously
>remapped queue.
>>
} else {
+/* If the queue is disabled in the guest, the
corresponding
>>
>> +static int
>> +netdev_dpdk_vhost_tx_burst(struct netdev_dpdk *dev, int qid) {
>> +struct dpdk_tx_queue *txq = &dev->tx_q[qid];
>> +struct rte_mbuf **cur_pkts = (struct rte_mbuf
>> +**)txq->vhost_burst_pkts;
>> +
>> +int tx_vid = netdev_dpdk_get_vid(dev);
>> +int tx_qid = qid
On 09.08.2017 13:03, Ilya Maximets wrote:
> One more comment inline.
>
> On 09.08.2017 11:06, Ilya Maximets wrote:
>> Not a full review.
>> One comment inline.
>>
>>> Add netdev_dpdk_vhost_txq_flush(), that flushes packets on vHost User
>>> port queues. Also add netdev_dpdk_vhost_tx_burst() functi
One more comment inline.
On 09.08.2017 11:06, Ilya Maximets wrote:
> Not a full review.
> One comment inline.
>
>> Add netdev_dpdk_vhost_txq_flush(), that flushes packets on vHost User
>> port queues. Also add netdev_dpdk_vhost_tx_burst() function that
>> uses rte_vhost_enqueue_burst() to enqueue
Not a full review.
One comment inline.
> Add netdev_dpdk_vhost_txq_flush(), that flushes packets on vHost User
> port queues. Also add netdev_dpdk_vhost_tx_burst() function that
> uses rte_vhost_enqueue_burst() to enqueue burst of packets on vHost User
> ports.
>
> Signed-off-by: Bhanuprakash Bod
Add netdev_dpdk_vhost_txq_flush(), that flushes packets on vHost User
port queues. Also add netdev_dpdk_vhost_tx_burst() function that
uses rte_vhost_enqueue_burst() to enqueue burst of packets on vHost User
ports.
Signed-off-by: Bhanuprakash Bodireddy
Signed-off-by: Antonio Fischetti
Co-authore
11 matches
Mail list logo