Re: [PATCH 2/2] thermal: shorten too long mcast group name
From: Thomas Graf <tgraf@suug.ch>
Date: 2013-03-21 13:04:13
On 03/19/13 at 08:47pm, Masatake YAMATO wrote:
quoted hunk ↗ jump to hunk
The original name is too long. Signed-off-by: Masatake YAMATO <redacted> --- include/linux/thermal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/include/linux/thermal.h b/include/linux/thermal.h index f0bd7f9..e3c0ae9 100644 --- a/include/linux/thermal.h +++ b/include/linux/thermal.h@@ -44,7 +44,7 @@ /* Adding event notification support elements */ #define THERMAL_GENL_FAMILY_NAME "thermal_event" #define THERMAL_GENL_VERSION 0x01 -#define THERMAL_GENL_MCAST_GROUP_NAME "thermal_mc_group" +#define THERMAL_GENL_MCAST_GROUP_NAME "thermal_mc_grp"
Just saying that this would not be OK in general as this would break any application looking up the multicast group id by name via genetlink but since it's broken anyway right now it's OK I guess.