From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yerden Zhumabekov Subject: Re: [PATCH v6 2/7] hash: add assembly implementation of CRC32 intrinsics Date: Mon, 2 Feb 2015 11:34:01 +0600 Message-ID: <54CF0C49.5000002@sts.kz> References: <1409724351-23786-1-git-send-email-e_zhumabekov@sts.kz> <3fdc339aa3f0206ae2200937b9791af69e83409e.1417092208.git.e_zhumabekov@sts.kz> <54CF0805.8040103@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable To: "Liang, Cunming" , Return-path: In-Reply-To: <54CF0805.8040103-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 02.02.2015 11:15, Liang, Cunming =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > >> +static inline uint32_t >> +crc32c_sse42_u64(uint64_t data, uint64_t init_val) >> +{ >> + __asm__ volatile( >> + "crc32q %[data], %[init_val];" >> + : [init_val] "+r" (init_val) >> + : [data] "rm" (data)); >> + return init_val; >> +} > [LCM] I'm curious about the benefit of replacing CRC32 intrinsic > "_mm_crc32_u32/64". These intrinsics are not available on a platform which has no SSE4.2 support so the build would fail. See previous suggestion from Neil:=20 http://dpdk.org/ml/archives/dev/2014-November/008353.html --=20 Sincerely, Yerden Zhumabekov State Technical Service Astana, KZ