From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 2E2EF17B4FC; Tue, 11 Jun 2024 11:02:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718103736; cv=none; b=GjGvTXEPMaj2c+AL3yy62T3O1dS+DL5w+VPTy9CWk4+rpHYsEcNawmDdtJU5/hrwVRUY46IYcpMbzNaxiOq2tD65ZInFFVFX0tztF+iIpJT1gzYxRRcYzSnueqeY0FHXeSl29O2Mg4Wnk9sjzKK+7aPiEYAYiZqRxmJCFv/WkZo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718103736; c=relaxed/simple; bh=FkEqDypJsvCjoLDctfdedTpHjOn5E73IEktJ+zQG6f0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=bFHFUGmFl19nZsAFXN//d2xvaFabfFjcVcHn/NoK0Zw/MB0b2I9U+y5r67MZNNf01Le4VhlLuylXp8UUbp/GaOUQ6RVH0G7GIiWRCtZb5A3PxR1LbXiHzl7DjPnBlhTqY8PALTGfIE8z1XKRKkfPab6wC5Xqd0fgo94EYrvHvc0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 1696A1424; Tue, 11 Jun 2024 04:02:38 -0700 (PDT) Received: from [10.162.41.16] (e116581.arm.com [10.162.41.16]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id ED5093F64C; Tue, 11 Jun 2024 04:02:08 -0700 (PDT) Message-ID: Date: Tue, 11 Jun 2024 16:32:05 +0530 Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] selftests: Rename sigaltstack to generic signal To: Mark Brown Cc: shuah@kernel.org, oleg@redhat.com, stsp2@yandex.ru, mingo@kernel.org, tglx@linutronix.de, mark.rutland@arm.com, ryan.roberts@arm.com, suzuki.poulose@arm.com, Anshuman.Khandual@arm.com, DeepakKumar.Mishra@arm.com, AneeshKumar.KizhakeVeetil@arm.com, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org References: <20240611075650.814397-1-dev.jain@arm.com> <20240611075650.814397-2-dev.jain@arm.com> Content-Language: en-US From: Dev Jain In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 6/11/24 16:24, Mark Brown wrote: > On Tue, Jun 11, 2024 at 01:26:49PM +0530, Dev Jain wrote: >> Rename sigaltstack to signal, and rename the existing test to >> sigaltstack.c. >> >> Signed-off-by: Dev Jain >> --- > If people review patches and give a tag for them you should carry the > tag forward to avoid having to duplicate the review. From v1: > > I think this is reasonable if we're going to add more generic signal > tests - sigaltstack is a fairly small bit of functionality and having it > covered as part of a broader signal suite and the overhead of setting up > the suite separately is probably not worth it. > > Reviewed-by: Mark Brown Ah yes, I did know it, I forgot. Shall take care.