From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Martin Subject: [PATCH v4 05/28] arm64: fpsimd: Simplify uses of {set, clear}_ti_thread_flag() Date: Fri, 27 Oct 2017 11:50:47 +0100 Message-ID: <1509101470-7881-6-git-send-email-Dave.Martin@arm.com> References: <1509101470-7881-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: <1509101470-7881-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 , kvmarm@lists.cs.columbia.edu List-Id: linux-arch.vger.kernel.org VGhlIGV4aXN0aW5nIEZQU0lNRCBjb250ZXh0IHN3aXRjaCBjb2RlIGNvbnRhaW5zIGEgY291cGxl IG9mCmluc3RhbmNlcyBvZiB7c2V0LGNsZWFyfV90aV90aHJlYWQodGFza190aHJlYWRfaW5mbyh0 YXNrKSkuICBTaW5jZQp0aGVyZSBhcmUgdGhyZWFkIGZsYWcgbWFuaXB1bGF0b3JzIHRoYXQgb3Bl cmF0ZSBkaXJlY3RseSBvbgp0YXNrX3N0cnVjdCwgdGhpcyB2ZXJib3NpdHkgaXNuJ3Qgc3RyaWN0 bHkgbmVlZGVkLgoKRm9yIGNvbnNpc3RlbmN5LCB0aGlzIHBhdGNoIHNpbXBsaWZpZXMgdGhlIGFm ZmVjdGVkIGNhbGxzLiAgVGhpcwpzaG91bGQgaGF2ZSBubyBpbXBhY3Qgb24gYmVoYXZpb3VyLgoK U2lnbmVkLW9mZi1ieTogRGF2ZSBNYXJ0aW4gPERhdmUuTWFydGluQGFybS5jb20+ClJldmlld2Vk LWJ5OiBBbGV4IEJlbm7DqWUgPGFsZXguYmVubmVlQGxpbmFyby5vcmc+ClJldmlld2VkLWJ5OiBD YXRhbGluIE1hcmluYXMgPGNhdGFsaW4ubWFyaW5hc0Bhcm0uY29tPgpBY2tlZC1ieTogQXJkIEJp ZXNoZXV2ZWwgPGFyZC5iaWVzaGV1dmVsQGxpbmFyby5vcmc+Ci0tLQogYXJjaC9hcm02NC9rZXJu ZWwvZnBzaW1kLmMgfCA2ICsrLS0tLQogMSBmaWxlIGNoYW5nZWQsIDIgaW5zZXJ0aW9ucygrKSwg NCBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9hcmNoL2FybTY0L2tlcm5lbC9mcHNpbWQuYyBi L2FyY2gvYXJtNjQva2VybmVsL2Zwc2ltZC5jCmluZGV4IDVkNTQ3ZGUuLjdhODY1YzggMTAwNjQ0 Ci0tLSBhL2FyY2gvYXJtNjQva2VybmVsL2Zwc2ltZC5jCisrKyBiL2FyY2gvYXJtNjQva2VybmVs L2Zwc2ltZC5jCkBAIC0xNTksMTEgKzE1OSw5IEBAIHZvaWQgZnBzaW1kX3RocmVhZF9zd2l0Y2go c3RydWN0IHRhc2tfc3RydWN0ICpuZXh0KQogCiAJCWlmIChfX3RoaXNfY3B1X3JlYWQoZnBzaW1k X2xhc3Rfc3RhdGUpID09IHN0CiAJCSAgICAmJiBzdC0+Y3B1ID09IHNtcF9wcm9jZXNzb3JfaWQo KSkKLQkJCWNsZWFyX3RpX3RocmVhZF9mbGFnKHRhc2tfdGhyZWFkX2luZm8obmV4dCksCi0JCQkJ CSAgICAgVElGX0ZPUkVJR05fRlBTVEFURSk7CisJCQljbGVhcl90c2tfdGhyZWFkX2ZsYWcobmV4 dCwgVElGX0ZPUkVJR05fRlBTVEFURSk7CiAJCWVsc2UKLQkJCXNldF90aV90aHJlYWRfZmxhZyh0 YXNrX3RocmVhZF9pbmZvKG5leHQpLAotCQkJCQkgICBUSUZfRk9SRUlHTl9GUFNUQVRFKTsKKwkJ CXNldF90c2tfdGhyZWFkX2ZsYWcobmV4dCwgVElGX0ZPUkVJR05fRlBTVEFURSk7CiAJfQogfQog Ci0tIAoyLjEuNAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X18Ka3ZtYXJtIG1haWxpbmcgbGlzdAprdm1hcm1AbGlzdHMuY3MuY29sdW1iaWEuZWR1Cmh0dHBz Oi8vbGlzdHMuY3MuY29sdW1iaWEuZWR1L21haWxtYW4vbGlzdGluZm8va3ZtYXJtCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com ([217.140.101.70]:56744 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752311AbdJ0Kva (ORCPT ); Fri, 27 Oct 2017 06:51:30 -0400 From: Dave Martin Subject: [PATCH v4 05/28] arm64: fpsimd: Simplify uses of {set,clear}_ti_thread_flag() Date: Fri, 27 Oct 2017 11:50:47 +0100 Message-ID: <1509101470-7881-6-git-send-email-Dave.Martin@arm.com> In-Reply-To: <1509101470-7881-1-git-send-email-Dave.Martin@arm.com> References: <1509101470-7881-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 , Okamoto Takayuki , kvmarm@lists.cs.columbia.edu, libc-alpha@sourceware.org, linux-arch@vger.kernel.org Message-ID: <20171027105047.4YjQL9gSFYQmhIWGZ-UWGs-QACL2b2JYAsuSD8ANmF0@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 Reviewed-by: Catalin Marinas 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