Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <eab8177d-eb73-ec64-ec1f-4f2a51be8aee@intel.com>
Date:   Mon, 7 Mar 2022 12:56:08 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.1
Subject: Re: [RFC 08/10] platform/x86/intel/ifs: Add IFS sysfs interface
Content-Language: en-US
To:     Dan Williams <dan.j.williams@intel.com>
CC:     "hdegoede@redhat.com" <hdegoede@redhat.com>,
        "markgross@kernel.org" <markgross@kernel.org>,
        "corbet@lwn.net" <corbet@lwn.net>,
        "Raj, Ashok" <ashok.raj@intel.com>,
        "Luck, Tony" <tony.luck@intel.com>,
        "dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
        "patches@lists.linux.dev" <patches@lists.linux.dev>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "mingo@redhat.com" <mingo@redhat.com>,
        "rostedt@goodmis.org" <rostedt@goodmis.org>,
        "Shankar, Ravi V" <ravi.v.shankar@intel.com>,
        "tglx@linutronix.de" <tglx@linutronix.de>,
        "platform-driver-x86@vger.kernel.org" 
        <platform-driver-x86@vger.kernel.org>,
        "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
        "hpa@zytor.com" <hpa@zytor.com>, "bp@alien8.de" <bp@alien8.de>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "andriy.shevchenko@linux.intel.com" 
        <andriy.shevchenko@linux.intel.com>,
        "x86@kernel.org" <x86@kernel.org>
References: <20220301195457.21152-1-jithu.joseph@intel.com>
 <20220301195457.21152-9-jithu.joseph@intel.com>
 <188492dc80c017375da76d444347b1d00c2031f6.camel@intel.com>
 <7b9c788e-21dc-eedc-a1b4-9c6877fa48fe@intel.com>
 <CAPcyv4h=qPFrP+mRqaZhkh5ZmYjuQawsqvf+-R036ZJVKBNK4Q@mail.gmail.com>
 <33d0e764-86d9-8504-17fa-14b31c87de4e@intel.com>
 <CAPcyv4g5bq9+u0iLjhpeJw8bkbCREUw60H2z_KfDz4hHCrKdFQ@mail.gmail.com>
 <7c620f8a-189e-5ac4-30fe-1fa14ba799ea@intel.com>
 <CAPcyv4iUuZ0aAWckWvwbxJJgt5tDJRpeonfE4DegWS6KPtJq8A@mail.gmail.com>
From:   "Joseph, Jithu" <jithu.joseph@intel.com>
In-Reply-To: <CAPcyv4iUuZ0aAWckWvwbxJJgt5tDJRpeonfE4DegWS6KPtJq8A@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MXZQMzVZMzR3THdBTU9vck9Za0ttdkk1ZFB4eHVYNXdKUFBlZ0NVU2VvYzJi?=
 =?utf-8?B?OCtRcndHTW1uUTBaNXdkVXRvSVRVbDVHYm5TLzdkNW5RR0tPQWkvL2JueERo?=
 =?utf-8?B?aXRuUVVVS21oU3pyanU4YlpabEg1TUl6TUUyNGtMd2pDU2ltUENiUVNzR0g4?=
 =?utf-8?B?VkFWZEZJRHNQSjNJUVpiRkdoQlNmc0xKR05HNnpuQjIzSE5YYWx4YWNEQjE3?=
 =?utf-8?B?djhXRk5Sd2xIc3FhVW9FSDhqRDd1MUQ3NE9UeGpnendlMGFEY1hOWEVOMGFh?=
 =?utf-8?B?SXdEaStHZjZmaXJYWG9wRXprTXc3T1AydEUxVWowdXJLVVJxMVVLcFIxWXRy?=
 =?utf-8?B?MGROMkxZWHo0b3ZQbVhyRkZQVFFrdDFXUGFOaU1DbzUycVZiaS9BSXgrZVMy?=
 =?utf-8?B?cVNQbWJhaW9zMWFxZ3Q0SVd1QmNGWTJtQnVMbVVjbVpmSDFBZGtsdjh6NEwr?=
 =?utf-8?B?YjZZS0FOT3JPZ1ZmUU1ycnZRajYrd25wVTQ4aVZadFdnbCtiN0wvd1BlM2Nn?=
 =?utf-8?B?S202OUk2OTNWUkYvRjI2ZkRPNkFhbmo5TEdIVVErMUxUNXBkVXhjVWprREho?=
 =?utf-8?B?TGhVKzJ3NGpadWRRSG83ZnpacjVtR1VGVjUvUHpTUzJlakVHaTBESVdWOGIv?=
 =?utf-8?B?YUwrUGJjRENnWDFOYW5FQkRXMi83bTM3SGNXUllTbk1tQ0cxclM4T2xpKzRw?=
 =?utf-8?B?ME1YZWhoR0RDZU5EUTJuaVVkS2FISEZYaXN2a25RYmNXNU1SQTZHdWdBYXgr?=
 =?utf-8?B?MVlOZ2VDR1RZYURzWmMxdzVFa21hMlV6QmwxM0p5VWhnK0JWUlc5R3ZIUDNj?=
 =?utf-8?B?UUNJTWlTK1pOd1lRRGFhZ29NZGhzQ3U5dFg0bVYwNlFmZkd3anhVS0Qvd2s5?=
 =?utf-8?B?cWl4MGorNmhEclhzRjZPSGdXemtRNHRCQmpRTzROY3BIQnFPWlQ0RzFLYXJM?=
 =?utf-8?B?SVFLYk9jbWRiVlRubkNPcVkzQTUza1ArR3FZTm94eG9DS0c4RWNseWlwckpY?=
 =?utf-8?B?SUg0cmZLbnAvQWFzRmVpZkxkTkgvSHpqeEZqcG1leTZnVXZ1czRaOGhzaVMy?=
 =?utf-8?B?UEJEOVhVZkpuZEl0Z0kvTTN0WkdIanRTcTd5SlI5V096N29YYzh0ejBSUFVx?=
 =?utf-8?B?NG5OVGtHVnk4SGRMNUpUM2JDQmFzd1RFemU2cnhJUC9qWnYrZ2YrdVRZTDI2?=
 =?utf-8?B?MXBlMnNiNTF3c05vaVg2NWM4bDlsemhIbkc4blVTS1pHdkVWZFY0WlRjTFZQ?=
 =?utf-8?B?dmM3NnMzMVE2OC9xNTZmcDJsTEhCSWxPcDBNQnhMakFxVjZIZ3huSlcySzhi?=
 =?utf-8?B?NGJDekxVU0x5NlA0ZGg0SGVDSHF2N3FEby9nMTdGbm9MVTViYmtNNEV3NkJV?=
 =?utf-8?B?VmtLV3pXaFgwUitSaG4rS0hWeUVwMk1GSDhKSlREZDhEaEc3dzZzTTlIOGFJ?=
 =?utf-8?B?dUZtQm14a044VVlWNkhFSWJDWUgwZTJyRHpDNng1eElNdzR6R2xhNGJWRENt?=
 =?utf-8?B?Mk1KNGY4UlJPbEV5S05HVUdkNnN0NWdhYjFpVGZMN2F6cVVtSVBkbXNURlRH?=
 =?utf-8?B?MWhFQkkwa1pFZlRnQno5UVBBNEFUUzZpK2RCYkUrRlRxUnVna3hETE93MWhn?=
 =?utf-8?B?UjlORmptZWxxaXZoSHNXaVRsbThva0VEVGhMU21sWjU3aUN5d0NibWlCdlF6?=
 =?utf-8?B?ejMzck5OMWVnUDB4MjhzcktYalpxUlJaUUgrZFhabjc0YjlYUUlYNE5XQ0VH?=
 =?utf-8?B?R0J6c0VmM1NJZFVBRVZKQXowSUp5dmhLenZWOGxaN1Rib2wwMUZlcWlubnVn?=
 =?utf-8?B?Mi9ENHhBNDY4ZElJVUxMdlpld1MyVisrNGdYa05LUXZrZms5N1h1Y05hK1pu?=
 =?utf-8?B?N0FTQ0tTM3cvVFdqckFGY3FGcjNkTlFkNWpYdWFnVjlQeUlrYW0xWG1Tb1VS?=
 =?utf-8?B?S3JlZXRNYVRnY1NnYUpqbDczSERrek9TdmNHTGRWUXB3M2ZIQXpDRWo3WS84?=
 =?utf-8?B?TnVxdXRrcm43YXBFMnhwaWFJZ0c4TDcvMExDbFFuK0NNaEJVb3R5SE9nbS9U?=
 =?utf-8?B?aURDMGgrWU9VaGpiMHhYdCtPdHNRSGNpd0xLaEljNWphZVJoTWhEbndHSVlV?=
 =?utf-8?B?T1JvMVBFdTZTMHU0c2ZpSWhqNWJhN2FQYmxFUDEyK0RjRW5lZnIrRWhlQUVa?=
 =?utf-8?Q?A4lBqaDrQ33WRaekrjuCv1s=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 2991fbcb-aaa4-4ace-465f-08da007ce978
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3768.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Mar 2022 20:56:12.1920
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ukiGRtEVP9DpLju4NqvJKzzkbKWRgJ/61AG5w8I5EBYuHN5yyKhKBxvOAgxu6jBgv/Cnk9RYt3DTc6Cl9nucZw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR11MB1959
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 3/7/2022 12:25 PM, Dan Williams wrote:

> 
> I am speaking of the state of the case where 2 threads are doing
> run_test and polling for results. Unless you can guarantee that run2
> does not start before the results of run1 have been collected then
> they are lost in that scenario. No amount of kernel locking can
> resolve that race to collect previous result which would not be a
> problem in the first place if there was an atomic way to log test
> results.


Yes "status" shows the status of the latest run. You cannot get the status of the previous run.

Also some context on test frequency: Hardware has  strict rate limiting of tests.
Every core can be tested only once in every 30 minutes. So it is pointless to test at high frequency.