On Sun, Mar 09, 2025 at 10:01:33PM +0000, Haiyang Zhang wrote:
>
>
> > -----Original Message-----
> > From: Jakub Kicinski <k...@kernel.org>
> > Sent: Friday, March 7, 2025 10:50 PM
> > To: Haiyang Zhang <haiya...@microsoft.com>
> > Cc: linux-hyperv@vger.kernel.org; net...@vger.kernel.org; Dexuan Cui
> > <de...@microsoft.com>; step...@networkplumber.org; KY Srinivasan
> > <k...@microsoft.com>; Paul Rosswurm <paul...@microsoft.com>;
> > o...@aepfle.de; vkuzn...@redhat.com; da...@davemloft.net;
> > wei....@kernel.org; eduma...@google.com; pab...@redhat.com;
> > l...@kernel.org; Long Li <lon...@microsoft.com>;
> > ssen...@linux.microsoft.com; linux-r...@vger.kernel.org;
> > dan...@iogearbox.net; john.fastab...@gmail.com; b...@vger.kernel.org;
> > a...@kernel.org; h...@kernel.org; t...@linutronix.de;
> > shradhagu...@linux.microsoft.com; linux-ker...@vger.kernel.org;
> > sta...@vger.kernel.org
> > Subject: [EXTERNAL] Re: [PATCH net] net: mana: Support holes in device
> > list reply msg
> >
> > On Wed, 5 Mar 2025 13:46:21 -0800 Haiyang Zhang wrote:
> > > - for (i = 0; i < max_num_devs; i++) {
> > > + for (i = 0; i < GDMA_DEV_LIST_SIZE &&
> > > + found_dev < resp.num_of_devs; i++) {
> >
> > unfortunate mis-indent here, it blend with the code.
> > checkpatch is right that it should be aligned with opening bracket
> Will fix it.
>
> >
> > > dev = resp.devs[i];
> > > dev_type = dev.type;
> > >
> > > + /* Skip empty devices */
> > > + if (dev.as_uint32 == 0)
> > > + continue;
> > > +
> > > + found_dev++;
> > > + dev_info(gc->dev, "Got devidx:%u, type:%u, instance:%u\n", i,
> > > + dev.type, dev.instance);
> >
> > Are you sure you want to print this info message for each device,
> > each time it's probed? Seems pretty noisy. We generally recommend
> > printing about _unusual_ things.
> Ok. I can remove it.
How about a dev_dbg instead?
>
> Thanks,
> - Haiyang
