RE: [PATCH net-next 1/3] selftests: lan743x: Add testfile for lan743x network driver

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

 



Hi Brett,

Thank you for the review comments.

> EXTERNAL EMAIL: Do not click links or open attachments unless you know the
> content is safe
> 
> On 9/3/2024 3:15 PM, Mohan Prasad J wrote:
> > Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
> >
> >
> > Add testfile for lan743x network driver.
> > Testfile includes the verification of status of autonegotiation.
> > Ksft modules and ethtool are used for the testing.
> > net/lib libraries are included for testing.
> > Add the __init__.py file.
> > Include /microchip/lan743x as a target in Makefile.
> > Updated MAINTAINERS list.
> >
> > Signed-off-by: Mohan Prasad J <mohan.prasad@xxxxxxxxxxxxx>
> > ---
> >   MAINTAINERS                                   |  2 +
> >   tools/testing/selftests/Makefile              |  2 +-
> >   .../drivers/net/hw/microchip/lan743x/Makefile |  7 +++
> >   .../net/hw/microchip/lan743x/lan743x.py       | 51 +++++++++++++++++++
> >   .../hw/microchip/lan743x/lib/py/__init__.py   | 16 ++++++
> >   5 files changed, 77 insertions(+), 1 deletion(-)
> >   create mode 100644
> tools/testing/selftests/drivers/net/hw/microchip/lan743x/Makefile
> >   create mode 100755
> tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> >   create mode 100644
> > tools/testing/selftests/drivers/net/hw/microchip/lan743x/lib/py/__init
> > __.py
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS index baf88e74c..461f94ae0
> > 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14960,10 +14960,12 @@ F:    net/dsa/tag_ksz.c
> >
> >   MICROCHIP LAN743X ETHERNET DRIVER
> >   M:     Bryan Whitehead <bryan.whitehead@xxxxxxxxxxxxx>
> > +M:     Mohan Prasad J <mohan.prasad@xxxxxxxxxxxxx>
> 
> It seems like updating the maintainers list should be a separate patch.

I will update it in the next version.

> 
> Thanks,
> 
> Brett
> 
> >   M:     UNGLinuxDriver@xxxxxxxxxxxxx
> >   L:     netdev@xxxxxxxxxxxxxxx
> >   S:     Maintained
> >   F:     drivers/net/ethernet/microchip/lan743x_*
> > +F:     tools/testing/selftests/drivers/net/hw/microchip/lan743x/
> >
> >   MICROCHIP LAN87xx/LAN937x T1 PHY DRIVER
> >   M:     Arun Ramadoss <arun.ramadoss@xxxxxxxxxxxxx>
> > diff --git a/tools/testing/selftests/Makefile
> > b/tools/testing/selftests/Makefile
> > index a5f1c0c27..8059529c9 100644
> > --- a/tools/testing/selftests/Makefile
> > +++ b/tools/testing/selftests/Makefile
> > @@ -122,7 +122,7 @@ TARGETS_HOTPLUG = cpu-hotplug
> >   TARGETS_HOTPLUG += memory-hotplug
> >
> >   # Networking tests want the net/lib target, include it automatically
> > -ifneq ($(filter net drivers/net drivers/net/hw,$(TARGETS)),)
> > +ifneq ($(filter net drivers/net drivers/net/hw
> > +drivers/net/hw/microchip/lan743x,$(TARGETS)),)
> >   ifeq ($(filter net/lib,$(TARGETS)),)
> >          INSTALL_DEP_TARGETS := net/lib
> >   endif
> > diff --git
> > a/tools/testing/selftests/drivers/net/hw/microchip/lan743x/Makefile
> > b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/Makefile
> > new file mode 100644
> > index 000000000..542128678
> > --- /dev/null
> > +++ b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/Makefil
> > +++ e
> > @@ -0,0 +1,7 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +
> > +TEST_INCLUDES := $(wildcard lib/py/*.py ../../../lib/py/*.py)
> > +
> > +TEST_PROGS := lan743x.py
> > +
> > +include ../../../../../lib.mk
> > diff --git
> > a/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> > b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x.py
> > new file mode 100755
> > index 000000000..f1ad97dc2
> > --- /dev/null
> > +++ b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lan743x
> > +++ .py
> > @@ -0,0 +1,51 @@
> > +#!/usr/bin/env python3
> > +# SPDX-License-Identifier: GPL-2.0
> > +
> > +import time
> > +import re
> > +from lib.py import ksft_run, ksft_exit, ksft_pr, ksft_eq from lib.py
> > +import KsftFailEx, KsftSkipEx from lib.py import NetDrvEpEnv from
> > +lib.py import cmd from lib.py import ethtool
> > +
> > +def verify_link_up(ifname: str) -> None:
> > +    """Verify whether the link is up initially"""
> > +    with open(f"/sys/class/net/{ifname}/operstate", "r") as fp:
> > +        link_state = fp.read().strip()
> > +
> > +    if link_state == "down":
> > +        raise KsftSkipEx(f"Link state of interface {ifname} is DOWN")
> > +
> > +def set_autonegotiation_state(ifname: str, state: str) -> None:
> > +    """Set the autonegotiation state for the interface"""
> > +    process = ethtool(f"-s {ifname} speed 10 duplex half autoneg {state}")
> > +    if process.ret != 0:
> > +        raise KsftFailEx(f"Not able to set autoneg parameter for {ifname}")
> > +    ksft_pr(f"Autoneg set as {state} for {ifname}")
> > +
> > +def verify_autonegotiation(ifname: str, expected_state: str) -> None:
> > +    verify_link_up(ifname)
> > +    """Verifying the autonegotiation state"""
> > +    output = ethtool(f"{ifname}")
> > +    autoneg_match = re.search(r'Auto-negotiation:\s+(\w+)',
> > +output.stdout)
> > +
> > +    if not autoneg_match:
> > +        raise KsftFailEx("Failed to find autonegotiation information
> > + in ethtool output.")
> > +
> > +    actual_state = autoneg_match.group(1)
> > +    ksft_eq(actual_state, expected_state)
> > +
> > +def test_autonegotiation(cfg) -> None:
> > +    for state in ["off", "on"]:
> > +        set_autonegotiation_state(cfg.ifname, state)
> > +        time.sleep(5)
> > +        verify_autonegotiation(cfg.ifname, state)
> > +
> > +def main() -> None:
> > +    with NetDrvEpEnv(__file__) as cfg:
> > +        ksft_run(globs=globals(), case_pfx={"test_"}, args=(cfg,))
> > +    ksft_exit()
> > +
> > +if __name__ == "__main__":
> > +    main()
> > diff --git
> > a/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lib/py/__in
> > it__.py
> > b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lib/py/__in
> > it__.py
> > new file mode 100644
> > index 000000000..e571631af
> > --- /dev/null
> > +++ b/tools/testing/selftests/drivers/net/hw/microchip/lan743x/lib/py/
> > +++ __init__.py
> > @@ -0,0 +1,16 @@
> > +# SPDX-License-Identifier: GPL-2.0
> > +
> > +import sys
> > +from pathlib import Path
> > +
> > +KSFT_DIR = (Path(__file__).parent / "../../../../../../..").resolve()
> > +
> > +try:
> > +    sys.path.append(KSFT_DIR.as_posix())
> > +    from net.lib.py import *
> > +    from drivers.net.lib.py import *
> > +except ModuleNotFoundError as e:
> > +    ksft_pr("Failed importing `net` library from kernel sources")
> > +    ksft_pr(str(e))
> > +    ktap_result(True, comment="SKIP")
> > +    sys.exit(4)
> > --
> > 2.43.0
> >




[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux