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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D2A27CD80B0 for ; Tue, 10 Oct 2023 11:46:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231419AbjJJLqE (ORCPT ); Tue, 10 Oct 2023 07:46:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44380 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230320AbjJJLqD (ORCPT ); Tue, 10 Oct 2023 07:46:03 -0400 Received: from mail-wr1-x42f.google.com (mail-wr1-x42f.google.com [IPv6:2a00:1450:4864:20::42f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CEA909D for ; Tue, 10 Oct 2023 04:46:01 -0700 (PDT) Received: by mail-wr1-x42f.google.com with SMTP id ffacd0b85a97d-3232e96deaaso1349713f8f.0 for ; Tue, 10 Oct 2023 04:46:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; t=1696938360; x=1697543160; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=aiA4D11X2sFtXKvFj+BVZkBdxaJKzeEbh5cY8Yxytvg=; b=GWGMs53od8yUrlA7N4SoxSsh2ZToEPeBj3Rpp+VJFhZn5gtJWgk2cQfJ5xogoiQqUR jbV+gSb9JkOQS2z8iE6aFgQckIx8TNsSpye63zupsFBMDI1ULBRExnoNTE1r/7W2Nwv0 6vsSPucoUJzQO8M6+++2+Zc9IvadJXeA1sAnI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1696938360; x=1697543160; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=aiA4D11X2sFtXKvFj+BVZkBdxaJKzeEbh5cY8Yxytvg=; b=iIys9nKIKXOL3vfCBqqDdwCGRpLZIfNPt1mGbqMyljljJKlPMtQfNs0G5/UsDIhpap 06oMq/ueWP5ZMRtGREQObWz46twCci4YSvAooEz6lpLf10gl6yDOa2+9e06T8FqMtpAx U/7/JESy2/F2hrvLjjEFiFHZ4RMk8sKuJ4zxYH+n/Gy/FVGAiuDoXwIK8ZFE2k7eCQit OrEWgB1iBPzq4Gbph2vUHZ1C8BHyT3wRrdEb86j1syWouZklseB6Qaz7tf7eTsmjlLVC ow8qfVyWIWi/qtnB7jjjyt45ZfulCTF2EJbLyreW7ODbiSCUxCdRhqjC+HfDIuVcL+qD yfiQ== X-Gm-Message-State: AOJu0YxeKDAR0iqEA+UlWjEyNiIqflHQFVaFHcjWwH4ibYwuiiCa+4RO h+A4n7zmkVjvWD0SNIqpYrFi3A== X-Google-Smtp-Source: AGHT+IE4/UvRoJK4tGLVIxW4ixF9DbnO2siW/LMpb7C9C8dsXdtbYclfJKlYBA0WjLwnhm1UDv3LbA== X-Received: by 2002:a5d:6909:0:b0:320:8f0:b93d with SMTP id t9-20020a5d6909000000b0032008f0b93dmr14892975wru.3.1696938360227; Tue, 10 Oct 2023 04:46:00 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:57f4:0:efd0:b9e5:5ae6:c2fa]) by smtp.gmail.com with ESMTPSA id h9-20020a5d6889000000b0031c52e81490sm12640936wru.72.2023.10.10.04.45.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Oct 2023 04:45:59 -0700 (PDT) Date: Tue, 10 Oct 2023 13:45:57 +0200 From: Daniel Vetter To: Ondrej Zary Cc: Helge Deller , jiapeng.chong@linux.alibaba.com, linux-fbdev@vger.kernel.org, javierm@redhat.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, tzimmermann@suse.de, Zhang Shurong Subject: Re: [PATCH] video: fbdev: arkfb: fix possible object reference leak Message-ID: Mail-Followup-To: Ondrej Zary , Helge Deller , jiapeng.chong@linux.alibaba.com, linux-fbdev@vger.kernel.org, javierm@redhat.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, tzimmermann@suse.de, Zhang Shurong References: <202310061634.02110.linux@zary.sk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <202310061634.02110.linux@zary.sk> X-Operating-System: Linux phenom 6.5.0-1-amd64 Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org On Fri, Oct 06, 2023 at 04:34:01PM +0200, Ondrej Zary wrote: > On Friday 06 October 2023, Helge Deller wrote: > > On 10/5/23 09:01, Zhang Shurong wrote: > > > Add missing pci_disable_device() in error path in ark_pci_probe(). > > > > Do you have this hardware and tested your patch? > > I'm sure there is a reason, why "pci_disable_device()" was commented > > out in the original submission in commit 681e14730c73c... > > pci_disable_device() call is disabled in many fbdev drivers because > calling it might prevent display from working. Yeah I've recently hit some _really_ interesting regression report where essentially fbdev driver and vgacon where managing the hardware collaboratively. There's enormous amounts of "this shouldn't work like that, but it is how it does" going on in old fbdev drivers, and what you brought up is also what I immediately thought of. This patch feels way too risky to me that we should apply it ... -Sima > > > > > Additionally I'm wondering why your patch doesn't show up in > > the fbdev patchwork, although you added linux-fbdev mailing list. > > Probably a vger issue. > > > > Helge > > > > > > > Signed-off-by: Zhang Shurong > > > --- > > > drivers/video/fbdev/arkfb.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/video/fbdev/arkfb.c b/drivers/video/fbdev/arkfb.c > > > index 60a96fdb5dd8..6c4e5065646f 100644 > > > --- a/drivers/video/fbdev/arkfb.c > > > +++ b/drivers/video/fbdev/arkfb.c > > > @@ -1064,7 +1064,7 @@ static int ark_pci_probe(struct pci_dev *dev, const struct pci_device_id *id) > > > err_dac: > > > pci_release_regions(dev); > > > err_request_regions: > > > -/* pci_disable_device(dev); */ > > > + pci_disable_device(dev); > > > err_enable_device: > > > framebuffer_release(info); > > > return rc; > > > @@ -1085,7 +1085,7 @@ static void ark_pci_remove(struct pci_dev *dev) > > > > > > pci_iounmap(dev, info->screen_base); > > > pci_release_regions(dev); > > > -/* pci_disable_device(dev); */ > > > + pci_disable_device(dev); > > > > > > framebuffer_release(info); > > > } > > > > > > > > -- > Ondrej Zary -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch