Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/net need to membar_producer() *before* switching.



details:   https://anonhg.NetBSD.org/src/rev/ea0d380c1750
branches:  trunk
changeset: 822257:ea0d380c1750
user:      ryo <ryo%NetBSD.org@localhost>
date:      Fri Mar 10 07:35:58 2017 +0000

description:
need to membar_producer() *before* switching.

pointed out by riastradh@, thanks

diffstat:

 sys/net/pfil.c |  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diffs (28 lines):

diff -r 96ad225e24dd -r ea0d380c1750 sys/net/pfil.c
--- a/sys/net/pfil.c    Fri Mar 10 04:13:43 2017 +0000
+++ b/sys/net/pfil.c    Fri Mar 10 07:35:58 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: pfil.c,v 1.34 2017/01/23 02:32:54 ozaki-r Exp $        */
+/*     $NetBSD: pfil.c,v 1.35 2017/03/10 07:35:58 ryo Exp $    */
 
 /*
  * Copyright (c) 2013 Mindaugas Rasiukevicius <rmind at NetBSD org>
@@ -28,7 +28,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pfil.c,v 1.34 2017/01/23 02:32:54 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pfil.c,v 1.35 2017/03/10 07:35:58 ryo Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_net_mpsafe.h"
@@ -232,8 +232,8 @@
        pfh->pfil_arg  = arg;
 
        /* switch from oldlist to newlist */
+       membar_producer();
        phlistset->active = newlist;
-       membar_producer();
 #ifdef NET_MPSAFE
        pserialize_perform(pfil_psz);
 #endif



Home | Main Index | Thread Index | Old Index