[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] net: remove an assert call in eth_get_gso_type
From: |
P J P |
Subject: |
Re: [PATCH v2] net: remove an assert call in eth_get_gso_type |
Date: |
Wed, 21 Oct 2020 15:43:20 +0530 (IST) |
+-- On Wed, 21 Oct 2020, Philippe Mathieu-Daudé wrote --+
| $ git grep qemu_log\( | wc -l
| 661
|
| This function seems used mostly by very old code.
| It is declared in "qemu/log-for-trace.h" which looks like an internal API.
|
| Should we add a checkpatch rule to refuse new uses of qemu_log()?
* That'll help, if it's not meant to be called directly.
* Better still would be to make qemu_log() static, called only via
qemu_log_mask(). That way compiler will show an error if qemu_log() is
called directly.
* While at it, could it be made to print '__func__' string by default?
Thank you.
--
Prasad J Pandit / Red Hat Product Security Team
8685 545E B54C 486B C6EB 271E E285 8B5A F050 DE8D