From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuanhan Liu Subject: Re: [PATCH v2] net/virtio-user: fix not working on 32-bit system Date: Wed, 19 Apr 2017 14:24:03 +0800 Message-ID: <20170419062403.GC31523@yliu-dev.sh.intel.com> References: <1492092776-57008-1-git-send-email-jianfeng.tan@intel.com> <1492569033-140348-1-git-send-email-jianfeng.tan@intel.com> <20170419055349.GA31523@yliu-dev.sh.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "dev@dpdk.org" , "olivier.matz@6wind.com" , "stable@dpdk.org" To: "Tan, Jianfeng" Return-path: Content-Disposition: inline In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, Apr 19, 2017 at 06:21:59AM +0000, Tan, Jianfeng wrote: > > > > I've expected it to be plain english, something like following: > > > > /* > > * Return the physical address (or virtual address in case of > > * virtio-user) of mbuf data buffer. > > * > > * The address is firstly casted to the word size (sizeof(uintptr_t)) > > * before casting it to uint64_t. This is to make it work with different > > * combination of word size (64 bit and 32 bit) and virtio device > > * (virtio-pci and virtio-user). > > */ > > > > Okay to you? > > Yep, sounds better. Thanks! Good. Applied to dpdk-next-virtio, with above change. Thanks. --yliu