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 bc9d1713560f1afbc60a5ce7d82f541227b0ae79..97056fdeed57e3acc7b53d65451a8bc6d0d19d0a 100644 (file)
--- 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 (file)
index 0000000..9a87704
--- /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 <zanussi@kernel.org>
+Date: Thu, 27 Jan 2022 15:44:17 -0600
+Subject: tracing: Propagate is_signed to expression
+
+From: Tom Zanussi <zanussi@kernel.org>
+
+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 <ykaradzhov@vmware.com>
+BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=215513
+Signed-off-by: Tom Zanussi <zanussi@kernel.org>
+Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
+[sudip: adjust context]
+Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ 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);