DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
@ 2018-06-29  0:10 Yeddula, Avinash
  2018-06-29 11:17 ` Dumitrescu, Cristian
  0 siblings, 1 reply; 5+ messages in thread
From: Yeddula, Avinash @ 2018-06-29  0:10 UTC (permalink / raw)
  To: dev, dev, users; +Cc: Bly, Mike

Hello,

We are in process of migrating our design from DPDK 17.05 to 17.11 and we ran into a small problem. Within our design, we have some hash tables with 4-byte keys. While going through the changes done in 17.11, we have found there was an added key_size check, which now requires key_size >= 8 bytes (see check_params_create() in rte_table_hash_ext.c). Not seeing any other options, so I was hoping someone could advise on how to support a 4-byte hash key size in 17.11 and on a go forward basis.

Regards,
Avinash


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

* Re: [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
  2018-06-29  0:10 [dpdk-dev] 17.05 --> 17.11, minimum hash table key size Yeddula, Avinash
@ 2018-06-29 11:17 ` Dumitrescu, Cristian
  2018-06-29 20:00   ` Bly, Mike
  0 siblings, 1 reply; 5+ messages in thread
From: Dumitrescu, Cristian @ 2018-06-29 11:17 UTC (permalink / raw)
  To: Yeddula, Avinash, dev, dev, users; +Cc: Bly, Mike


> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Yeddula, Avinash
> Sent: Friday, June 29, 2018 1:11 AM
> To: dev@dpdk.org; dev <dev-bounces@dpdk.org>; users@dpdk.org
> Cc: Bly, Mike <mbly@ciena.com>
> Subject: [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
> 
> Hello,
> 
> We are in process of migrating our design from DPDK 17.05 to 17.11 and we
> ran into a small problem. Within our design, we have some hash tables with
> 4-byte keys. While going through the changes done in 17.11, we have found
> there was an added key_size check, which now requires key_size >= 8 bytes
> (see check_params_create() in rte_table_hash_ext.c). Not seeing any other
> options, so I was hoping someone could advise on how to support a 4-byte
> hash key size in 17.11 and on a go forward basis.
> 
> Regards,
> Avinash

Hi Avinash,

Simply declare you key size to be 8 bytes for any key size less or equal to 8 bytes and set the key mask parameter appropriately so that the additional bytes are masked out.

Regards,
Cristian

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

* Re: [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
  2018-06-29 11:17 ` Dumitrescu, Cristian
@ 2018-06-29 20:00   ` Bly, Mike
  2018-07-02  9:33     ` Dumitrescu, Cristian
  0 siblings, 1 reply; 5+ messages in thread
From: Bly, Mike @ 2018-06-29 20:00 UTC (permalink / raw)
  To: Dumitrescu, Cristian, Yeddula, Avinash, dev, dev, users

Cristian,

Thank you. I have done the recommended change. The next thing we have run into is the f_hash selection. We were historically using rte_hash_crc_8byte() from rte_has_crc.h. In looking at DPDK content, I do not see anything close in an rte_*.h file to use with the appropriate input parameter list. I did find use of hash_default_key8() in examples/ip_pipeline/pipeline/hash_func.h. Is there no formal rte_*.h replacement/update for this relative to the hash-table changes in 17.11+? Is the expectation that the implementer will create their own as done with the aforementioned ip_pipeline example?

-MikeB

-----Original Message-----
From: Dumitrescu, Cristian <cristian.dumitrescu@intel.com> 
Sent: Friday, June 29, 2018 4:18 AM
To: Yeddula, Avinash <ayeddula@ciena.com>; dev@dpdk.org; dev <dev-bounces@dpdk.org>; users@dpdk.org
Cc: Bly, Mike <mbly@ciena.com>
Subject: [**EXTERNAL**] RE: 17.05 --> 17.11, minimum hash table key size


> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Yeddula, Avinash
> Sent: Friday, June 29, 2018 1:11 AM
> To: dev@dpdk.org; dev <dev-bounces@dpdk.org>; users@dpdk.org
> Cc: Bly, Mike <mbly@ciena.com>
> Subject: [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
> 
> Hello,
> 
> We are in process of migrating our design from DPDK 17.05 to 17.11 and 
> we ran into a small problem. Within our design, we have some hash 
> tables with 4-byte keys. While going through the changes done in 
> 17.11, we have found there was an added key_size check, which now 
> requires key_size >= 8 bytes (see check_params_create() in 
> rte_table_hash_ext.c). Not seeing any other options, so I was hoping 
> someone could advise on how to support a 4-byte hash key size in 17.11 and on a go forward basis.
> 
> Regards,
> Avinash

Hi Avinash,

Simply declare you key size to be 8 bytes for any key size less or equal to 8 bytes and set the key mask parameter appropriately so that the additional bytes are masked out.

Regards,
Cristian

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

* Re: [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
  2018-06-29 20:00   ` Bly, Mike
@ 2018-07-02  9:33     ` Dumitrescu, Cristian
  0 siblings, 0 replies; 5+ messages in thread
From: Dumitrescu, Cristian @ 2018-07-02  9:33 UTC (permalink / raw)
  To: Bly, Mike, Yeddula, Avinash, dev, dev, users, Singh, Jasvinder,
	Laatz, Kevin



> -----Original Message-----
> From: Bly, Mike [mailto:mbly@ciena.com]
> Sent: Friday, June 29, 2018 9:01 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu@intel.com>; Yeddula, Avinash
> <ayeddula@ciena.com>; dev@dpdk.org; dev <dev-bounces@dpdk.org>;
> users@dpdk.org
> Subject: RE: 17.05 --> 17.11, minimum hash table key size
> 
> Cristian,
> 
> Thank you. I have done the recommended change. The next thing we have
> run into is the f_hash selection. We were historically using
> rte_hash_crc_8byte() from rte_has_crc.h. In looking at DPDK content, I do
> not see anything close in an rte_*.h file to use with the appropriate input
> parameter list. I did find use of hash_default_key8() in
> examples/ip_pipeline/pipeline/hash_func.h. Is there no formal rte_*.h
> replacement/update for this relative to the hash-table changes in 17.11+? Is
> the expectation that the implementer will create their own as done with the
> aforementioned ip_pipeline example?
> 
> -MikeB
> 

Hi Mike,

Yes, we are working to move the code from examples/ip_pipeline/hash_func.h into librte_table, Kevin is working on it right now.

Regards,
Cristian


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

* [dpdk-dev] 17.05 --> 17.11, minimum hash table key size
@ 2018-06-29  0:03 Bly, Mike
  0 siblings, 0 replies; 5+ messages in thread
From: Bly, Mike @ 2018-06-29  0:03 UTC (permalink / raw)
  To: dev

Hello,

We are in process of migrating our design from DPDK 17.05 to 17.11 and we ran into a small problem. Within our design, we have some hash tables with 4-byte keys. While going through the changes done in 17.11, we have found there was an added key_size check, which now requires key_size >= 8 bytes (see check_params_create() in rte_table_hash_ext.c). Not seeing any other options, so I was hoping someone could advise on how to support a 4-byte hash key size in 17.11 and on a go forward basis.

Regards,
Mike

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

end of thread, other threads:[~2018-07-02  9:33 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-29  0:10 [dpdk-dev] 17.05 --> 17.11, minimum hash table key size Yeddula, Avinash
2018-06-29 11:17 ` Dumitrescu, Cristian
2018-06-29 20:00   ` Bly, Mike
2018-07-02  9:33     ` Dumitrescu, Cristian
  -- strict thread matches above, loose matches on Subject: below --
2018-06-29  0:03 Bly, Mike

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).