On 12/9/2019 9:46 PM, Alfredo Cardigliano wrote:
> Add support for the notify queue, which is used for events
> published by the NIC.
> 
> Signed-off-by: Alfredo Cardigliano <cardigli...@ntop.org>
> Reviewed-by: Shannon Nelson <snel...@pensando.io>

<...>

> +static int
> +ionic_lif_notifyq_init(struct ionic_lif *lif)
> +{
> +     struct ionic_dev *idev = &lif->adapter->idev;
> +     struct ionic_qcq *qcq = lif->notifyqcq;
> +     struct ionic_queue *q = &qcq->q;
> +     int err;
> +
> +     struct ionic_admin_ctx ctx = {
> +             .pending_work = true,
> +             .cmd.q_init = {
> +                     .opcode = IONIC_CMD_Q_INIT,
> +                     .lif_index = lif->index,
> +                     .type = q->type,
> +                     .index = q->index,
> +                     .flags = (IONIC_QINIT_F_IRQ | IONIC_QINIT_F_ENA),
> +                     .intr_index = qcq->intr.index,
> +                     .pid = q->pid,
> +                     .ring_size = ilog2(q->num_descs),
> +                     .ring_base = q->base_pa,
> +             }
> +     };
> +
> +     IONIC_PRINT(DEBUG, "notifyq_init.pid %d", ctx.cmd.q_init.pid);
> +     IONIC_PRINT(DEBUG, "notifyq_init.index %d",
> +             ctx.cmd.q_init.index);
> +     IONIC_PRINT(DEBUG, "notifyq_init.ring_base 0x%lx",
> +             ctx.cmd.q_init.ring_base);
> +     IONIC_PRINT(DEBUG, "notifyq_init.ring_size %d",
> +             ctx.cmd.q_init.ring_size);

Can you plesae use PRIx64 etc format identifiers to have correct type? The build
is failing in 32 bits just because of log format identifiers.

There are a few cases this is needed, building for i686 should highlight them.

Reply via email to