Re: [PATCH 2/2] selftests/mm: Do not fail test for a single migration failure

2024-08-11 Thread Dev Jain
On 8/9/24 22:43, Shuah Khan wrote: On 8/9/24 04:31, Dev Jain wrote: Do not fail the test for just a single instance of migration failure, since migration is a best-effort service. The cover letter says: "Given that migration is a best-effort service, it is wrong to fail the test for just a

Re: [PATCH 2/2] selftests/mm: Do not fail test for a single migration failure

2024-08-09 Thread Christoph Lameter (Ampere)
On Fri, 9 Aug 2024, Shuah Khan wrote: "Given that migration is a best-effort service, it is wrong to fail the test for just a single failure; hence, fail the test after 100 consecutive failures (where 100 is still a subjective choice)." You do want to mention the above here. The reason being,

Re: [PATCH 2/2] selftests/mm: Do not fail test for a single migration failure

2024-08-09 Thread Shuah Khan
On 8/9/24 04:31, Dev Jain wrote: Do not fail the test for just a single instance of migration failure, since migration is a best-effort service. The cover letter says: "Given that migration is a best-effort service, it is wrong to fail the test for just a single failure; hence, fail the test a

[PATCH 2/2] selftests/mm: Do not fail test for a single migration failure

2024-08-09 Thread Dev Jain
Do not fail the test for just a single instance of migration failure, since migration is a best-effort service. Signed-off-by: Dev Jain Suggested-by: David Hildenbrand Reviewed-by: Ryan Roberts Tested-by: Ryan Roberts --- tools/testing/selftests/mm/migration.c | 17 +++-- 1 file c