From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Carstens Subject: Re: [PATCH v3 net-next] fix unsafe set_memory_rw from softirq Date: Fri, 4 Oct 2013 08:47:05 +0200 Message-ID: <20131004064705.GA4143@osiris> References: <1380853446-30537-1-git-send-email-ast@plumgrid.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Eric Dumazet , Paul Mackerras , "H. Peter Anvin" , sparclinux@vger.kernel.org, Nicolas Dichtel , linux-s390@vger.kernel.org, Russell King , x86@kernel.org, James Morris , Ingo Molnar , Alexey Kuznetsov , "Paul E. McKenney" , Xi Wang , Matt Evans , Thomas Gleixner , linux-arm-kernel@lists.infradead.org, Stelian Nirlu , Nicolas Schichan , Hideaki YOSHIFUJI , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, "David S. Miller" , Mircea Gherzan , Daniel Borkmann , Martin Schwidefsky Return-path: Content-Disposition: inline In-Reply-To: <1380853446-30537-1-git-send-email-ast@plumgrid.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane.org@lists.ozlabs.org Sender: "Linuxppc-dev" List-Id: netdev.vger.kernel.org On Thu, Oct 03, 2013 at 07:24:06PM -0700, Alexei Starovoitov wrote: > diff --git a/arch/s390/net/bpf_jit_comp.c b/arch/s390/net/bpf_jit_comp.c > index 7092392..a5df511 100644 > --- a/arch/s390/net/bpf_jit_comp.c > +++ b/arch/s390/net/bpf_jit_comp.c > @@ -881,7 +881,9 @@ void bpf_jit_free(struct sk_filter *fp) > struct bpf_binary_header *header = (void *)addr; > > if (fp->bpf_func == sk_run_filter) > - return; > + goto free_filter; > set_memory_rw(addr, header->pages); > module_free(NULL, header); > +free_filter: > + kfree(fp); > } For the s390 part: Acked-by: Heiko Carstens