From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 90974C43214 for ; Wed, 4 Aug 2021 19:48:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6F91761078 for ; Wed, 4 Aug 2021 19:48:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236829AbhHDTs6 (ORCPT ); Wed, 4 Aug 2021 15:48:58 -0400 Received: from out03.mta.xmission.com ([166.70.13.233]:46084 "EHLO out03.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234805AbhHDTsz (ORCPT ); Wed, 4 Aug 2021 15:48:55 -0400 Received: from in02.mta.xmission.com ([166.70.13.52]:38462) by out03.mta.xmission.com with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.93) (envelope-from ) id 1mBMsb-00G5UJ-3o; Wed, 04 Aug 2021 13:48:41 -0600 Received: from ip68-227-160-95.om.om.cox.net ([68.227.160.95]:43610 helo=email.xmission.com) by in02.mta.xmission.com with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.93) (envelope-from ) id 1mBMsa-00AESH-1k; Wed, 04 Aug 2021 13:48:40 -0600 From: ebiederm@xmission.com (Eric W. Biederman) To: Nathan Chancellor Cc: Sven Schnelle , Alexey Gladkov , linux-kernel@vger.kernel.org References: <20210730062854.3601635-1-svens@linux.ibm.com> Date: Wed, 04 Aug 2021 14:47:57 -0500 In-Reply-To: (Nathan Chancellor's message of "Tue, 3 Aug 2021 19:40:26 -0700") Message-ID: <875ywlat5e.fsf@disp2133> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-XM-SPF: eid=1mBMsa-00AESH-1k;;;mid=<875ywlat5e.fsf@disp2133>;;;hst=in02.mta.xmission.com;;;ip=68.227.160.95;;;frm=ebiederm@xmission.com;;;spf=neutral X-XM-AID: U2FsdGVkX19DF2GFSI/LRYKxEBZpK6iMObDDI8QO/9M= X-SA-Exim-Connect-IP: 68.227.160.95 X-SA-Exim-Mail-From: ebiederm@xmission.com Subject: Re: [PATCH v3] ucounts: add missing data type changes X-SA-Exim-Version: 4.2.1 (built Sat, 08 Feb 2020 21:53:50 +0000) X-SA-Exim-Scanned: Yes (on in02.mta.xmission.com) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Nathan Chancellor writes: > On Fri, Jul 30, 2021 at 08:28:54AM +0200, Sven Schnelle wrote: >> commit f9c82a4ea89c3 ("Increase size of ucounts to atomic_long_t") >> changed the data type of ucounts/ucounts_max to long, but missed to >> adjust a few other places. This is noticeable on big endian platforms >> from user space because the /proc/sys/user/max_*_names files all >> contain 0. >> >> Fixes: f9c82a4ea89c ("Increase size of ucounts to atomic_long_t") >> Signed-off-by: Sven Schnelle > > This patch in -next as commit e43fc41d1f7f ("ucounts: add missing data type > changes") causes Windows Subsystem for Linux to fail to start: > > [error 0x8007010b when launching `wsl.exe -d Arch'] Could not access starting > directory "\\wsl$\Arch\home\nathan" > > Specifically, it is the change to max_user_watches in > fs/notify/inotify/inotify_user.c, as the below diff gets me back to working. > Unfortunately, I have no additional information to offer beyond that as WSL's > init is custom and closed source (as far as I am aware) and there are no real > debugging utilities. Could you try this patch and tell us what value is being set? The only think I can imagine is that someone wants unlimited watches and sets the value to a ridiculously large value and the interpretation of that value winds up being different between int and long. This should allow you to read either dmesg or the kernel's log as it boots up and see what value is being written. From there it should be relatively straight forward to figure out what is going on. Thank you, Eric diff --git a/kernel/sysctl.c b/kernel/sysctl.c index 272f4a272f8c..733c4cfa1f60 100644 --- a/kernel/sysctl.c +++ b/kernel/sysctl.c @@ -975,6 +975,14 @@ int proc_dointvec_minmax(struct ctl_table *table, int write, .min = (int *) table->extra1, .max = (int *) table->extra2, }; +#if 1 + size_t lenv = *lenp; + if (write && (lenv > 0) && (lenv < INT_MAX)) { + int len = lenv; + printk(KERN_EMERG "intvec: %s <- %*.*s\n", + table->procname, len, len, (char *)buffer); + } +#endif return do_proc_dointvec(table, write, buffer, lenp, ppos, do_proc_dointvec_minmax_conv, ¶m); }