DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1"
@ 2014-10-18 22:50 Matthew Hall
  2014-10-19 13:46 ` Marc Sune
  0 siblings, 1 reply; 5+ messages in thread
From: Matthew Hall @ 2014-10-18 22:50 UTC (permalink / raw)
  To: dev

Hello,

I'm just trying to understand what you're supposed to do about this error to 
get the optiomal configuration / performance. The error message and comments 
seem like they're designed for Intel ethernet driver hackers not security 
hackers like myself! ;-)

Note: I'm trying out the Intel 82545EM in VirtualBox because the virtio-net 
approach didn't work, for reasons I enumerated in some previous mails.

    /*
     * If rs_bit_thresh is greater than 1, then TX WTHRESH should be
     * set to 0. If WTHRESH is greater than zero, the RS bit is ignored
     * by the NIC and all descriptors are written back after the NIC
     * accumulates WTHRESH descriptors.
     */
    if (tx_conf->tx_thresh.wthresh != 0 && tx_rs_thresh != 1) {
        RTE_LOG(ERR, PMD, "TX WTHRESH must be set to 0 if "
            "tx_rs_thresh is greater than 1. (tx_rs_thresh=%u "
            "port=%d queue=%d)\n", (unsigned int)tx_rs_thresh,
                (int)dev->data->port_id, (int)queue_idx);
        return -(EINVAL);
    }

Thanks,
Matthew.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1"
  2014-10-18 22:50 [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1" Matthew Hall
@ 2014-10-19 13:46 ` Marc Sune
  2014-10-19 17:08   ` Matthew Hall
  0 siblings, 1 reply; 5+ messages in thread
From: Marc Sune @ 2014-10-19 13:46 UTC (permalink / raw)
  To: dev

Which DPDK version are you using

marc

On 19/10/14 00:50, Matthew Hall wrote:
> Hello,
>
> I'm just trying to understand what you're supposed to do about this error to
> get the optiomal configuration / performance. The error message and comments
> seem like they're designed for Intel ethernet driver hackers not security
> hackers like myself! ;-)
>
> Note: I'm trying out the Intel 82545EM in VirtualBox because the virtio-net
> approach didn't work, for reasons I enumerated in some previous mails.
>
>      /*
>       * If rs_bit_thresh is greater than 1, then TX WTHRESH should be
>       * set to 0. If WTHRESH is greater than zero, the RS bit is ignored
>       * by the NIC and all descriptors are written back after the NIC
>       * accumulates WTHRESH descriptors.
>       */
>      if (tx_conf->tx_thresh.wthresh != 0 && tx_rs_thresh != 1) {
>          RTE_LOG(ERR, PMD, "TX WTHRESH must be set to 0 if "
>              "tx_rs_thresh is greater than 1. (tx_rs_thresh=%u "
>              "port=%d queue=%d)\n", (unsigned int)tx_rs_thresh,
>                  (int)dev->data->port_id, (int)queue_idx);
>          return -(EINVAL);
>      }
>
> Thanks,
> Matthew.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1"
  2014-10-19 13:46 ` Marc Sune
@ 2014-10-19 17:08   ` Matthew Hall
  2014-10-20  9:36     ` Bruce Richardson
  0 siblings, 1 reply; 5+ messages in thread
From: Matthew Hall @ 2014-10-19 17:08 UTC (permalink / raw)
  To: Marc Sune, dev

1.7.1 with a few minor clang compatibility patches in the example apps
-- 
Sent from my mobile device.

On October 19, 2014 6:46:27 AM PDT, Marc Sune <marc.sune@bisdn.de> wrote:
>Which DPDK version are you using
>
>marc
>
>On 19/10/14 00:50, Matthew Hall wrote:
>> Hello,
>>
>> I'm just trying to understand what you're supposed to do about this
>error to
>> get the optiomal configuration / performance. The error message and
>comments
>> seem like they're designed for Intel ethernet driver hackers not
>security
>> hackers like myself! ;-)
>>
>> Note: I'm trying out the Intel 82545EM in VirtualBox because the
>virtio-net
>> approach didn't work, for reasons I enumerated in some previous
>mails.
>>
>>      /*
>>       * If rs_bit_thresh is greater than 1, then TX WTHRESH should be
>>       * set to 0. If WTHRESH is greater than zero, the RS bit is
>ignored
>>       * by the NIC and all descriptors are written back after the NIC
>>       * accumulates WTHRESH descriptors.
>>       */
>>      if (tx_conf->tx_thresh.wthresh != 0 && tx_rs_thresh != 1) {
>>          RTE_LOG(ERR, PMD, "TX WTHRESH must be set to 0 if "
>>              "tx_rs_thresh is greater than 1. (tx_rs_thresh=%u "
>>              "port=%d queue=%d)\n", (unsigned int)tx_rs_thresh,
>>                  (int)dev->data->port_id, (int)queue_idx);
>>          return -(EINVAL);
>>      }
>>
>> Thanks,
>> Matthew.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1"
  2014-10-19 17:08   ` Matthew Hall
@ 2014-10-20  9:36     ` Bruce Richardson
  2014-10-20 20:29       ` Matthew Hall
  0 siblings, 1 reply; 5+ messages in thread
From: Bruce Richardson @ 2014-10-20  9:36 UTC (permalink / raw)
  To: Matthew Hall; +Cc: dev

On Sun, Oct 19, 2014 at 10:08:29AM -0700, Matthew Hall wrote:
> 1.7.1 with a few minor clang compatibility patches in the example apps

Rather than trying to tune the results yourself, maybe you could look to 
merge in the following patch set also, to offload responsibility for the 
driver settings to the driver itself.

http://thread.gmane.org/gmane.comp.networking.dpdk.devel/6189

/Bruce

> -- 
> Sent from my mobile device.
> 
> On October 19, 2014 6:46:27 AM PDT, Marc Sune <marc.sune@bisdn.de> wrote:
> >Which DPDK version are you using
> >
> >marc
> >
> >On 19/10/14 00:50, Matthew Hall wrote:
> >> Hello,
> >>
> >> I'm just trying to understand what you're supposed to do about this
> >error to
> >> get the optiomal configuration / performance. The error message and
> >comments
> >> seem like they're designed for Intel ethernet driver hackers not
> >security
> >> hackers like myself! ;-)
> >>
> >> Note: I'm trying out the Intel 82545EM in VirtualBox because the
> >virtio-net
> >> approach didn't work, for reasons I enumerated in some previous
> >mails.
> >>
> >>      /*
> >>       * If rs_bit_thresh is greater than 1, then TX WTHRESH should be
> >>       * set to 0. If WTHRESH is greater than zero, the RS bit is
> >ignored
> >>       * by the NIC and all descriptors are written back after the NIC
> >>       * accumulates WTHRESH descriptors.
> >>       */
> >>      if (tx_conf->tx_thresh.wthresh != 0 && tx_rs_thresh != 1) {
> >>          RTE_LOG(ERR, PMD, "TX WTHRESH must be set to 0 if "
> >>              "tx_rs_thresh is greater than 1. (tx_rs_thresh=%u "
> >>              "port=%d queue=%d)\n", (unsigned int)tx_rs_thresh,
> >>                  (int)dev->data->port_id, (int)queue_idx);
> >>          return -(EINVAL);
> >>      }
> >>
> >> Thanks,
> >> Matthew.
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1"
  2014-10-20  9:36     ` Bruce Richardson
@ 2014-10-20 20:29       ` Matthew Hall
  0 siblings, 0 replies; 5+ messages in thread
From: Matthew Hall @ 2014-10-20 20:29 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: dev

On Mon, Oct 20, 2014 at 10:36:01AM +0100, Bruce Richardson wrote:
> On Sun, Oct 19, 2014 at 10:08:29AM -0700, Matthew Hall wrote:
> > 1.7.1 with a few minor clang compatibility patches in the example apps
> 
> Rather than trying to tune the results yourself, maybe you could look to 
> merge in the following patch set also, to offload responsibility for the 
> driver settings to the driver itself.
> 
> http://thread.gmane.org/gmane.comp.networking.dpdk.devel/6189
> 
> /Bruce

Sounds good... I'll try it out and see if I can cherry-pick it onto my 
codebase.

Matthew.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-10-20 20:22 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-18 22:50 [dpdk-dev] "TX WTHRESH must be set to 0 if tx_rs_thresh is greater than 1" Matthew Hall
2014-10-19 13:46 ` Marc Sune
2014-10-19 17:08   ` Matthew Hall
2014-10-20  9:36     ` Bruce Richardson
2014-10-20 20:29       ` Matthew Hall

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git