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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2E17BD743D9 for ; Wed, 20 Nov 2024 21:12:18 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id ADC006B0096; Wed, 20 Nov 2024 16:12:17 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id A8C046B009B; Wed, 20 Nov 2024 16:12:17 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 92CE26B009C; Wed, 20 Nov 2024 16:12:17 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 704F76B0096 for ; Wed, 20 Nov 2024 16:12:17 -0500 (EST) Received: from smtpin25.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id F32D214038F for ; Wed, 20 Nov 2024 21:12:16 +0000 (UTC) X-FDA: 82807719498.25.1784DFB Received: from mail-io1-f44.google.com (mail-io1-f44.google.com [209.85.166.44]) by imf21.hostedemail.com (Postfix) with ESMTP id 33B661C0006 for ; Wed, 20 Nov 2024 21:10:36 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=google header.b=CvpnfZni; dmarc=pass (policy=none) header.from=linuxfoundation.org; spf=pass (imf21.hostedemail.com: domain of skhan@linuxfoundation.org designates 209.85.166.44 as permitted sender) smtp.mailfrom=skhan@linuxfoundation.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1732136999; a=rsa-sha256; cv=none; b=uQQI+SMlBb7A/ae2L/sB05i8fHiNAPRMGGfWojRfz3bfnMTFEl3iFCnRqcV95YmmRI/JHU w0Cqenemfp7dajrPQJC7RC25Pi1sg3RxjJkLLMvwLc/Z1QpGtHFAW02UilqRZbdfo7ttiL Zd/byxbSFxmIrIfO2OFnm2F/sbbXCrI= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=google header.b=CvpnfZni; dmarc=pass (policy=none) header.from=linuxfoundation.org; spf=pass (imf21.hostedemail.com: domain of skhan@linuxfoundation.org designates 209.85.166.44 as permitted sender) smtp.mailfrom=skhan@linuxfoundation.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1732136999; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=pwRE5aWtMzLuuFheQxhwI6BjKxK09M8wwvrm4cZ/NdI=; b=j1E4TZd8P7xtbiMfjb7ycoJF8Gf9ahAnIXzF7yGnafqBKaIngqectPIFCDN1o1+m+6XM3r E3VJx2v5O8ymhWdYU/9YoDfq8Tff1d4cdVQGmOXS7f1yCf1AI7sVCaey1E4r3AlhB7eGbc ie90JdW2QTc+cjBEMAFJJ9teZH94uQ8= Received: by mail-io1-f44.google.com with SMTP id ca18e2360f4ac-83b430a4cfdso8381539f.2 for ; Wed, 20 Nov 2024 13:12:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxfoundation.org; s=google; t=1732137134; x=1732741934; darn=kvack.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=pwRE5aWtMzLuuFheQxhwI6BjKxK09M8wwvrm4cZ/NdI=; b=CvpnfZniQrPQ8c9MSCIYCXpgxqv9bwBJ9dch1y3VkqjQ0dtFqU9Cjjx6XdRwmfalEk 3eSF83wmmN7r0jSM0w5q8owFzkS4yTow0FoElpPrNfi5noFRS69iLqkvM0Y/WPo7ycKa R/hwjyCrPOxvpS96Y7IZ80pZygSZtOMN1r9f4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732137134; x=1732741934; 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=pwRE5aWtMzLuuFheQxhwI6BjKxK09M8wwvrm4cZ/NdI=; b=A5iNJD8MkrKHfVytgc9GhI+m33tv3DjrSAERUm0fe9oJ9hi2fILgkFhoCPmdVQvo0G zuytjDaZ1l9WWmmAF/den+lbxrtn/3sJvhW/IXfnFkHMWaTD/51IszCyWhBrYZiCT/WL Wae8Zkep9M4GQbNv5SR7YED+zJKDLYMUdMMpMxHuiaqr8Uc5k9/udbfV5dQykbn4yU7q hXRDkHRdc1hsILga9Ul4q09qWC3XCuesc8wQ/WliUSVMQiHIa6AcaMfJjypR11Mq9IWp 7edSLpGoWkfmpLx9BzDUGCU/0122v6xIg0TSQ/cGVztQoernwE0D8bfvI+dSEihQ/ox1 8toA== X-Forwarded-Encrypted: i=1; AJvYcCWHTxaX8lYUNmpYBv77xVpdJKxhNk1Mrw/3LvCiIoLoMdzO8+fGZ+GLBRTy23OrtKFYNIvYRJA2AQ==@kvack.org X-Gm-Message-State: AOJu0YzS2Muz8OOnuj5P1Xfq7VKvgpAigo7sfKFHITwLVUzqw0nWKZ+a UnTojXiPk9bw8P9tsleszqBRT8jkJooOCgTYxdpLtjfg0kSfcKbgBOeM8qsVgKo= X-Gm-Gg: ASbGncuuEuWn8B8Zl9wmB0pxQEyWHbXLQkG5kxidcplrhzhKL9DGGvuK1i3oHHyqJ8L hGORK5V0cX+b+R84LyY0lo4DR/zP5gNtAwfMnjLuPNrkh+bfEMvMSDyBDE3WB1lbEG3X7dwPqA5 o/eNiUAng+sM2mKLIujFM1lc9sZey4byahbG61GBTvnHL9acm1TuslWHXYmjX/Z8AppEyxV3VqK YUGPs/n3VkZpFQj4iZTHE6lZiIJWYaxt526F2FIOCl7ZYpvv0CDahY6IgEbhA== X-Google-Smtp-Source: AGHT+IGF0X7D1su9KBsDu7YgP31JToa/OjzK2+lpjEy03i6Xz35JeAk4n1thkDF51vnmohZStzHzvA== X-Received: by 2002:a05:6602:2b86:b0:835:45f9:c2ee with SMTP id ca18e2360f4ac-83eb5fb48d1mr532791239f.4.1732137133729; Wed, 20 Nov 2024 13:12:13 -0800 (PST) Received: from [192.168.1.128] ([38.175.170.29]) by smtp.gmail.com with ESMTPSA id 8926c6da1cb9f-4e06d6e7374sm3286443173.32.2024.11.20.13.12.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 20 Nov 2024 13:12:13 -0800 (PST) Message-ID: <71b51954-15ba-4e73-baea-584463d43a5c@linuxfoundation.org> Date: Wed, 20 Nov 2024 14:12:12 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/2 v2] bcachefs: do not use PF_MEMALLOC_NORECLAIM To: Kent Overstreet , Michal Hocko Cc: Dave Chinner , Andrew Morton , Christoph Hellwig , Yafang Shao , jack@suse.cz, Christian Brauner , Alexander Viro , Paul Moore , James Morris , "Serge E. Hallyn" , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-bcachefs@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org, "conduct@kernel.org" , Shuah Khan References: <22a3da3d-6bca-48c6-a36f-382feb999374@linuxfoundation.org> Content-Language: en-US From: Shuah Khan In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 33B661C0006 X-Stat-Signature: dzqgstqjk53eg6exp9fao646f8wsworm X-Rspam-User: X-Rspamd-Server: rspam05 X-HE-Tag: 1732137036-717062 X-HE-Meta: U2FsdGVkX1+eFA4mnGMoHlsvGnUy7VdKfsHdNm33u0vsbxUB+NAx2O8bbCnJrXaafYFMOrK0AJASI3ENsqm/MhhZ+348P7Rk+FvyROflypAOBOQA/W+LzIDCuFBWjvqPzSUJwFpKNfBykj77gK5x2E1tru2x0qp3DKelZnL5HGE7Ni4lFPq0C0YnagNqJQWxSFOLHnNhMhd/hV1+QDIAMeKUWZ6eVdcKSjHsda/aWcdjDxfLeIyq5WWnDafKH28Vi9YjLGYkDs4SUeF1A9jdfaWvmb7d99Ati/SMag2sa7oJxf8SHyWNJyiSmYR9CwBaJxe5ZWWdMCqDixKZCDqavgj7+NW4pyeSITEEx7Bb1354M8J14MVOUC1LKONTuUk9xyxaWAdX/k/ODwmeS05Lk5oP9a1/2pAc65g2FzJ6yEI0Yqb/ySGgsW4/MVTb8zxTyGo4pJn87umOhIVP0G7R0DwRmG7wRPmmTLgwboMGgQ2gq/8u5GI4Zs5NuaA8xTwCq2jxgjnXIuX6ziTRk2FIb0xSy5thQxClvE5kJUXRGwxodfzlWaoHZFGLiKryhsVXXPHqSyCZpMN+tN6TWFjJ83cnHTA0g4ERxlHu3eGYCkzi5eXTC2VMlzNotnbUa9vdjnGSp4J4W4+j5Rao/g8rozjiKEWm9DZUeWbYc2JYs+SFFKiUaPmnjh/yGsG0XKXdeFp2LgylOTYDsMr+3R1AMLnixPnIQJoIE+pG7Mz9y3KPF11fleebntu2AMUOrXyUOMo4beDcZwEmC6nmZBbGbhOAdqYaWUqPMbJELmiTLab2bybNUoqv74nI/owd/OBrqWCmRyX5lNB6Rw6h2HE2lbas6S+3KmO0WB5dFUlGcd37nAKwDCVc5SDcOsQX0ISeCwhnYz1DHx09LsOIguS0Pi8EdpdCJGG2idJB2zkYdZitSbeEW7GrVVFCI6MoaLlAb4oAtczRqVyuOTfMLea 595c07AO ZGJ2FOadtGcJifVnXeUkDQRMuZRWv4bDvY6auWLRmPdTnr3wnEH+OSbzIFYKLFCvCxubWuYODHcM4ffrwHoMomIiP5ZopfsCNe8J7HRzF7+KNGMSo7jQDLEimlWThb+IW744csGUrdue7NLjjBMlWUPA7gURf2FxK7jAcuddZowrAlrxSYNupA2kNBcB41uYqLx8MQz5Y/ZBWac1pEbbHlBzsXhBV1sL5yVDppofe4/jqZ4XF1zygDDuOcRg6Zd5lJ39rnWOU1Iz+bYfAT3sucn+2qPPUDHHumHjqkrBsGRHwz6RqI889dMREhd5e72tjDQJSx5a7y602tBLYEUS4c52O8ALnoJpej8DbPuTl1pG/4mX/QDRjzs6U5ONd2KtZN4TsKXlfMVSxmBjo4zNwn5p9y6qmv5U4xVzGvHFjpYTyQfKdIkPus/wiqGT9IH8ylAvmyVNp3l0c/yyqfW6Urnxryt83TOAvPfuFdyui6ogdgYWI4rYKUBRqwl46CEb1eYhovcpHpzZ4rYl2tMqNJ1OBR/R0ZMmqQAat4yZrVmqAB4KYUhnShjEfsHHxKaiES3N+ECsj6Awt8iY= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000004, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 11/20/24 13:34, Kent Overstreet wrote: > On Wed, Sep 04, 2024 at 12:01:50PM -0600, Shuah Khan wrote: >> On 9/2/24 03:51, Kent Overstreet wrote: >>> On Mon, Sep 02, 2024 at 11:39:41AM GMT, Michal Hocko wrote: >>>> On Mon 02-09-24 04:52:49, Kent Overstreet wrote: >>>>> On Mon, Sep 02, 2024 at 10:41:31AM GMT, Michal Hocko wrote: >>>>>> On Sun 01-09-24 21:35:30, Kent Overstreet wrote: >>>>>> [...] >>>>>>> But I am saying that kmalloc(__GFP_NOFAIL) _should_ fail and return NULL >>>>>>> in the case of bugs, because that's going to be an improvement w.r.t. >>>>>>> system robustness, in exactly the same way we don't use BUG_ON() if it's >>>>>>> something that we can't guarantee won't happen in the wild - we WARN() >>>>>>> and try to handle the error as best we can. >>>>>> >>>>>> We have discussed that in a different email thread. And I have to say >>>>>> that I am not convinced that returning NULL makes a broken code much >>>>>> better. Why? Because we can expect that broken NOFAIL users will not have a >>>>>> error checking path. Even valid NOFAIL users will not have one because >>>>>> they _know_ they do not have a different than retry for ever recovery >>>>>> path. >>>>> >>>>> You mean where I asked you for a link to the discussion and rationale >>>>> you claimed had happened? Still waiting on that >>>> >>>> I am not your assistent to be tasked and search through lore archives. >>>> Find one if you need that. >>>> >>>> Anyway, if you read the email and even tried to understand what is >>>> written there rather than immediately started shouting a response then >>>> you would have noticed I have put actual arguments here. You are free to >>>> disagree with them and lay down your arguments. You have decided to >>>> >>>> [...] >>>> >>>>> Yeah, enough of this insanity. >>>> >>>> so I do not think you are able to do that. Again... >>> >>> Michal, if you think crashing processes is an acceptable alternative to >>> error handling _you have no business writing kernel code_. >>> >>> You have been stridently arguing for one bad idea after another, and >>> it's an insult to those of us who do give a shit about writing reliable >>> software. >>> >>> You're arguing against basic precepts of kernel programming. >>> >>> Get your head examined. And get the fuck out of here with this shit. >>> >> >> Kent, >> >> Using language like this is clearly unacceptable and violates the >> Code of Conduct. This type of language doesn't promote respectful >> and productive discussions and is detrimental to the health of the >> community. >> >> You should be well aware that this type of language and personal >> attack is a clear violation of the Linux kernel Contributor Covenant >> Code of Conduct as outlined in the following: >> >> https://www.kernel.org/doc/html/latest/process/code-of-conduct.html >> >> Refer to the Code of Conduct and refrain from violating the Code of >> Conduct in the future. > > I believe Michal and I have more or less worked this out privately (and > you guys have been copied on that as well). Thank you for updating us on the behind the scenes work between you and Michal. I will make one correction to your statement, "you guys have been copied on that as well" - which is inaccurate. You have shared your email exchanges with Michal with us to let us know that the issue has been sorted out. You might have your reasons and concerns about the direction of the code and design that pertains to the discussion in this email thread. You might have your reasons for expressing your frustration. However, those need to be worked out as separate from this Code of Conduct violation. In the case of unacceptable behaviors as defined in the Code of Conduct document, the process is to work towards restoring productive and respectful discussions. It is reasonable to ask for an apology to help us get to the goal as soon as possible. I urge you once again to apologize for using language that negatively impacts productive discussions. thanks, -- Shuah (On behalf of the Code of Conduct Committee)