From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9DEF1425E6; Wed, 20 Sep 2023 06:35:24 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 25D1E402EE; Wed, 20 Sep 2023 06:35:24 +0200 (CEST) Received: from mail-vs1-f51.google.com (mail-vs1-f51.google.com [209.85.217.51]) by mails.dpdk.org (Postfix) with ESMTP id 620C9402DD for ; Wed, 20 Sep 2023 06:35:22 +0200 (CEST) Received: by mail-vs1-f51.google.com with SMTP id ada2fe7eead31-452951b27d0so486133137.2 for ; Tue, 19 Sep 2023 21:35:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1695184522; x=1695789322; darn=dpdk.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=lZ+MRThVEKzHIRXV/jyVZiBgk5GElzmibA+z6IGtmMo=; b=LY8WUI8CZDAvCy+kgIiaUh8kaN88vbFwOxQrgM9jOo35M+mXnKOIEaeisvtM/UyLIj 7gXqv3eXSOIpeGcQfIzZYUQ9iaORw/i94L3DvQllEeKrp30pONGdsutKpeqyROM+P9N5 p30GcIHmhUPtIcCgrAm3swo+Ct2hTisFwSpfhaxZsAZh+ETsdM3GjE9C1Oylhnu9dSZd 2H2bZSaCUjpzP+Ken0NSBdXI7Rw04XLMeNIkg5anzwJ33wCbL4FTeOcO+p26dvj1JAHW /3S8cMmtWLbYVMIVFsJ+DoilfqEOS5Rf24lvYMZyJRtibNjVHKuisYZ1E2APadg9Pv1I 0vmA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1695184522; x=1695789322; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=lZ+MRThVEKzHIRXV/jyVZiBgk5GElzmibA+z6IGtmMo=; b=iu8J40FxpHdBBTmjMxMSK3VbUdPLILoi4g0BfrrY3HShr+bOvXbFg3qamtFRAqBteN YC6g1XaKmGEV/37Tcxb9P3ZSyaTbR9ERghBEaQDZONjh//X2fSCY4DF2nd86RnwExLgN xz1Hqevz9ZSqu91rLCZtjHGVnoHd4mbf/F9i29vr4FBKCiyAnInWqR/egFkx58hM23j6 hI/d9h9MV4SAvp/wL9KUMDvkf3PipTCGIZUN3wN6e2s6yWMp0aUdLCRT8vPVJcAtsW/l RjSK45BzH5YI+Xe5Q8p/oqy52ntv9JC7aI1skjqAifHQJEAbwmLQpo9pBHmjWQDfnxtu Mf7w== X-Gm-Message-State: AOJu0YywqBn7IHAys0hCrIQb/WrCI7ZzUH/LLlv7XdEkadyVfhApF+i4 NhipJzUuZef+bv7KsUWJb1km68UG3TH0jrXwTNo= X-Google-Smtp-Source: AGHT+IGBfcPsX0GwQrLCNNHJ+896ceX4W/p95rD377oqn5tjgENQPSt0kLbA4QbvyC4MuYxPdL7JQI8/Jr8jogFLD4Q= X-Received: by 2002:a67:e9c9:0:b0:452:6451:5e16 with SMTP id q9-20020a67e9c9000000b0045264515e16mr1512474vso.31.1695184521778; Tue, 19 Sep 2023 21:35:21 -0700 (PDT) MIME-Version: 1.0 References: <20230908104907.4060511-1-skori@marvell.com> <20230919160455.1678716-1-skori@marvell.com> <20230919160455.1678716-2-skori@marvell.com> In-Reply-To: <20230919160455.1678716-2-skori@marvell.com> From: Jerin Jacob Date: Wed, 20 Sep 2023 10:04:55 +0530 Message-ID: Subject: Re: [PATCH v4 02/14] app/graph: add telnet connectivity framework To: skori@marvell.com Cc: Rakesh Kudurumalla , dev@dpdk.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Tue, Sep 19, 2023 at 9:35=E2=80=AFPM wrote: > > From: Sunil Kumar Kori > > It adds framework to initate a telnet session with application. > > Some configurations and debug commands are exposed as runtime APIs. > Those commands can be invoked using telnet session. > > Application initiates a telnet server with host address 0.0.0.0 > and port number 8086 by default. > > To make it configurable, "-h" and "-p" options are provided. > Using them user can pass host address and port number on which > application will start telnet server. > > Using same host address and port number, telnet client can connect > to application. > > Syntax to connect with application: > # telnet > > Once session is connected, "graph> " prompt will be available. > Example: > # telnet 10.28.35.207 50000 > Trying 10.28.35.207... > Connected to 10.28.35.207. > Escape character is '^]'. > > Welcome! > > graph> Some improvements 1) Please squash 14/14 patch to 1/14. 2) Ctrl - C doesn't work which is a serious issue. We have to kill it via kill -9 from a separate window. This is probably because of command line library. 3). In case app launch fails due to error, it will leave terminal into a bad state until "reset" command is executed. This might also be because of command line library. 4). Just a wishlist: If on terminal console I could do a tab and get commands help, just like testpmd console.