From: Liang Chen <liangchen.linux@gmail.com> To: "Michael S. Tsirkin" <mst@redhat.com> Cc: jasowang@redhat.com, virtualization@lists.linux-foundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, xuanzhuo@linux.alibaba.com, kuba@kernel.org, edumazet@google.com, davem@davemloft.net, pabeni@redhat.com, alexander.duyck@gmail.com Subject: Re: [PATCH net-next 1/5] virtio_net: Fix an unsafe reference to the page chain Date: Mon, 29 May 2023 15:25:04 +0800 [thread overview] Message-ID: <CAKhg4tLA9z0rMEzRJ8tRt+1jQtS9z74Se2Wvwc5SKEk=WgSd2Q@mail.gmail.com> (raw) In-Reply-To: <20230528021008-mutt-send-email-mst@kernel.org> On Sun, May 28, 2023 at 2:16 PM Michael S. Tsirkin <mst@redhat.com> wrote: > > On Fri, May 26, 2023 at 01:46:17PM +0800, Liang Chen wrote: > > "private" of buffer page is currently used for big mode to chain pages. > > But in mergeable mode, that offset of page could mean something else, > > e.g. when page_pool page is used instead. So excluding mergeable mode to > > avoid such a problem. > > > > Signed-off-by: Liang Chen <liangchen.linux@gmail.com> > > Ugh the subject makes it looks like current code has a problem > but I don't think so because I don't think anything besides > big packets uses page->private. > > The reason patch is needed is because follow up patches > use page_pool. > pls adjust commit log and subject to make all this clear. > > > > --- > > drivers/net/virtio_net.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > > index 5a7f7a76b920..c5dca0d92e64 100644 > > --- a/drivers/net/virtio_net.c > > +++ b/drivers/net/virtio_net.c > > @@ -497,7 +497,7 @@ static struct sk_buff *page_to_skb(struct virtnet_info *vi, > > return NULL; > > > > page = (struct page *)page->private; > > - if (page) > > + if (!vi->mergeable_rx_bufs && page) > > To be safe let's limit to big packets too: > > if (!vi->mergeable_rx_bufs && vi->big_packets && page) > > > Sure, thanks! > > give_pages(rq, page); > > goto ok; > > } > > -- > > 2.31.1 >
WARNING: multiple messages have this Message-ID (diff)
From: Liang Chen <liangchen.linux@gmail.com> To: "Michael S. Tsirkin" <mst@redhat.com> Cc: xuanzhuo@linux.alibaba.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, alexander.duyck@gmail.com, virtualization@lists.linux-foundation.org, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, davem@davemloft.net Subject: Re: [PATCH net-next 1/5] virtio_net: Fix an unsafe reference to the page chain Date: Mon, 29 May 2023 15:25:04 +0800 [thread overview] Message-ID: <CAKhg4tLA9z0rMEzRJ8tRt+1jQtS9z74Se2Wvwc5SKEk=WgSd2Q@mail.gmail.com> (raw) In-Reply-To: <20230528021008-mutt-send-email-mst@kernel.org> On Sun, May 28, 2023 at 2:16 PM Michael S. Tsirkin <mst@redhat.com> wrote: > > On Fri, May 26, 2023 at 01:46:17PM +0800, Liang Chen wrote: > > "private" of buffer page is currently used for big mode to chain pages. > > But in mergeable mode, that offset of page could mean something else, > > e.g. when page_pool page is used instead. So excluding mergeable mode to > > avoid such a problem. > > > > Signed-off-by: Liang Chen <liangchen.linux@gmail.com> > > Ugh the subject makes it looks like current code has a problem > but I don't think so because I don't think anything besides > big packets uses page->private. > > The reason patch is needed is because follow up patches > use page_pool. > pls adjust commit log and subject to make all this clear. > > > > --- > > drivers/net/virtio_net.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > > index 5a7f7a76b920..c5dca0d92e64 100644 > > --- a/drivers/net/virtio_net.c > > +++ b/drivers/net/virtio_net.c > > @@ -497,7 +497,7 @@ static struct sk_buff *page_to_skb(struct virtnet_info *vi, > > return NULL; > > > > page = (struct page *)page->private; > > - if (page) > > + if (!vi->mergeable_rx_bufs && page) > > To be safe let's limit to big packets too: > > if (!vi->mergeable_rx_bufs && vi->big_packets && page) > > > Sure, thanks! > > give_pages(rq, page); > > goto ok; > > } > > -- > > 2.31.1 > _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
next prev parent reply other threads:[~2023-05-29 7:25 UTC|newest] Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-05-26 5:46 [PATCH net-next 1/5] virtio_net: Fix an unsafe reference to the page chain Liang Chen 2023-05-26 5:46 ` Liang Chen 2023-05-26 5:46 ` [PATCH net-next 2/5] virtio_net: Add page_pool support to improve performance Liang Chen 2023-05-26 5:46 ` Liang Chen 2023-05-26 6:50 ` Jason Wang 2023-05-26 6:50 ` Jason Wang 2023-05-27 12:35 ` Liang Chen 2023-05-27 12:35 ` Liang Chen 2023-05-28 6:40 ` Michael S. Tsirkin 2023-05-28 6:40 ` Michael S. Tsirkin 2023-05-29 7:28 ` Liang Chen 2023-05-29 7:28 ` Liang Chen 2023-05-31 3:10 ` Xuan Zhuo 2023-05-31 3:10 ` Xuan Zhuo 2023-06-07 9:11 ` Liang Chen 2023-06-07 9:11 ` Liang Chen 2023-06-07 9:33 ` Xuan Zhuo 2023-06-07 9:33 ` Xuan Zhuo 2023-05-26 16:11 ` kernel test robot 2023-05-26 16:11 ` kernel test robot 2023-05-28 6:27 ` Michael S. Tsirkin 2023-05-28 6:27 ` Michael S. Tsirkin 2023-05-29 7:28 ` Liang Chen 2023-05-29 7:28 ` Liang Chen 2023-05-28 6:20 ` Michael S. Tsirkin 2023-05-28 6:20 ` Michael S. Tsirkin 2023-05-29 7:27 ` Liang Chen 2023-05-29 7:27 ` Liang Chen 2023-05-29 9:55 ` Michael S. Tsirkin 2023-05-29 9:55 ` Michael S. Tsirkin 2023-05-30 1:19 ` Liang Chen 2023-05-30 1:19 ` Liang Chen 2023-06-07 9:08 ` Liang Chen 2023-06-07 9:08 ` Liang Chen 2023-06-07 9:35 ` Xuan Zhuo 2023-06-07 9:35 ` Xuan Zhuo 2023-06-07 13:58 ` Liang Chen 2023-06-07 13:58 ` Liang Chen 2023-06-07 20:17 ` Michael S. Tsirkin 2023-06-07 20:17 ` Michael S. Tsirkin 2023-06-08 0:38 ` Jason Wang 2023-06-08 0:38 ` Jason Wang 2023-06-08 3:54 ` Xuan Zhuo 2023-06-08 3:54 ` Xuan Zhuo 2023-06-09 2:57 ` Liang Chen 2023-06-09 2:57 ` Liang Chen 2023-07-05 5:41 ` Liang Chen 2023-07-05 5:41 ` Liang Chen 2023-07-05 6:04 ` Jason Wang 2023-07-05 6:04 ` Jason Wang 2023-07-06 1:01 ` Liang Chen 2023-07-06 1:01 ` Liang Chen 2023-06-09 2:57 ` Liang Chen 2023-06-09 2:57 ` Liang Chen 2023-11-29 14:50 ` Zhu Yanjun 2023-11-29 14:59 ` Michael S. Tsirkin 2023-11-29 15:22 ` Zhu Yanjun 2023-11-29 15:29 ` Zhu Yanjun 2023-11-30 2:34 ` Xuan Zhuo 2023-11-30 5:30 ` Zhu Yanjun 2023-12-01 1:38 ` Xuan Zhuo 2023-12-04 5:24 ` Zhu Yanjun 2023-11-30 7:17 ` Zhu Yanjun 2023-05-26 5:46 ` [PATCH net-next 3/5] virtio_net: Add page pool fragmentation support Liang Chen 2023-05-26 5:46 ` Liang Chen 2023-05-26 8:29 ` Horatiu Vultur 2023-05-27 12:36 ` Liang Chen 2023-05-27 12:36 ` Liang Chen 2023-05-26 17:44 ` kernel test robot 2023-05-26 17:44 ` kernel test robot 2023-05-28 6:25 ` Michael S. Tsirkin 2023-05-28 6:25 ` Michael S. Tsirkin 2023-05-29 7:29 ` Liang Chen 2023-05-29 7:29 ` Liang Chen 2023-05-29 1:33 ` Yunsheng Lin 2023-05-29 7:30 ` Liang Chen 2023-05-29 7:30 ` Liang Chen 2023-05-26 5:46 ` [PATCH net-next 4/5] virtio_ring: Introduce DMA pre-handler Liang Chen 2023-05-26 5:46 ` Liang Chen 2023-05-26 6:57 ` Jason Wang 2023-05-26 6:57 ` Jason Wang 2023-05-26 5:46 ` [PATCH net-next 5/5] virtio_net: Implement " Liang Chen 2023-05-26 5:46 ` Liang Chen 2023-05-26 7:06 ` Jason Wang 2023-05-26 7:06 ` Jason Wang 2023-05-27 12:35 ` Liang Chen 2023-05-27 12:35 ` Liang Chen 2023-05-26 17:34 ` kernel test robot 2023-05-26 17:34 ` kernel test robot 2023-05-26 6:38 ` [PATCH net-next 1/5] virtio_net: Fix an unsafe reference to the page chain Jason Wang 2023-05-26 6:38 ` Jason Wang 2023-05-27 12:33 ` Liang Chen 2023-05-27 12:33 ` Liang Chen 2023-05-28 6:29 ` Michael S. Tsirkin 2023-05-28 6:29 ` Michael S. Tsirkin 2023-05-29 7:25 ` Liang Chen 2023-05-29 7:25 ` Liang Chen 2023-05-28 6:16 ` Michael S. Tsirkin 2023-05-28 6:16 ` Michael S. Tsirkin 2023-05-29 7:25 ` Liang Chen [this message] 2023-05-29 7:25 ` Liang Chen
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='CAKhg4tLA9z0rMEzRJ8tRt+1jQtS9z74Se2Wvwc5SKEk=WgSd2Q@mail.gmail.com' \ --to=liangchen.linux@gmail.com \ --cc=alexander.duyck@gmail.com \ --cc=davem@davemloft.net \ --cc=edumazet@google.com \ --cc=jasowang@redhat.com \ --cc=kuba@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mst@redhat.com \ --cc=netdev@vger.kernel.org \ --cc=pabeni@redhat.com \ --cc=virtualization@lists.linux-foundation.org \ --cc=xuanzhuo@linux.alibaba.com \ /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.