public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Saloni Kasbekar" <saloni.kasbekar@intel.com>
To: "Luo, Heng" <heng.luo@intel.com>,
	"Clark-williams, Zachary" <zachary.clark-williams@intel.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [edk2-devel] [PATCH v2] NetworkPkg: Triger regularly scan only if not connect to AP
Date: Thu, 14 Dec 2023 20:36:17 +0000	[thread overview]
Message-ID: <BY5PR11MB3861C4179A24431CA5995B83F18CA@BY5PR11MB3861.namprd11.prod.outlook.com> (raw)
In-Reply-To: <CH3PR11MB8776E4F2E26AE79C09E0FEA8938CA@CH3PR11MB8776.namprd11.prod.outlook.com>

Mike,

Would you be able to help us merge the patch?

Thanks,
Saloni

-----Original Message-----
From: Luo, Heng <heng.luo@intel.com> 
Sent: Thursday, December 14, 2023 3:20 AM
To: Clark-williams, Zachary <zachary.clark-williams@intel.com>; Kasbekar, Saloni <saloni.kasbekar@intel.com>; devel@edk2.groups.io
Subject: RE: [PATCH v2] NetworkPkg: Triger regularly scan only if not connect to AP

Hi Saloni,
Could you help to merge the patch?

Thanks,
Heng

> -----Original Message-----
> From: Luo, Heng
> Sent: Monday, December 11, 2023 4:13 PM
> To: Clark-williams, Zachary <Zachary.Clark-williams@intel.com>; 
> Kasbekar, Saloni <saloni.kasbekar@intel.com>; devel@edk2.groups.io
> Subject: RE: [PATCH v2] NetworkPkg: Triger regularly scan only if not 
> connect to AP
> 
> Hi Zack,
> Could you please review the change?
> Thanks,
> Heng
> 
> > -----Original Message-----
> > From: Kasbekar, Saloni <saloni.kasbekar@intel.com>
> > Sent: Saturday, December 2, 2023 5:25 AM
> > To: Luo, Heng <heng.luo@intel.com>; devel@edk2.groups.io
> > Cc: Clark-williams, Zachary <zachary.clark-williams@intel.com>
> > Subject: RE: [PATCH v2] NetworkPkg: Triger regularly scan only if 
> > not connect to AP
> >
> > Reviewed-by: Kasbekar, Saloni <saloni.kasbekar@intel.com>
> >
> > Thanks,
> > Saloni
> >
> > -----Original Message-----
> > From: Luo, Heng <heng.luo@intel.com>
> > Sent: Monday, November 27, 2023 7:07 PM
> > To: devel@edk2.groups.io
> > Cc: Kasbekar, Saloni <saloni.kasbekar@intel.com>; Clark-williams, 
> > Zachary <zachary.clark-williams@intel.com>
> > Subject: [PATCH v2] NetworkPkg: Triger regularly scan only if not 
> > connect to AP
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4605
> >
> > When UEFI Wi-Fi is in BSS connected state, the platform is 
> > considered as a static and Wi-Fi roaming support is not needed.
> > Wifi connection manager should not initiate Scan requests in this 
> > state affect BSS client connectivity and must be avoided.
> > Triger regularly scan only if not connect to AP.
> >
> > Signed-off-by: Heng Luo <heng.luo@intel.com>
> > Cc: Saloni Kasbekar <saloni.kasbekar@intel.com>
> > Cc: Zachary Clark-williams <zachary.clark-williams@intel.com>
> > ---
> >  NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrImpl.c | 4 ++-
> -
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git
> > a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrImpl.c
> > b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrImpl.c
> > index d1182e52bd..4c5460b65c 100644
> > --- a/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrImpl.c
> > +++ b/NetworkPkg/WifiConnectionManagerDxe/WifiConnectionMgrImpl.c
> > @@ -1506,8 +1506,8 @@ WifiMgrOnTimerTick (
> >    }    Nic->ScanTickTime++;-  if (((Nic->ScanTickTime >
> > WIFI_SCAN_FREQUENCY) || Nic->OneTimeScanRequest) &&-      (Nic-
> > >ScanState == WifiMgrScanFinished))+  if ((((Nic->ScanTickTime >
> > WIFI_SCAN_FREQUENCY) && (Nic->ConnectState !=
> WifiMgrConnectedToAp))
> > ||+       Nic->OneTimeScanRequest) && (Nic->ScanState ==
> > WifiMgrScanFinished))   {     Nic->OneTimeScanRequest = FALSE;     Nic-
> > >ScanTickTime       = 0;--
> > 2.31.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#112548): https://edk2.groups.io/g/devel/message/112548
Mute This Topic: https://groups.io/mt/102844565/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2023-12-14 20:36 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-28  3:06 [edk2-devel] [PATCH v2] NetworkPkg: Triger regularly scan only if not connect to AP Heng Luo
2023-12-01 21:24 ` Saloni Kasbekar
2023-12-11  8:12   ` Heng Luo
2023-12-14 11:19     ` Heng Luo
2023-12-14 20:36       ` Saloni Kasbekar [this message]
2024-01-02  8:54         ` Heng Luo
2024-01-03  0:54           ` 回复: " gaoliming via groups.io
2024-01-03  5:55             ` Heng Luo
2024-01-09  3:17               ` Heng Luo
     [not found]               ` <17A88F83F824C8C1.24350@groups.io>
2024-01-15  3:08                 ` Heng Luo
2024-01-15  5:01                   ` 回复: " gaoliming via groups.io
2024-01-15  5:31                     ` Heng Luo

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-list 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=BY5PR11MB3861C4179A24431CA5995B83F18CA@BY5PR11MB3861.namprd11.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox