From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Martin Subject: [PATCH v3 05/28] arm64: fpsimd: Simplify uses of {set, clear}_ti_thread_flag() Date: Tue, 10 Oct 2017 19:38:22 +0100 Message-ID: <1507660725-7986-6-git-send-email-Dave.Martin@arm.com> References: <1507660725-7986-1-git-send-email-Dave.Martin@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1507660725-7986-1-git-send-email-Dave.Martin@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu To: linux-arm-kernel@lists.infradead.org Cc: linux-arch@vger.kernel.org, Okamoto Takayuki , libc-alpha@sourceware.org, Ard Biesheuvel , Szabolcs Nagy , Catalin Marinas , Will Deacon , Richard Sandiford , kvmarm@lists.cs.columbia.edu List-Id: linux-arch.vger.kernel.org VGhlIGV4aXN0aW5nIEZQU0lNRCBjb250ZXh0IHN3aXRjaCBjb2RlIGNvbnRhaW5zIGEgY291cGxl IG9mCmluc3RhbmNlcyBvZiB7c2V0LGNsZWFyfV90aV90aHJlYWQodGFza190aHJlYWRfaW5mbyh0 YXNrKSkuICBTaW5jZQp0aGVyZSBhcmUgdGhyZWFkIGZsYWcgbWFuaXB1bGF0b3JzIHRoYXQgb3Bl cmF0ZSBkaXJlY3RseSBvbgp0YXNrX3N0cnVjdCwgdGhpcyB2ZXJib3NpdHkgaXNuJ3Qgc3RyaWN0 bHkgbmVlZGVkLgoKRm9yIGNvbnNpc3RlbmN5LCB0aGlzIHBhdGNoIHNpbXBsaWZpZXMgdGhlIGFm ZmVjdGVkIGNhbGxzLiAgVGhpcwpzaG91bGQgaGF2ZSBubyBpbXBhY3Qgb24gYmVoYXZpb3VyLgoK U2lnbmVkLW9mZi1ieTogRGF2ZSBNYXJ0aW4gPERhdmUuTWFydGluQGFybS5jb20+ClJldmlld2Vk LWJ5OiBBbGV4IEJlbm7DqWUgPGFsZXguYmVubmVlQGxpbmFyby5vcmc+CkFja2VkLWJ5OiBBcmQg Qmllc2hldXZlbCA8YXJkLmJpZXNoZXV2ZWxAbGluYXJvLm9yZz4KLS0tCiBhcmNoL2FybTY0L2tl cm5lbC9mcHNpbWQuYyB8IDYgKystLS0tCiAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCsp LCA0IGRlbGV0aW9ucygtKQoKZGlmZiAtLWdpdCBhL2FyY2gvYXJtNjQva2VybmVsL2Zwc2ltZC5j IGIvYXJjaC9hcm02NC9rZXJuZWwvZnBzaW1kLmMKaW5kZXggNWQ1NDdkZS4uN2E4NjVjOCAxMDA2 NDQKLS0tIGEvYXJjaC9hcm02NC9rZXJuZWwvZnBzaW1kLmMKKysrIGIvYXJjaC9hcm02NC9rZXJu ZWwvZnBzaW1kLmMKQEAgLTE1OSwxMSArMTU5LDkgQEAgdm9pZCBmcHNpbWRfdGhyZWFkX3N3aXRj aChzdHJ1Y3QgdGFza19zdHJ1Y3QgKm5leHQpCiAKIAkJaWYgKF9fdGhpc19jcHVfcmVhZChmcHNp bWRfbGFzdF9zdGF0ZSkgPT0gc3QKIAkJICAgICYmIHN0LT5jcHUgPT0gc21wX3Byb2Nlc3Nvcl9p ZCgpKQotCQkJY2xlYXJfdGlfdGhyZWFkX2ZsYWcodGFza190aHJlYWRfaW5mbyhuZXh0KSwKLQkJ CQkJICAgICBUSUZfRk9SRUlHTl9GUFNUQVRFKTsKKwkJCWNsZWFyX3Rza190aHJlYWRfZmxhZyhu ZXh0LCBUSUZfRk9SRUlHTl9GUFNUQVRFKTsKIAkJZWxzZQotCQkJc2V0X3RpX3RocmVhZF9mbGFn KHRhc2tfdGhyZWFkX2luZm8obmV4dCksCi0JCQkJCSAgIFRJRl9GT1JFSUdOX0ZQU1RBVEUpOwor CQkJc2V0X3Rza190aHJlYWRfZmxhZyhuZXh0LCBUSUZfRk9SRUlHTl9GUFNUQVRFKTsKIAl9CiB9 CiAKLS0gCjIuMS40CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwprdm1hcm0gbWFpbGluZyBsaXN0Cmt2bWFybUBsaXN0cy5jcy5jb2x1bWJpYS5lZHUKaHR0 cHM6Ly9saXN0cy5jcy5jb2x1bWJpYS5lZHUvbWFpbG1hbi9saXN0aW5mby9rdm1hcm0K From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com ([217.140.101.70]:48846 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932232AbdJJSjG (ORCPT ); Tue, 10 Oct 2017 14:39:06 -0400 From: Dave Martin Subject: [PATCH v3 05/28] arm64: fpsimd: Simplify uses of {set,clear}_ti_thread_flag() Date: Tue, 10 Oct 2017 19:38:22 +0100 Message-ID: <1507660725-7986-6-git-send-email-Dave.Martin@arm.com> In-Reply-To: <1507660725-7986-1-git-send-email-Dave.Martin@arm.com> References: <1507660725-7986-1-git-send-email-Dave.Martin@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: linux-arm-kernel@lists.infradead.org Cc: Catalin Marinas , Will Deacon , Ard Biesheuvel , =?UTF-8?q?Alex=20Benn=C3=A9e?= , Szabolcs Nagy , Richard Sandiford , Okamoto Takayuki , kvmarm@lists.cs.columbia.edu, libc-alpha@sourceware.org, linux-arch@vger.kernel.org Message-ID: <20171010183822.AtSD0EyzjRBnocYABh8PlPOezF8Ag1euInJcReEhumQ@z> The existing FPSIMD context switch code contains a couple of instances of {set,clear}_ti_thread(task_thread_info(task)). Since there are thread flag manipulators that operate directly on task_struct, this verbosity isn't strictly needed. For consistency, this patch simplifies the affected calls. This should have no impact on behaviour. Signed-off-by: Dave Martin Reviewed-by: Alex Bennée Acked-by: Ard Biesheuvel --- arch/arm64/kernel/fpsimd.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/arm64/kernel/fpsimd.c b/arch/arm64/kernel/fpsimd.c index 5d547de..7a865c8 100644 --- a/arch/arm64/kernel/fpsimd.c +++ b/arch/arm64/kernel/fpsimd.c @@ -159,11 +159,9 @@ void fpsimd_thread_switch(struct task_struct *next) if (__this_cpu_read(fpsimd_last_state) == st && st->cpu == smp_processor_id()) - clear_ti_thread_flag(task_thread_info(next), - TIF_FOREIGN_FPSTATE); + clear_tsk_thread_flag(next, TIF_FOREIGN_FPSTATE); else - set_ti_thread_flag(task_thread_info(next), - TIF_FOREIGN_FPSTATE); + set_tsk_thread_flag(next, TIF_FOREIGN_FPSTATE); } } -- 2.1.4