From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [192.168.43.69] (unknown [182.2.41.243]) by gnuweeb.org (Postfix) with ESMTPSA id 332EA7E216; Thu, 3 Mar 2022 02:07:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gnuweeb.org; s=default; t=1646273241; bh=6hdPeUDTpFCH5tm6GIDzeefZQ1DijYbUe34dlL4yiHc=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=UIH3FdIHhEtHk9207mgmSkpczcVJySwwhRd+ngT+WnK2kIMqnXD+y1/jr1F7gCnOI Oytv61vKSH1RjRiybyD9OcV+fXo992onmw6sDyNjdO5v/4FBiBI3SCdRzJcsZhVvLs GqQzJXlA1qa8vPLNHn9VYvzGX0EOGde+wd4MB2KB5AnfLJBZx+Ue1NwZgJlk3VDLMf +CGFGCi6A2gjPDAJyGhUDIWNwZbxgJyLXh4rMia+u4hw1rtEaDCWWBTKp+NsblZxgk Q8OXX7WXFf/recNzBhiPcnCYbycBihpDoTwZX7VNTRBkDj3J6TBM0xQFitYIb9/xLC kyje9KbfxM6Tg== Message-ID: <49313736-61f8-d001-0fe4-b6166c859585@gnuweeb.org> Date: Thu, 3 Mar 2022 09:07:12 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH v4 2/2] x86/mce/amd: Fix memory leak when `threshold_create_bank()` fails Content-Language: en-US To: Alviro Iskandar Setiawan , Yazen Ghannam Cc: Borislav Petkov , Dave Hansen , "H. Peter Anvin" , Ingo Molnar , Thomas Gleixner , Tony Luck , linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org, gwml@vger.gnuweeb.org, x86@kernel.org, stable@vger.kernel.org, Alviro Iskandar Setiawan , Jiri Hladky , Greg Kroah-Hartman References: <20220301094608.118879-1-ammarfaizi2@gnuweeb.org> <20220301094608.118879-3-ammarfaizi2@gnuweeb.org> <4371a592-6686-c535-4daf-993dedb43cd4@gnuweeb.org> <109a10da-d1d1-c47a-2f04-31796457f6ff@gnuweeb.org> <20220303015826.4176416-1-alviro.iskandar@gnuweeb.org> From: Ammar Faizi In-Reply-To: <20220303015826.4176416-1-alviro.iskandar@gnuweeb.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: On 3/3/22 8:58 AM, Alviro Iskandar Setiawan wrote: > hi sir, i think this can be improved again, we can avoid calling > this_cpu_read(mce_num_banks) twice if we pass the numbanks as an > argument, plz review the changes below OK, nice improvement. I will fold this in... -- Ammar Faizi