Message ID | 20190417144617.14922-1-fbl@redhat.com |
---|---|
Headers | show |
Series | openvswitch: load and reference the NAT helper | expand |
From: Flavio Leitner <fbl@redhat.com> Date: Wed, 17 Apr 2019 11:46:13 -0300 > The request_module() is quite expensive and triggers the > usermode helper in userspace. Instead, load only if the > module is not present and keep module references to avoid > problems. > > The first patch standardize the module alias which is already > there, but not in a formal way. > > The second patch adds an API to point to the NAT helper. > > The third patch will register each NAT helper using the > new API. > > The last patch fixes openvswitch to use the new API to > load and reference the NAT helper and also report an error > if the operation fails. Netfilter folks, besides a review, I was wondering if you'd like to take this series the netfilter tree since it's mostly changes in that area? Thanks.
On Wed, Apr 17, 2019 at 11:25:09PM -0700, David Miller wrote: > From: Flavio Leitner <fbl@redhat.com> > Date: Wed, 17 Apr 2019 11:46:13 -0300 > > > The request_module() is quite expensive and triggers the > > usermode helper in userspace. Instead, load only if the > > module is not present and keep module references to avoid > > problems. > > > > The first patch standardize the module alias which is already > > there, but not in a formal way. > > > > The second patch adds an API to point to the NAT helper. > > > > The third patch will register each NAT helper using the > > new API. > > > > The last patch fixes openvswitch to use the new API to > > load and reference the NAT helper and also report an error > > if the operation fails. > > Netfilter folks, besides a review, I was wondering if you'd like > to take this series the netfilter tree since it's mostly changes > in that area? I'll place this in the nf-next tree, thanks.
On Wed, Apr 17, 2019 at 11:46:13AM -0300, Flavio Leitner wrote: > The request_module() is quite expensive and triggers the > usermode helper in userspace. Instead, load only if the > module is not present and keep module references to avoid > problems. > > The first patch standardize the module alias which is already > there, but not in a formal way. > > The second patch adds an API to point to the NAT helper. > > The third patch will register each NAT helper using the > new API. > > The last patch fixes openvswitch to use the new API to > load and reference the NAT helper and also report an error > if the operation fails. Series applied. Thanks Flavio.