From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon@6wind.com>
Subject: Re: [ [PATCH v2] 01/13] virtio: Introduce config
	RTE_VIRTIO_INC_VECTOR
Date: Thu, 17 Dec 2015 13:03:45 +0100
Message-ID: <2955349.3QHN8DHquR@xps13>
References: <1450098032-21198-1-git-send-email-sshukla@mvista.com>
 <1450098032-21198-2-git-send-email-sshukla@mvista.com>
 <CAAyOgsbAyU1VhPCbZ4FjMwkk8OYs8gORP=+rXbmNQOxCX2mvSQ@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org
To: Santosh Shukla <sshukla@mvista.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47])
 by dpdk.org (Postfix) with ESMTP id 267435A86
 for <dev@dpdk.org>; Thu, 17 Dec 2015 13:05:03 +0100 (CET)
Received: by mail-wm0-f47.google.com with SMTP id l126so20375490wml.1
 for <dev@dpdk.org>; Thu, 17 Dec 2015 04:05:03 -0800 (PST)
In-Reply-To: <CAAyOgsbAyU1VhPCbZ4FjMwkk8OYs8gORP=+rXbmNQOxCX2mvSQ@mail.gmail.com>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

2015-12-17 17:32, Santosh Shukla:
> On Mon, Dec 14, 2015 at 6:30 PM, Santosh Shukla <sshukla@mvista.com> wrote:
> > virtio_recv_pkts_vec and other virtio vector friend apis are written for sse/avx
> > instructions. For arm64 in particular, virtio vector implementation does not
> > exist(todo).
> >
> > So virtio pmd driver wont build for targets like i686, arm64.  By making
> > RTE_VIRTIO_INC_VECTOR=n, Driver can build for non-sse/avx targets and will work
> > in non-vectored virtio mode.
> >
> > Signed-off-by: Santosh Shukla <sshukla@mvista.com>
> > ---
> 
> Ping?
> 
> any review  / comment on this patch much appreciated. Thanks

Why not check for SSE/AVX support instead of adding yet another config option?