18.11.2014 23:46, Neil Horman ?????: > On Tue, Nov 18, 2014 at 11:13:17PM +0600, Yerden Zhumabekov wrote: >> 18.11.2014 22:00, Neil Horman ?????: >>> >>> You need to edit the makefile so that the compiler gets passed the option >>> -msse42. That way it will know to emit sse42 instructions. It will also >>> allow >>> you to remove the ifdef from the include file >> In this case, I guess there are two options: >> 1) modify all makefiles which use librte_hash >> 2) move all function bodies from rte_hash_crc.h to separate module, >> leaving prototype definitions there only. >> >> Everybody's up for the second option? :) >> > Crud, you're right, I didn't think about the header inclusion issue. Is it > worth adding the jump to enable the dynamic hash selection?
If I understood you correctly - I've already added a function to dynamically change the CRC32 implementation in the runtime, rte_hash_crc_set_alg(). I can rework patches once again, if everybody's fine with the separate module. -- Sincerely, Yerden Zhumabekov State Technical Service Astana, KZ
