Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp128960pxb; Mon, 31 Jan 2022 17:13:48 -0800 (PST) X-Google-Smtp-Source: ABdhPJwwCgQBb5qX9dUzgaSiX6DN7hjZ87wKrcLmM8101B1zYwjpb/XexvOKtqdv9QD0o9i07tMs X-Received: by 2002:a17:907:6094:: with SMTP id ht20mr18916853ejc.628.1643678028501; Mon, 31 Jan 2022 17:13:48 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643678028; cv=none; d=google.com; s=arc-20160816; b=OVpLUozV3ZAhu03yegSiijM6XmPXXiEPCXaF0bqk2QlJEWy2714E9WI7THHsHbtqF9 tJvzyzxOSU2zgA0tBubsU482sApx9Si3l11egzoU//m+QWQpg+pK1yCx+ZU0Acu3A6CL 9fisF9tKU7EvcgXSnWHnwSK46nCfka09kd/wRXm/3kH9JXKVx8FBBce2bCGgbgcaxi3/ YvuSCAniwmoUIqurGlOTZbUWmZdJqjdXMp+Sm7fOiAaywCfqsHG8qqqUYs2/7XmLgay2 bd3pe5VBu0/CyMhcGY9ZAb2Md8Vn782DYHuMxu1hwruM0REbmnzZs25hTNiIcNLxfy9k GEuA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=fpXYd8qILvfSSJSACckqfoaHLEMTY+fmEu0bCn+QNB4=; b=o02pT6My0JlAlOAQ85iSPZsTJCCDRd3E2VHoIGbt7G73/dvELXI7VcEYk0G8iVdnxg d+3QFVkiLZTzSHFHkQotj4B2GHTC+AklSzuno7DpxmqVkSZzbVyZ6uYDYyoVZIfYAe1w V/q1EOxHqV5hOmBdh9LIPC/yqbuk3WL0SFAfEkseVoDjzkRun7khBMdhj+otsKfF8uqa hGXKnwzdjHz6IyCSw+em7jNhtVARZb8hNj6BTs1r6PAy1TxuFcp51kE7mfu4acnu8Wq+ VBM1iaCFSAG2lrkSNsBcCiJeufX3vXuIgOdwUj3lgVDG1DM3ndMH0i9CN646GLaAjXTM Zy8g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=2BwzNVJe; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id o7si9785396edc.425.2022.01.31.17.13.22; Mon, 31 Jan 2022 17:13:48 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=2BwzNVJe; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236180AbiA2Iai (ORCPT + 99 others); Sat, 29 Jan 2022 03:30:38 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59080 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1352517AbiA2Iah (ORCPT ); Sat, 29 Jan 2022 03:30:37 -0500 Received: from sin.source.kernel.org (sin.source.kernel.org [IPv6:2604:1380:40e1:4800::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 131F2C061714 for ; Sat, 29 Jan 2022 00:30:36 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sin.source.kernel.org (Postfix) with ESMTPS id 67C12CE284E for ; Sat, 29 Jan 2022 08:30:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1AED5C340E5; Sat, 29 Jan 2022 08:30:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1643445032; bh=LmS6TDU/ucq+n9hRSsa4AQxVYWYfepde97p1cN1bCsA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=2BwzNVJeD3uuyqaJ4EXIsk7Dwgrcm4eTfSCBkcnUsZHXoJ/taxE3koywyfkt7XuSa 6CuQcQPDaVS4pzRk9Hlew2NbGOTDajfAPm+y+RomuL2YpqbVQ36TamuMymC6up9f26 UocQBfhhoOjMLgZL+cXVDqTUbGFT4Bd8uWVX4qMI= Date: Sat, 29 Jan 2022 09:30:29 +0100 From: Greg KH To: Karol Herbst Cc: Alex Deucher , Lyude Paul , Zhou Qingyang , David Airlie , nouveau , Kangjie Lu , LKML , Maling list - DRI developers , Ben Skeggs Subject: Re: [PATCH] drm/nouveau/acr: Fix undefined behavior in nvkm_acr_hsfw_load_bl() Message-ID: References: <20220124165856.57022-1-zhou1615@umn.edu> <536c833413ccbe0b8ad653a50c5ea867bf975290.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 28, 2022 at 08:57:56PM +0100, Karol Herbst wrote: > On Fri, Jan 28, 2022 at 8:54 PM Alex Deucher wrote: > > > > On Fri, Jan 28, 2022 at 2:20 PM Lyude Paul wrote: > > > > > > Sigh-thank you for catching this - I had totally forgot about the umn.edu ban. > > > I pushed this already but I will go ahead and send a revert for this patch. > > > Will cc you on it as well. > > > > This seems short-sighted. If the patch is valid I see no reason to > > not accept it. I'm not trying to downplay the mess umn got into, but > > as long as the patch is well scrutinized and fixes a valid issue, it > > should be applied rather than leaving potential bugs in place. > > > > Alex > > > > Even though knowing that malicious code can be introduced via > perfectly fine looking patches, and sometimes one will never spot the > problem, this patch isn't all that bad tbh. > > So should we reject patches out of "policies" or should we just be > extra careful? But not addressing the concerns as Greg pointed out is > also kind of a bad move, but also not knowing what the state of > resolving this mess is anyway. If you think the change is correct, and are willing to sign off on it, that's fine for now. The big issue here is that the umn.edu "researchers" are not folling the very basic and simple proceedures that we worked to set up with them, and have resumed sending buggy patches to the kernel community. If you think this one is not a problem, then feel free to keep it. As I just now told their administration, it is very difficult to tell malice from incompetence. If this is malice, then we need to defend our community. If this is just incompetence, then the university has to handle that as the contributions are coming in their name, _and_ we also need to defend from that. What we have tried in the past for this group is obviously not working, so we now will need to do something else. {sigh} greg k-h