Re: [dpdk-dev] [PATCH v2 4/4] net/mlx4: add new Memory Region support

[Yongseok Koh <yskoh@mellanox.com>]

> On May 9, 2018, at 8:00 PM, Yongseok Koh <yskoh@mellanox.com> wrote:
> 
> 
>> On May 9, 2018, at 4:12 PM, Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>> 
>> On 5/9/2018 12:09 PM, Yongseok Koh wrote:
>> <...>
>> 
>>> +/**
>>> + * Insert an entry to B-tree lookup table.
>>> + *
>>> + * @param bt
>>> + *   Pointer to B-tree structure.
>>> + * @param entry
>>> + *   Pointer to new entry to insert.
>>> + *
>>> + * @return
>>> + *   0 on success, -1 on failure.
>>> + */
>>> +static int
>>> +mr_btree_insert(struct mlx4_mr_btree *bt, struct mlx4_mr_cache *entry)
>>> +{
>>> +	struct mlx4_mr_cache *lkp_tbl;
>>> +	uint16_t idx = 0;
>>> +	size_t shift;
>>> +
>>> +	assert(bt != NULL);
>>> +	assert(bt->len <= bt->size);
>>> +	assert(bt->len > 0);
>>> +	lkp_tbl = *bt->table;
>>> +	/* Find out the slot for insertion. */
>>> +	if (mr_btree_lookup(bt, &idx, entry->start) != UINT32_MAX) {
>>> +		DEBUG("abort insertion to B-tree(%p):"
>>> +		      " already exist at idx=%u [0x%lx, 0x%lx) lkey=0x%x",
>>> +		      (void *)bt, idx, entry->start, entry->end, entry->lkey);
>> 
>> This and various other logs causing 32bits build error because of %lx usage. Can
>> you please check them?
>> 
>> I am feeling sad to complain a patch like this just because of log format issue,
>> we should find a solution to this issue as community, either checkpatch checks
>> or automated 32bit builds, I don't know.
> 
> Bummer. I have to change my bad habit of using %lx. And we will add 32-bit build
> check to our internal system to filter this kind of mistakes beforehand.
> 
> Will work with Shahaf to fix it and rebase next-net-mlx.

Ferruh, I've sent out a patch to Shahaf to change printing format specifiers and
Shahaf will squash it into the previous patches.

However, it seems we had stopped supporting 32-bit compilation since Nelio's
commit [1]

Not sure I'm doing right but I'm compiling it for T=i686-native-linuxapp-gcc and
still having a few more errors even except for my code. And even if I fix all of
the errors, linkage fails as explained in the commit message of [1].

Are you sure you encountered this 32b compilation issue for the first time?


[1] http://dpdk.org/browse/dpdk/commit/?id=ebbb81eb27daca0a89ee8f228fcf141d9eb6ef1c


Thanks,
Yongseok