> -----Original Message-----
> From: David Miller <[email protected]>
> Sent: Monday, September 16, 2019 10:45 PM
> To: Jubran, Samih <[email protected]>
> Cc: [email protected]; Woodhouse, David <[email protected]>;
> Machulsky, Zorik <[email protected]>; Matushevsky, Alexander
> <[email protected]>; Bshara, Saeed <[email protected]>; Wilson,
> Matt <[email protected]>; Liguori, Anthony <[email protected]>;
> Bshara, Nafea <[email protected]>; Tzalik, Guy <[email protected]>;
> Belgazal, Netanel <[email protected]>; Saidi, Ali
> <[email protected]>; Herrenschmidt, Benjamin <[email protected]>;
> Kiyanovski, Arthur <[email protected]>
> Subject: Re: [PATCH V1 net-next 2/5] net: ena: multiple queue creation
> related cleanups
>
> From: <[email protected]>
> Date: Sun, 15 Sep 2019 18:27:19 +0300
>
> > @@ -1885,6 +1885,13 @@ static int ena_up(struct ena_adapter *adapter)
> > if (rc)
> > goto err_req_irq;
> >
> > + netif_info(adapter, ifup, adapter->netdev, "creating %d io queues.
> rx queue size: %d tx queue size. %d LLQ is %s\n",
> > + adapter->num_io_queues,
> > + adapter->requested_rx_ring_size,
> > + adapter->requested_tx_ring_size,
> > + (adapter->ena_dev->tx_mem_queue_type ==
> ENA_ADMIN_PLACEMENT_POLICY_DEV) ?
> > + "ENABLED" : "DISABLED");
>
> Please don't clog up the kernel log with stuff like this.
>
> Maybe netif_debug() at best, but I'd rather you remove this entirely. It's so
> easy to make a device go up and down repeatedly multiple times in one
> second.
Dropped in v2.
