Message-ID: <74a17cb6-9dcd-749b-5d06-b2c364d36ae2@oracle.com>
Date:   Sat, 11 Mar 2023 10:11:26 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.8.0
Subject: Re: [PATCH 03/11] kthread: Pass in the thread's name during creation
To:     Christian Brauner <brauner@kernel.org>
Cc:     hch@infradead.org, stefanha@redhat.com, jasowang@redhat.com,
        mst@redhat.com, sgarzare@redhat.com,
        virtualization@lists.linux-foundation.org, ebiederm@xmission.com,
        torvalds@linux-foundation.org, konrad.wilk@oracle.com,
        linux-kernel@vger.kernel.org
References: <20230310220332.5309-1-michael.christie@oracle.com>
 <20230310220332.5309-4-michael.christie@oracle.com>
 <20230311085326.m7aub5duy76hnnam@wittgenstein>
Content-Language: en-US
From:   michael.christie@oracle.com
In-Reply-To: <20230311085326.m7aub5duy76hnnam@wittgenstein>
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?d0k3WVlsOG5yOFhweFJ4MVA4MnZmbWRiVXdZR3daU2JyRTZZVjZNVzdOZU10?=
 =?utf-8?B?SHVHVjFoSFIvemlhK0h0ajE2MXVUMkgxSnpFSnZZZW5LTEovT1E3Z2tLSGF1?=
 =?utf-8?B?UHg5N3ZZMUdmTDRXaFJSUU8xUUNBemdteG9lTXhRVTFkVHhDQjg2QTJuNGlB?=
 =?utf-8?B?WjNEeHJudzhESS92Vkd0YnJPQUNIYmdyemxQQ3cvaGdjNlhKQ1k3aUZyZ2tM?=
 =?utf-8?B?c0VKTW80TjdMa1prOWNQZEJBeVZ4N01VemgrY3d3K0ducGRjR01pUUFOY3NM?=
 =?utf-8?B?VTBkcXZxYzJUb2R6bGp6V1VEQmUzREpldFk2MmFBaTQ5cTlMQ3ZKdWs2cHFy?=
 =?utf-8?B?NzNLVW0yYTJoclVBdE50ZlRIODd1UDJGZVVZU2FsZ0RrM0c0cnV2MjNEdThK?=
 =?utf-8?B?SzBXcE14Y082R2t0TGRmTlFaWG01OEZmYXQ0VExnbDRHZEkzVjJnUjArMjJX?=
 =?utf-8?B?MDJtTGppRnhERkxiY1F5dE9vdUZOMS9yY3IzTXZQL0htZCtRdmU4T3VXOTNV?=
 =?utf-8?B?M1Zaa1d3LzJXeUE5VSt1Zkk0TnBGaEZoNUZqL2kzaS82VE1TYUpBT3M3MGdR?=
 =?utf-8?B?QUlFcXJ0akdwZ2Z4UkkzU3o5aGg5MnNaUlVTaElGZUpyemQreFBKYUcxbk0w?=
 =?utf-8?B?bWFrQStScmU0T2owQ3BTbU5lMkpmQmhEcE1FRXhQRjdrZjlPak82T2pFaU8v?=
 =?utf-8?B?QUQxUU8vQnRVaTM5cnlJRU9sT0JMMHpCWHpPdTZkUjJmeC9OeUV2ZUUzK3ly?=
 =?utf-8?B?MjR1dnBVRkpTSUlMQ0x4emNoRFh6N2lPOFZXMWxZK0lKL0FkK2RmS3JjZkRq?=
 =?utf-8?B?NU9xWU1Xb0hySjJrYXE4enMyVXBGYlk5a3VVdEYrMTZEWm9UMnBYU2FOTE5Z?=
 =?utf-8?B?eFFrYUJ4Z3ZQRzR6YS8zSElzL1UrSFdGV1U0UzNDKzM3MWNuQTJOLzVZY3R3?=
 =?utf-8?B?UzJyeHRqQ0VnV0t5ZWZpS09rbnQvRXFqQnFwODlBcGlYMDFUQVFwRDJCTWt1?=
 =?utf-8?B?OFVCL3BsVVJXNHZkU3NCSnoxVkRUcm00eHJMdm1uWml6N1BKUEhUZlpGS2dP?=
 =?utf-8?B?NDBaU1hyY0d5VURDSVc0YjlTNnlqRXUwbE94elhUQ09MOFllaWs1NHNHZkhy?=
 =?utf-8?B?cGQvOTczWVh2SlRrZXB2cVQvb0MwUDlUUGJCRWRSS3ordjBUVzZQR0dSMDll?=
 =?utf-8?B?NFdpUGIyazlHVE43VHQwSmxCVCtseEpOZDhUdDNLZ2d3cmpJWGZ6VVNyeXVo?=
 =?utf-8?B?V0tVRkNJeFEwNExQamdHYVJSNE96OUNsaHVpZUpVVjF5WDdxRHQwWUN5MXls?=
 =?utf-8?B?WWVhVHZ3enZ1OE9ucjVVL1gzSW9veXBxSnkzbGRnRk5YQmhtT3d3M2ZGelhz?=
 =?utf-8?B?U3B5dDgwYWJHbEd6bGZGSkxPVGVjZkVnNGtrYk5ldTNLY0l5V0cvR0kwK2M1?=
 =?utf-8?B?enpGV2xwNUpBa0JuOFR2VktsTU13ME15TlNZWEZDTnR5ZzlUREtLL1pabW1S?=
 =?utf-8?B?ZE5lS05VaHZYSUlnbVc1dHlValpXZ2JzamgvL1dsQlhUQnNvMm1JT0RTWG1S?=
 =?utf-8?B?aHZqV3BWTVYrOWRPRVlZY3NVbDgyTzdKenplYlZwQ3Q2aTdyRHROMjRwTFJa?=
 =?utf-8?B?d29Pd0Z6VFdoVUo3L21KNVptNFprY1kzOWtjRk50eFJsZm83eS9lb2s5cTNm?=
 =?utf-8?B?RnBzR3ZsZzkyQnNEajg2R0hqY1NVZnJLeENTYm9SaFpVUHhqMUc4ampoVFF1?=
 =?utf-8?B?N3JiTjdDTWY4b2d5N1pPMlhFUHh0TU83ODZZY0lxNUJJOHVadDA3cFVORmR5?=
 =?utf-8?B?VjdaMktJUVMrZEY4dE5mZ2xtcFRLQWl5WXdsRytxVGFLU21nd0lVTHVlTmZu?=
 =?utf-8?B?bnBZeWdsUWFUUlUrUEk5QmxEcm1NUzhjc0hoQzNTMjVZQWdOZXQyZmhCejRr?=
 =?utf-8?B?ajByQzExTHRUMC8zc3pRVk5jUW1Qc2w3dkI1MVg2bzJ3RjIwUVpCY0ZCQkQ5?=
 =?utf-8?B?THZadGRRWTd4WVk0SkFpcC9EV3FwL2VoT3REYURWcDJvNUlkTk9IK2hKU3JU?=
 =?utf-8?B?UGIzbENkMTRjVWdXTnlSVXJjSjlxL055ZkUrTG9sRTk1OUdId0NDd2dGaVBC?=
 =?utf-8?B?U0xMNHNzdDk4RStjRHUwNnhnbVY1NFpCelBQWlk4YXZUYVd3ZGY4bGcxTEhy?=
 =?utf-8?B?eEE9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?utf-8?B?QUZ4Q3haNkpLWkdvQzd3aWVJR21vTDNkNG9UMEQwQlRzWXQ3N1NRbWdibzFI?=
 =?utf-8?B?cFJxME85bGNZb0pSNmRhbk4vWHZLb1FRTjNwUkNWQ0dJNzhoRjBnYmZEelVr?=
 =?utf-8?B?bHNCVzFuV3gyczc2RXFiN25CNURBWHJUdHhIWStVY0tOQVh5eXZVWUFSaUJJ?=
 =?utf-8?B?RU8zdU9hK01nMFpCRnhOSzU0ZW1LNGIvcUFxc044eWhORmt5VkxiYTRRZWJ0?=
 =?utf-8?B?bUpkTENjNGJ5M1BxejhDSVJBbGtjeklKQnFCS2Z2V1IyQmNzYjMwdFRpdUN0?=
 =?utf-8?B?OXNLWTlVeGQvR2hUenVmbklkWnJDc0hKV2tGcWNPZ1g4QXE0Y0xucEhFUDJQ?=
 =?utf-8?B?djlxd3pqVEkwQks0UHUwSHNJeVdwN1YwQStrQ29BL0Fram5BZGY3Wk1aTE5i?=
 =?utf-8?B?SlBibDJBL0RuL21FWDVWMlhTZ3l5VHlxcDVZWnZveUtObm9aazkxTFBZeUV6?=
 =?utf-8?B?Vkpic3Z4YzBsb2Q2QTZLTUJkaU8yOTk5dW5lNCs3a1VPSWo4ZXQ0bDdueC9D?=
 =?utf-8?B?dFVlVlgzNkdOOUxtQ1RzMnpsckpFcGVSdXlEUU9kSzdudDR4Q3RmUW9Fb29R?=
 =?utf-8?B?VTJ6SFppNjl1N3c4ZklGeXhJWjhqVzYwdjBuUlF1YjRWNVl6UG4raFgwQ3BP?=
 =?utf-8?B?d3BkaStubmQrSkJVRkJ2Q2JYSFZtTkZBRmY1NmIrR0FBeXcxOGNueDJqSUd3?=
 =?utf-8?B?VUVoMFpHOFpkRlgwN1IwcW1aOU1QaGlONnhEOG5TMEJPV0tPMTFTN3lLU2Nz?=
 =?utf-8?B?N21pc2ZXNnBFcStYeEpHWndqK2c3dUpOSE5IWjA4UUpKOEgyKzJrcit2Zzh3?=
 =?utf-8?B?K25zdzFYQ1VMa2JvUnpkM0ozUTVmVi81RVNIb0Z3USs0b1hRenUraVV6V3VR?=
 =?utf-8?B?a01YT1FaMjZDa29ZTkpza2d6eGFQdS90MndQUnh5SnovbjBHTzhoM2NBNFN4?=
 =?utf-8?B?YWFFK2dzYU9GZEpKREE0MzJpYW5URmxMRHR2OWdVcnhVZGpWSUFvcTVXY1lW?=
 =?utf-8?B?ejRjbVpxN0h6MTRMWXVGbHZ0elkwNWxyNFhHenFWd1pUamZtUzJBV3U1TjYw?=
 =?utf-8?B?QmIwU0dsY0Y5eVJRZUMvQXBQOUdrREU0SmJyckovdUlKTEN6NFozNEZlSVcw?=
 =?utf-8?B?M2JsTTlwaVhidjRrRm4zdXNQVXpaZVpaVGR5VkptUnFCVkxOcUp2YmdHbEJs?=
 =?utf-8?B?cmRDSlhXc3VXeDFyUDlCa2VsSnFhZjhRMHRTRmwyQXkzVGZVRE94Z0sweTRR?=
 =?utf-8?B?cHBPS29uTFFXOGpJazBnTVBtRXE3cEVhKzhubzI5M0MwQWZSbG5mYjZZMDls?=
 =?utf-8?B?QmtkTnd1V1ZuUHhqbzlRWjFud3V2MS9EcjRaWG9JdldMcEIvQllkcXVEVTlk?=
 =?utf-8?B?aEJvVVo1eThNb2c9PQ==?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 89be88ee-70a8-4125-f97d-08db224b4599
X-MS-Exchange-CrossTenant-AuthSource: DM5PR10MB1466.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Mar 2023 16:11:29.1011
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: PEW4tYvzumqPKe1YIBFZWpls3nd+nfumqv64MNhBax3XixT3b2GwUWHPldjAWbibRPY5xxx+TEFuywlVXBEI02kGcSnHEP0J93e8MjJkqRY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR10MB7412
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22
 definitions=2023-03-11_04,2023-03-10_01,2023-02-09_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=999 phishscore=0
 suspectscore=0 spamscore=0 malwarescore=0 adultscore=0 mlxscore=0
 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2212070000 definitions=main-2303110146
X-Proofpoint-GUID: 0-tlBm7_auJbzo5ionK4l_WjflC8wIiy
X-Proofpoint-ORIG-GUID: 0-tlBm7_auJbzo5ionK4l_WjflC8wIiy
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 3/11/23 2:53 AM, Christian Brauner wrote:
> On Fri, Mar 10, 2023 at 04:03:24PM -0600, Mike Christie wrote:
>> This has us pass in the thread's name during creation in kernel_thread.
>>
>> Signed-off-by: Mike Christie <michael.christie@oracle.com>
>> ---
>>  kernel/kthread.c | 35 ++++++++++++++---------------------
>>  1 file changed, 14 insertions(+), 21 deletions(-)
>>
>> diff --git a/kernel/kthread.c b/kernel/kthread.c
>> index 63574cee925e..831a55b406d8 100644
>> --- a/kernel/kthread.c
>> +++ b/kernel/kthread.c
>> @@ -38,6 +38,7 @@ struct task_struct *kthreadd_task;
>>  struct kthread_create_info
>>  {
>>  	/* Information passed to kthread() from kthreadd. */
>> +	char *full_name;
>>  	int (*threadfn)(void *data);
>>  	void *data;
>>  	int node;
>> @@ -343,10 +344,15 @@ static int kthread(void *_create)
>>  	/* Release the structure when caller killed by a fatal signal. */
>>  	done = xchg(&create->done, NULL);
>>  	if (!done) {
>> +		kfree(create->full_name);
>>  		kfree(create);
>>  		kthread_exit(-EINTR);
>>  	}
>>  
>> +	if (strlen(create->full_name) >= TASK_COMM_LEN)
>> +		self->full_name = create->full_name;
>> +	else
>> +		kfree(create->full_name);
> 
> This is monir but wwiw, this looks suspicious when reading it without
> more context. It took me a while to see that kthread->full_name is
> intended to store the untruncated name only if truncation actually needs
> to happen. So either we should always initialize this or we should add a
> comment. You can just send a tiny patch that I can fold into this one so
> you don't have to resend the whole series...
Ok. Thanks. I think always initializing it would make the code a lot easier
to understand and be nicer.

I don't think it would be too much extra memory used, and we don't have
to worry about some random userspace app breaking because it wanted to
configure the thread but the name got truncated.