Re: [PATCH v5 01/11] virtio: Introduce config RTE_VIRTIO_INC_VECTOR

On Fri, Jan 29, 2016 at 10:12 AM, Yuanhan Liu
[off-list ref] wrote:

On Fri, Jan 29, 2016 at 10:02:26AM +0530, Santosh Shukla wrote:

quoted

Hi Yuan,

It's Yuanhan, but not Yuan :)

Sorry for that,

quoted

On Wed, Jan 27, 2016 at 8:03 AM, Yuanhan Liu
[off-list ref] wrote:

quoted

On Wed, Jan 27, 2016 at 07:53:21AM +0530, Santosh Shukla wrote:

quoted

Ping?

I was on vacation late last week. And I was quite busy till now after
the vacation. So, sorry that I still don't have time to do more detailed
reviews in 1 or 2 days. Hopefully I can make it by this Friday.

BTW, I had a very glimpse of this patchset, overall, it looks much
better now, except the EAL changes (I'm not the maintainer) and the
virtio io port read/write stuff: Tetsuay suggested to add another
access wraps, but I have few concerns about that. Anyway, I don't
have time for deeper thoughts, and I will re-think it later.

        --yliu

did you got the chance for reviewing virtio specific patches? Thanks.

Sorry for the long delay, it's very likely I will check your patches
this noon.

Thanks

        --yliu