From: Michal Hocko <mhocko@kernel.org> To: Mikulas Patocka <mpatocka@redhat.com> Cc: Matthew Wilcox <willy@infradead.org>, David Miller <davem@davemloft.net>, Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org, eric.dumazet@gmail.com, edumazet@google.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, mst@redhat.com, jasowang@redhat.com, virtualization@lists.linux-foundation.org, dm-devel@redhat.com, Vlastimil Babka <vbabka@suse.cz> Subject: Re: [PATCH v3] kvmalloc: always use vmalloc if CONFIG_DEBUG_SG Date: Tue, 24 Apr 2018 06:51:21 -0600 [thread overview] Message-ID: <20180424125121.GA17484@dhcp22.suse.cz> (raw) In-Reply-To: <alpine.LRH.2.02.1804232003100.2299@file01.intranet.prod.int.rdu2.redhat.com> On Mon 23-04-18 20:06:16, Mikulas Patocka wrote: [...] > @@ -404,6 +405,12 @@ void *kvmalloc_node(size_t size, gfp_t f > */ > WARN_ON_ONCE((flags & GFP_KERNEL) != GFP_KERNEL); > > +#ifdef CONFIG_DEBUG_SG > + /* Catch bugs when the caller uses DMA API on the result of kvmalloc. */ > + if (!(prandom_u32_max(2) & 1)) > + goto do_vmalloc; > +#endif I really do not think there is anything DEBUG_SG specific here. Why you simply do not follow should_failslab path or even reuse the function? > + > /* > * We want to attempt a large physically contiguous block first because > * it is less likely to fragment multiple larger blocks and therefore > @@ -427,6 +434,9 @@ void *kvmalloc_node(size_t size, gfp_t f > if (ret || size <= PAGE_SIZE) > return ret; > > +#ifdef CONFIG_DEBUG_SG > +do_vmalloc: > +#endif > return __vmalloc_node_flags_caller(size, node, flags, > __builtin_return_address(0)); > } -- Michal Hocko SUSE Labs
WARNING: multiple messages have this Message-ID (diff)
From: Michal Hocko <mhocko@kernel.org> To: Mikulas Patocka <mpatocka@redhat.com> Cc: dm-devel@redhat.com, eric.dumazet@gmail.com, mst@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Matthew Wilcox <willy@infradead.org>, virtualization@lists.linux-foundation.org, linux-mm@kvack.org, edumazet@google.com, Andrew Morton <akpm@linux-foundation.org>, David Miller <davem@davemloft.net>, Vlastimil Babka <vbabka@suse.cz> Subject: Re: [PATCH v3] kvmalloc: always use vmalloc if CONFIG_DEBUG_SG Date: Tue, 24 Apr 2018 06:51:21 -0600 [thread overview] Message-ID: <20180424125121.GA17484@dhcp22.suse.cz> (raw) In-Reply-To: <alpine.LRH.2.02.1804232003100.2299@file01.intranet.prod.int.rdu2.redhat.com> On Mon 23-04-18 20:06:16, Mikulas Patocka wrote: [...] > @@ -404,6 +405,12 @@ void *kvmalloc_node(size_t size, gfp_t f > */ > WARN_ON_ONCE((flags & GFP_KERNEL) != GFP_KERNEL); > > +#ifdef CONFIG_DEBUG_SG > + /* Catch bugs when the caller uses DMA API on the result of kvmalloc. */ > + if (!(prandom_u32_max(2) & 1)) > + goto do_vmalloc; > +#endif I really do not think there is anything DEBUG_SG specific here. Why you simply do not follow should_failslab path or even reuse the function? > + > /* > * We want to attempt a large physically contiguous block first because > * it is less likely to fragment multiple larger blocks and therefore > @@ -427,6 +434,9 @@ void *kvmalloc_node(size_t size, gfp_t f > if (ret || size <= PAGE_SIZE) > return ret; > > +#ifdef CONFIG_DEBUG_SG > +do_vmalloc: > +#endif > return __vmalloc_node_flags_caller(size, node, flags, > __builtin_return_address(0)); > } -- Michal Hocko SUSE Labs
next prev parent reply other threads:[~2018-04-24 12:51 UTC|newest] Thread overview: 218+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-04-18 14:34 [PATCH] net: don't use kvzalloc for DMA memory Mikulas Patocka 2018-04-18 16:05 ` Eric Dumazet 2018-04-18 16:44 ` Mikulas Patocka 2018-04-18 16:44 ` Mikulas Patocka 2018-04-18 16:51 ` Eric Dumazet 2018-04-18 16:51 ` Eric Dumazet 2018-04-18 17:47 ` David Miller 2018-04-18 17:47 ` David Miller 2018-04-18 17:55 ` Mikulas Patocka 2018-04-18 17:55 ` Mikulas Patocka 2018-04-18 18:00 ` Michael S. Tsirkin 2018-04-18 18:00 ` Michael S. Tsirkin 2018-04-18 17:49 ` Mikulas Patocka 2018-04-18 17:49 ` Mikulas Patocka 2018-04-18 16:51 ` Eric Dumazet 2018-04-18 17:46 ` David Miller 2018-04-18 17:53 ` Mikulas Patocka 2018-04-19 16:12 ` [PATCH] kvmalloc: always use vmalloc if CONFIG_DEBUG_VM Mikulas Patocka 2018-04-19 16:12 ` Mikulas Patocka 2018-04-19 16:25 ` Eric Dumazet 2018-04-19 16:25 ` Eric Dumazet 2018-04-19 16:28 ` Mikulas Patocka 2018-04-19 16:28 ` Mikulas Patocka 2018-04-19 16:43 ` Michael S. Tsirkin 2018-04-19 21:27 ` [PATCH] kvmalloc: always use vmalloc if CONFIG_DEBUG_SG Mikulas Patocka 2018-04-19 21:27 ` Mikulas Patocka 2018-04-19 16:43 ` [PATCH] kvmalloc: always use vmalloc if CONFIG_DEBUG_VM Michael S. Tsirkin 2018-04-19 18:28 ` Vlastimil Babka 2018-04-19 18:28 ` Vlastimil Babka 2018-04-19 19:47 ` Andrew Morton 2018-04-19 19:47 ` Andrew Morton 2018-04-19 21:19 ` Mikulas Patocka 2018-04-19 21:19 ` Mikulas Patocka 2018-04-19 23:22 ` Andrew Morton 2018-04-19 23:22 ` Andrew Morton 2018-04-20 12:16 ` Mikulas Patocka 2018-04-20 12:16 ` Mikulas Patocka 2018-04-20 11:47 ` Matthew Wilcox 2018-04-20 11:47 ` Matthew Wilcox 2018-04-20 12:20 ` Mikulas Patocka 2018-04-23 15:25 ` Michael S. Tsirkin 2018-04-23 15:25 ` Michael S. Tsirkin 2018-04-20 12:20 ` Mikulas Patocka 2018-04-20 13:08 ` Michal Hocko 2018-04-20 13:08 ` Michal Hocko 2018-04-20 13:41 ` Matthew Wilcox 2018-04-20 13:41 ` Matthew Wilcox 2018-04-20 13:49 ` Michal Hocko 2018-04-20 13:49 ` Michal Hocko 2018-04-20 20:56 ` Mikulas Patocka 2018-04-20 20:56 ` Mikulas Patocka 2018-04-20 20:54 ` Mikulas Patocka 2018-04-20 21:02 ` Matthew Wilcox 2018-04-20 21:02 ` Matthew Wilcox 2018-04-20 21:21 ` Mikulas Patocka 2018-04-21 14:47 ` Matthew Wilcox 2018-04-22 13:03 ` Michal Hocko 2018-04-22 13:03 ` Michal Hocko 2018-04-23 14:24 ` Mikulas Patocka 2018-04-23 15:10 ` Michal Hocko 2018-04-23 15:10 ` Michal Hocko 2018-04-23 23:20 ` Mikulas Patocka 2018-04-23 23:20 ` Mikulas Patocka 2018-04-23 14:24 ` Mikulas Patocka 2018-04-23 14:06 ` Mikulas Patocka 2018-04-23 14:06 ` Mikulas Patocka 2018-04-23 15:15 ` Michal Hocko 2018-04-23 15:15 ` Michal Hocko 2018-04-24 0:06 ` [PATCH v3] kvmalloc: always use vmalloc if CONFIG_DEBUG_SG Mikulas Patocka 2018-04-24 0:06 ` Mikulas Patocka 2018-04-24 2:47 ` David Rientjes 2018-04-24 11:04 ` Mikulas Patocka 2018-04-24 11:04 ` Mikulas Patocka 2018-04-24 3:46 ` Matthew Wilcox 2018-04-24 3:46 ` Matthew Wilcox 2018-04-24 12:29 ` Mikulas Patocka 2018-04-24 12:29 ` Mikulas Patocka 2018-04-24 17:16 ` Matthew Wilcox 2018-04-24 18:41 ` Mikulas Patocka 2018-04-24 18:41 ` Mikulas Patocka 2018-05-15 1:13 ` Joonsoo Kim 2018-05-15 1:13 ` Joonsoo Kim 2018-04-24 17:16 ` Matthew Wilcox 2018-04-24 12:51 ` Michal Hocko [this message] 2018-04-24 12:51 ` Michal Hocko 2018-04-24 15:50 ` Mikulas Patocka 2018-04-24 16:29 ` Michal Hocko 2018-04-24 17:00 ` Mikulas Patocka 2018-04-24 17:00 ` Mikulas Patocka 2018-04-24 17:03 ` Michal Hocko 2018-04-24 17:03 ` Michal Hocko 2018-04-24 17:28 ` Mikulas Patocka 2018-04-24 17:38 ` Michal Hocko 2018-04-24 17:38 ` Michal Hocko 2018-04-25 20:02 ` [PATCH] fault-injection: reorder config entries Mikulas Patocka 2018-04-25 20:02 ` Mikulas Patocka 2018-04-26 3:21 ` Randy Dunlap 2018-04-26 3:21 ` Randy Dunlap 2018-04-25 20:02 ` [PATCH v4] fault-injection: introduce kvmalloc fallback options Mikulas Patocka 2018-04-25 20:02 ` Mikulas Patocka 2018-04-25 20:20 ` Randy Dunlap 2018-04-25 20:20 ` Randy Dunlap 2018-04-25 20:57 ` [PATCH v5] " Mikulas Patocka 2018-04-25 20:57 ` Mikulas Patocka 2018-04-25 21:11 ` Randy Dunlap 2018-04-25 21:11 ` Randy Dunlap 2018-04-25 21:18 ` David Rientjes 2018-04-25 21:22 ` Mikulas Patocka 2018-04-25 21:22 ` Mikulas Patocka 2018-04-25 22:17 ` [dm-devel] " James Bottomley 2018-04-25 22:17 ` James Bottomley 2018-04-25 22:42 ` Mikulas Patocka 2018-04-25 22:42 ` Mikulas Patocka 2018-04-25 22:49 ` David Rientjes 2018-04-25 22:56 ` Mikulas Patocka 2018-04-25 22:56 ` Mikulas Patocka 2018-04-26 12:58 ` Michal Hocko 2018-04-26 12:58 ` Michal Hocko 2018-04-26 14:28 ` Mikulas Patocka 2018-04-26 14:28 ` Mikulas Patocka 2018-04-26 14:45 ` [dm-devel] " James Bottomley 2018-04-26 14:45 ` James Bottomley 2018-04-26 15:05 ` Mikulas Patocka 2018-04-26 15:05 ` Mikulas Patocka 2018-04-26 15:24 ` James Bottomley 2018-04-26 15:24 ` James Bottomley 2018-04-26 15:44 ` Mikulas Patocka 2018-04-26 15:44 ` Mikulas Patocka 2018-04-26 15:44 ` Mikulas Patocka 2018-04-26 15:59 ` Michael S. Tsirkin 2018-04-26 15:59 ` Michael S. Tsirkin 2018-04-26 16:07 ` Mikulas Patocka 2018-04-26 16:07 ` Mikulas Patocka 2018-04-26 18:49 ` Michael S. Tsirkin 2018-04-26 18:54 ` Mikulas Patocka 2018-04-26 18:54 ` Mikulas Patocka 2018-04-26 19:14 ` Michael S. Tsirkin 2018-04-26 19:14 ` Michael S. Tsirkin 2018-04-26 19:36 ` Mikulas Patocka 2018-04-26 19:36 ` Mikulas Patocka 2018-04-26 19:45 ` Michael S. Tsirkin 2018-04-26 19:45 ` Michael S. Tsirkin 2018-04-26 20:05 ` Mikulas Patocka 2018-04-26 20:05 ` Mikulas Patocka 2018-04-26 18:49 ` Michael S. Tsirkin 2018-04-26 16:07 ` Mikulas Patocka 2018-04-26 18:58 ` Mikulas Patocka 2018-04-26 18:58 ` Mikulas Patocka 2018-04-26 19:05 ` Michael S. Tsirkin 2018-04-26 19:05 ` Michael S. Tsirkin 2018-04-26 15:59 ` Michael S. Tsirkin 2018-04-26 15:55 ` Mikulas Patocka 2018-04-26 15:55 ` Mikulas Patocka 2018-04-26 15:05 ` Mikulas Patocka 2018-04-26 14:28 ` Mikulas Patocka 2018-04-25 22:42 ` Mikulas Patocka 2018-04-25 23:00 ` Mikulas Patocka 2018-04-25 23:00 ` Mikulas Patocka 2018-04-25 23:00 ` Mikulas Patocka 2018-04-25 23:08 ` James Bottomley 2018-04-25 23:08 ` James Bottomley 2018-04-26 14:55 ` Mikulas Patocka 2018-04-26 14:55 ` Mikulas Patocka 2018-04-26 14:55 ` Mikulas Patocka 2018-04-26 14:55 ` Mikulas Patocka 2018-04-26 15:22 ` Mikulas Patocka 2018-04-26 15:22 ` Mikulas Patocka 2018-04-26 15:22 ` Mikulas Patocka 2018-04-26 18:58 ` John Stoffel 2018-04-26 21:50 ` Mikulas Patocka 2018-04-26 21:50 ` Mikulas Patocka 2018-04-26 22:21 ` Michael S. Tsirkin 2018-04-26 22:52 ` Mikulas Patocka 2018-04-26 22:52 ` Mikulas Patocka 2018-04-27 8:25 ` Michal Hocko 2018-04-27 8:25 ` Michal Hocko 2018-04-27 10:20 ` Mikulas Patocka 2018-04-27 10:20 ` Mikulas Patocka 2018-04-27 23:20 ` Mikulas Patocka 2018-04-27 23:20 ` Mikulas Patocka 2018-04-26 22:21 ` Michael S. Tsirkin 2018-04-30 18:27 ` John Stoffel 2018-04-30 21:07 ` Mikulas Patocka 2018-05-02 13:38 ` John Stoffel 2018-05-03 17:40 ` Mikulas Patocka 2018-05-03 17:40 ` Mikulas Patocka 2018-04-30 21:07 ` Mikulas Patocka 2018-04-24 17:28 ` [PATCH v3] kvmalloc: always use vmalloc if CONFIG_DEBUG_SG Mikulas Patocka 2018-04-24 16:29 ` Michal Hocko 2018-04-24 15:50 ` Mikulas Patocka 2018-04-24 0:25 ` [PATCH] kvmalloc: always use vmalloc if CONFIG_DEBUG_VM Mikulas Patocka 2018-04-24 13:31 ` Michal Hocko 2018-04-24 13:31 ` Michal Hocko 2018-04-24 15:30 ` Mikulas Patocka 2018-04-24 15:30 ` Mikulas Patocka 2018-04-24 16:12 ` Michal Hocko 2018-04-24 16:12 ` Michal Hocko 2018-04-24 16:29 ` Michal Hocko 2018-04-24 16:29 ` Michal Hocko 2018-04-24 16:33 ` Mikulas Patocka 2018-04-24 16:33 ` Mikulas Patocka 2018-05-02 0:36 ` Andrew Morton 2018-05-02 0:36 ` Andrew Morton 2018-05-02 13:33 ` Mike Snitzer 2018-05-02 13:40 ` [dm-devel] " John Stoffel 2018-05-02 13:33 ` Mike Snitzer 2018-05-03 17:32 ` [PATCH] " Mikulas Patocka 2018-05-03 17:32 ` Mikulas Patocka 2018-04-24 0:25 ` Mikulas Patocka 2018-04-21 14:47 ` Matthew Wilcox 2018-04-20 21:21 ` Mikulas Patocka 2018-04-20 20:54 ` Mikulas Patocka 2018-04-18 17:53 ` [PATCH] net: don't use kvzalloc for DMA memory Mikulas Patocka 2018-04-18 17:46 ` David Miller 2018-04-18 17:45 ` David Miller 2018-04-18 17:55 ` Michael S. Tsirkin 2018-04-18 20:38 ` Eric Dumazet 2018-04-19 4:00 ` Michael S. Tsirkin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180424125121.GA17484@dhcp22.suse.cz \ --to=mhocko@kernel.org \ --cc=akpm@linux-foundation.org \ --cc=davem@davemloft.net \ --cc=dm-devel@redhat.com \ --cc=edumazet@google.com \ --cc=eric.dumazet@gmail.com \ --cc=jasowang@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mpatocka@redhat.com \ --cc=mst@redhat.com \ --cc=netdev@vger.kernel.org \ --cc=vbabka@suse.cz \ --cc=virtualization@lists.linux-foundation.org \ --cc=willy@infradead.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.