Re: [Intel-gfx] [PATCH] drm/i915/tc: Fix -Wformat-truncation in intel_tc_port_init
On 10/25/2023 5:45 PM, Imre Deak wrote: On Wed, Oct 25, 2023 at 03:57:26PM +0200, Nirmoy Das wrote: Fix below compiler warning: intel_tc.c:1879:11: error: ‘%d’ directive output may be truncated writing between 1 and 11 bytes into a region of size 3 [-Werror=format-truncation=] "%c/TC#%d", port_name(port), tc_port + 1); ^~ intel_tc.c:1878:2: note: ‘snprintf’ output between 7 and 17 bytes into a destination of size 8 snprintf(tc->port_name, sizeof(tc->port_name), ^~ "%c/TC#%d", port_name(port), tc_port + 1); ~ Fixes: fe03f637b7bd ("drm/i915/tc: Fix -Wformat-truncation in intel_tc_port_init") intel_tc_port struct") Cc: Mika Kahola Cc: Imre Deak Cc: Jani Nikula Signed-off-by: Nirmoy Das --- drivers/gpu/drm/i915/display/intel_tc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_tc.c b/drivers/gpu/drm/i915/display/intel_tc.c index 37b0f8529b4f..0afcee279aff 100644 --- a/drivers/gpu/drm/i915/display/intel_tc.c +++ b/drivers/gpu/drm/i915/display/intel_tc.c @@ -58,7 +58,7 @@ struct intel_tc_port { struct delayed_work link_reset_work; int link_refcount; bool legacy_port:1; - char port_name[8]; + char port_name[17]; I'd use instead kasprintf() to suppress the warn. Imo (at one point) the 'Port %s:', tc->port_name prefix could be replaced by the usual [ENCODER:%d:%s]. That is even better, thanks I will resend. Regards, Nirmoy enum tc_port_mode mode; enum tc_port_mode init_mode; enum phy_fia phy_fia; -- 2.41.0
Re: [Intel-gfx] [PATCH] drm/i915/tc: Fix -Wformat-truncation in intel_tc_port_init
On Wed, Oct 25, 2023 at 03:57:26PM +0200, Nirmoy Das wrote: > Fix below compiler warning: > > intel_tc.c:1879:11: error: ‘%d’ directive output may be truncated > writing between 1 and 11 bytes into a region of size 3 > [-Werror=format-truncation=] > "%c/TC#%d", port_name(port), tc_port + 1); >^~ > intel_tc.c:1878:2: note: ‘snprintf’ output between 7 and 17 bytes > into a destination of size 8 > snprintf(tc->port_name, sizeof(tc->port_name), > ^~ > "%c/TC#%d", port_name(port), tc_port + 1); > ~ > > Fixes: fe03f637b7bd ("drm/i915/tc: Fix -Wformat-truncation in > intel_tc_port_init") > intel_tc_port struct") > Cc: Mika Kahola > Cc: Imre Deak > Cc: Jani Nikula > Signed-off-by: Nirmoy Das > --- > drivers/gpu/drm/i915/display/intel_tc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_tc.c > b/drivers/gpu/drm/i915/display/intel_tc.c > index 37b0f8529b4f..0afcee279aff 100644 > --- a/drivers/gpu/drm/i915/display/intel_tc.c > +++ b/drivers/gpu/drm/i915/display/intel_tc.c > @@ -58,7 +58,7 @@ struct intel_tc_port { > struct delayed_work link_reset_work; > int link_refcount; > bool legacy_port:1; > - char port_name[8]; > + char port_name[17]; I'd use instead kasprintf() to suppress the warn. Imo (at one point) the 'Port %s:', tc->port_name prefix could be replaced by the usual [ENCODER:%d:%s]. > enum tc_port_mode mode; > enum tc_port_mode init_mode; > enum phy_fia phy_fia; > -- > 2.41.0 >