Page MenuHomeFreeBSD
Differential D29772

ipfw: do not use sleepable malloc in callout context.
ClosedPublic
Actions

Authored by ae on Apr 15 2021, 12:35 PM.
Tags
None
Referenced Files
F154314447: D29772.id.diff
Mon, Apr 27, 7:02 PM
Unknown Object (File)
Sun, Apr 26, 2:16 AM
Unknown Object (File)
Thu, Apr 23, 6:51 AM
Unknown Object (File)
Thu, Apr 16, 8:22 AM
Unknown Object (File)
Mon, Apr 13, 3:40 AM
Unknown Object (File)
Fri, Apr 10, 2:04 AM
Unknown Object (File)
Thu, Apr 9, 8:53 PM
Unknown Object (File)
Sun, Mar 29, 1:33 AM
View All Files
Subscribers
freebsd_igalic.co
imp
ish_amail.plala.or.jp
melifaro

Details

Reviewers
melifaro
kevans
Group Reviewers
network
Commits
rG5ebfa80f7543: ipfw: do not use sleepable malloc in callout context.
rG29246f2cb0c5: ipfw: do not use sleepable malloc in callout context.
rG9bacbf1ae243: ipfw: do not use sleepable malloc in callout context.
Summary

Use malloc(M_NOWAIT) when hash growing is called from callout.

PR: 255041

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 38591
Build 35480: arc lint + arc unit

Event Timeline

ae created this revision.Apr 15 2021, 12:35 PM
ae held this revision as a draft.
Herald added subscribers: melifaro, imp. · View Herald TranscriptApr 15 2021, 12:35 PM
Harbormaster completed remote builds in B38590: Diff 87481.Apr 15 2021, 12:35 PM
ae published this revision for review.Apr 15 2021, 12:37 PM
ae edited the summary of this revision. (Show Details)
ae added reviewers: network, melifaro, kevans.
ae updated this revision to Diff 87482.Apr 15 2021, 12:40 PM
  • Fix copy/paste bug.
Harbormaster completed remote builds in B38591: Diff 87482.Apr 15 2021, 12:40 PM
kevans accepted this revision.Apr 15 2021, 1:08 PM
This revision is now accepted and ready to land.Apr 15 2021, 1:08 PM
kevans added a comment.Apr 15 2021, 1:08 PM

Thanks!

ish_amail.plala.or.jp added a subscriber: ish_amail.plala.or.jp.Apr 15 2021, 2:55 PM

Thank you for this patch.
I'm applied this patch to 3a248c844 and it works fine.

Closed by commit rG9bacbf1ae243: ipfw: do not use sleepable malloc in callout context. (authored by ae). · Explain WhyApr 16 2021, 7:25 AM
This revision was automatically updated to reflect the committed changes.
ae added a commit: rG9bacbf1ae243: ipfw: do not use sleepable malloc in callout context..
freebsd_igalic.co added a subscriber: freebsd_igalic.co.Apr 16 2021, 7:44 AM
freebsd_igalic.co added inline comments.
sys/netpfil/ipfw/ip_fw_dynamic.c
3224

why is it okay to wait here

ae added inline comments.Apr 16 2021, 8:23 AM
sys/netpfil/ipfw/ip_fw_dynamic.c
3224

It is initialization code, it is called from userland when you are doing kldload, user can wait.

ae added a commit: rG29246f2cb0c5: ipfw: do not use sleepable malloc in callout context..Apr 28 2021, 10:06 AM
ae added a commit: rG5ebfa80f7543: ipfw: do not use sleepable malloc in callout context..

Revision Contents
Changeset List

PathSize
sys/
netpfil/
ipfw/
68 lines

Diff 87482

View Options

sys/netpfil/ipfw/ip_fw_dynamic.c

Loading...