From: Greg Kroah-Hartman Date: Fri, 11 Feb 2022 11:59:45 +0000 (+0100) Subject: 5.10-stable patches X-Git-Tag: v4.9.302~56 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=28bdb52f52f3d57d8826ed1789b19bd7c490accd;p=thirdparty%2Fkernel%2Fstable-queue.git 5.10-stable patches added patches: tracing-propagate-is_signed-to-expression.patch --- diff --git a/queue-5.10/series b/queue-5.10/series index bc9d1713560..97056fdeed5 100644 --- a/queue-5.10/series +++ b/queue-5.10/series @@ -10,3 +10,4 @@ nfs-fix-initialisation-of-nfs_client-cl_flags-field.patch nfsd-clamp-write-offsets.patch nfsd-fix-offset-type-in-i-o-trace-points.patch drm-amdgpu-set-a-suitable-dev_info.gart_page_size.patch +tracing-propagate-is_signed-to-expression.patch diff --git a/queue-5.10/tracing-propagate-is_signed-to-expression.patch b/queue-5.10/tracing-propagate-is_signed-to-expression.patch new file mode 100644 index 00000000000..9a877048048 --- /dev/null +++ b/queue-5.10/tracing-propagate-is_signed-to-expression.patch @@ -0,0 +1,50 @@ +From 097f1eefedeab528cecbd35586dfe293853ffb17 Mon Sep 17 00:00:00 2001 +From: Tom Zanussi +Date: Thu, 27 Jan 2022 15:44:17 -0600 +Subject: tracing: Propagate is_signed to expression + +From: Tom Zanussi + +commit 097f1eefedeab528cecbd35586dfe293853ffb17 upstream. + +During expression parsing, a new expression field is created which +should inherit the properties of the operands, such as size and +is_signed. + +is_signed propagation was missing, causing spurious errors with signed +operands. Add it in parse_expr() and parse_unary() to fix the problem. + +Link: https://lkml.kernel.org/r/f4dac08742fd7a0920bf80a73c6c44042f5eaa40.1643319703.git.zanussi@kernel.org + +Cc: stable@vger.kernel.org +Fixes: 100719dcef447 ("tracing: Add simple expression support to hist triggers") +Reported-by: Yordan Karadzhov +BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=215513 +Signed-off-by: Tom Zanussi +Signed-off-by: Steven Rostedt (Google) +[sudip: adjust context] +Signed-off-by: Sudip Mukherjee +Signed-off-by: Greg Kroah-Hartman +--- + kernel/trace/trace_events_hist.c | 3 +++ + 1 file changed, 3 insertions(+) + +--- a/kernel/trace/trace_events_hist.c ++++ b/kernel/trace/trace_events_hist.c +@@ -2154,6 +2154,8 @@ static struct hist_field *parse_unary(st + (HIST_FIELD_FL_TIMESTAMP | HIST_FIELD_FL_TIMESTAMP_USECS); + expr->fn = hist_field_unary_minus; + expr->operands[0] = operand1; ++ expr->size = operand1->size; ++ expr->is_signed = operand1->is_signed; + expr->operator = FIELD_OP_UNARY_MINUS; + expr->name = expr_str(expr, 0); + expr->type = kstrdup(operand1->type, GFP_KERNEL); +@@ -2293,6 +2295,7 @@ static struct hist_field *parse_expr(str + + /* The operand sizes should be the same, so just pick one */ + expr->size = operand1->size; ++ expr->is_signed = operand1->is_signed; + + expr->operator = field_op; + expr->name = expr_str(expr, 0);