From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-io1-f43.google.com (mail-io1-f43.google.com [209.85.166.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C752A16DEAB for ; Fri, 5 Apr 2024 14:56:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712328995; cv=none; b=V4CaVEmzd/l+H/IUd6smA/CFPfM4z6MffNDcb3fV8WmaOiFYhVn9NmRzXjXPAlvu/W6mCoKa1XrFW76F+Nkkxo/dwto9kxOm1fQ+5Sk2Dc0mPBajhIIpsmLa6nzhtX9WsMcPUZ5O/KMpiYiXEGDOqGZOW0Au1mJTHjVN5w6sDCw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712328995; c=relaxed/simple; bh=zTMZhwQ+6oLIfkGA01wNSBzTksqHQBG/2caFP1/JlXY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=FQEXW44rljenQ7MoElFm1O9CSm8xvVfQJL5TZz7jB7VNm9gZKSvxdxv9er3WFMidFnyDEeD6csK2aaKO/AboKC0Sb8P0ysTTV0xZ2/MCqK4n3ht8fTbQYvw/n8Sn9vWDlwguVc/V0Nh8kRbT7vMoafGlEHUVg4T34A42ucGxmpc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linuxfoundation.org; spf=pass smtp.mailfrom=linuxfoundation.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=O7LpCr7H; arc=none smtp.client-ip=209.85.166.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linuxfoundation.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="O7LpCr7H" Received: by mail-io1-f43.google.com with SMTP id ca18e2360f4ac-7cc0e831e11so45833939f.1 for ; Fri, 05 Apr 2024 07:56:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; t=1712328993; x=1712933793; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=eVVvp+398iheOw5gutAtm6G0zLb/uxNorrWUv4KFnio=; b=O7LpCr7HWxXvkHzrM5aMANERAPz/6RjV/hyI1ZZ5SIzoDzonLylgFSBEBKgRkdQ1Zb qz4r8jtYdS5Lwj4JwYxW3egougdpAxVQQ0TsPxJyLPXS8womcmcLl4A6e3VbiLXNvhPT +WJ5CM6HaAWp+DASAIKSrYqHhZgEMpEzZ1f6A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712328993; x=1712933793; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=eVVvp+398iheOw5gutAtm6G0zLb/uxNorrWUv4KFnio=; b=ojHtV2D2ng8zlsaKFOQMzhLNfviVVbUoATVDeUgCf7bZ0uu1Kvc0trYdoQb4KFTXbm hc7kuut5lhr+XA0cjpxcCdZTVnoO/+tkhpMR2X8iitnMOnMPChHFdR4tTYu65m8aqPV6 SHcHD5wCRHfEvyjPjgQPKzr/hxp1KMlT1ZzQf44VR56Buf2u1dtTeGd1Krj/Ikn75J2v 86KIW1Gt6q8dcuG2uKypG1EsXx5ViJQHYo//bI/ZQBcetA6SeFCcb9XdK8ACsHWOaJ4z gCFxYU6Ya+T9sl32bhrTW738PpEBqIPYXpqqCKm38GBkhwPF0ThtKTJ5iR4BnqMLNN8W ihcQ== X-Forwarded-Encrypted: i=1; AJvYcCXM1agMbictF0ow6HZw/WpKwSe7HQ9DKmqJNmO5DJe/9cjXEbJ9//hu+BV4A94+DmBDX+rLw47no3KhAqZsCPoMPbg0wtrA79jGb1Jr X-Gm-Message-State: AOJu0YxWpyh5XREt8LsBRgADnEwwMdI9fP5YCJf7TBGdbmXr+0/e0lgr I1vNch0LScTvIRn5SyKbpJs0T00VEqzwSrxLq/iegJTd+aV+oiQKTvrPF7OLL94= X-Google-Smtp-Source: AGHT+IEkdy1rdEG+rjc30ywKaQ9qlK6/njmx8r/DcDzK1jQaFuF4ReeRun+a1EcXGR4eE0eknNVLJw== X-Received: by 2002:a5e:c648:0:b0:7d3:4ff8:7bb1 with SMTP id s8-20020a5ec648000000b007d34ff87bb1mr1694674ioo.1.1712328992992; Fri, 05 Apr 2024 07:56:32 -0700 (PDT) Received: from [192.168.1.128] ([38.175.170.29]) by smtp.gmail.com with ESMTPSA id j4-20020a05663822c400b0047ad636c705sm601613jat.73.2024.04.05.07.56.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 05 Apr 2024 07:56:32 -0700 (PDT) Message-ID: <2a77adeb-ed22-4a9b-a1d9-ac5d29ae696d@linuxfoundation.org> Date: Fri, 5 Apr 2024 08:56:31 -0600 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 0/2] selftests: Replace "Bail out" with "Error" in ksft_exit_fail_msg() To: Muhammad Usama Anjum , Eric Biederman , Kees Cook , Shuah Khan , Andrew Morton , linux-mm@kvack.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kernel@collabora.com, Shuah Khan References: <20240405131748.1582646-1-usama.anjum@collabora.com> Content-Language: en-US From: Shuah Khan In-Reply-To: <20240405131748.1582646-1-usama.anjum@collabora.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 4/5/24 07:17, Muhammad Usama Anjum wrote: > "Bail out! " is not descriptive. It rather should be: "Failed: " and > then this added prefix doesn't need to be added everywhere. Usually in > the logs, we are searching for "Failed" or "Error" instead of "Bail > out" so it must be replace. > > Remove Error/Failed prefixes from all usages as well. > > Muhammad Usama Anjum (2): > selftests: Replace "Bail out" with "Error" > selftests: Remove Error/Failed prefix from ksft_exit_fail*() usages > > tools/testing/selftests/exec/load_address.c | 8 +- > .../testing/selftests/exec/recursion-depth.c | 10 +- > tools/testing/selftests/kselftest.h | 2 +- > .../selftests/mm/map_fixed_noreplace.c | 24 +-- > tools/testing/selftests/mm/map_populate.c | 2 +- > tools/testing/selftests/mm/mremap_dontunmap.c | 2 +- > tools/testing/selftests/mm/pagemap_ioctl.c | 166 +++++++++--------- > .../selftests/mm/split_huge_page_test.c | 2 +- > 8 files changed, 108 insertions(+), 108 deletions(-) > Andrew, Kees, I will apply these to linux-kselftest next as a series since these changes depend on change to tools/testing/selftests/kselftest.h and need to go together. Are you okay with that? thanks, -- Shuah