Message ID | 20230907171540.36736-1-atrajeev@linux.vnet.ibm.com (mailing list archive) |
---|---|
Headers | show |
Series | Fix for shellcheck issues with version "0.6" | expand |
On Thu, Sep 7, 2023 at 10:17 AM Athira Rajeev <atrajeev@linux.vnet.ibm.com> wrote: > > From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> > > shellcheck was run on perf tool shell scripts s a pre-requisite > to include a build option for shellcheck discussed here: > https://www.spinics.net/lists/linux-perf-users/msg25553.html > > And fixes were added for the coding/formatting issues in > two patchsets: > https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ > https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ > > Three additional issues are observed with shellcheck "0.6" and > this patchset covers those. With this patchset, > > # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done > # echo $? > 0 > > Athira Rajeev (3): > tests/shell: Fix shellcheck SC1090 to handle the location of sourced > files > tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh > tetscase > tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts Series: Tested-by: Ian Rogers <irogers@google.com> Thanks, Ian > tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ > tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ > tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ > tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ > tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ > tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ > tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ > tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ > tools/perf/tests/shell/record.sh | 1 + > tools/perf/tests/shell/stat+csv_output.sh | 1 + > tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- > tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- > tools/perf/tests/shell/stat+std_output.sh | 1 + > tools/perf/tests/shell/test_intel_pt.sh | 1 + > tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + > 15 files changed, 35 insertions(+), 4 deletions(-) > > -- > 2.31.1 >
> On 08-Sep-2023, at 5:20 AM, Ian Rogers <irogers@google.com> wrote: > > On Thu, Sep 7, 2023 at 10:17 AM Athira Rajeev > <atrajeev@linux.vnet.ibm.com> wrote: >> >> From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> >> >> shellcheck was run on perf tool shell scripts s a pre-requisite >> to include a build option for shellcheck discussed here: >> https://www.spinics.net/lists/linux-perf-users/msg25553.html >> >> And fixes were added for the coding/formatting issues in >> two patchsets: >> https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ >> https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ >> >> Three additional issues are observed with shellcheck "0.6" and >> this patchset covers those. With this patchset, >> >> # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done >> # echo $? >> 0 >> >> Athira Rajeev (3): >> tests/shell: Fix shellcheck SC1090 to handle the location of sourced >> files >> tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh >> tetscase >> tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts > > Series: > Tested-by: Ian Rogers <irogers@google.com> > > Thanks, > Ian Thanks Ian for checking the patch series Athira > >> tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ >> tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ >> tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ >> tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ >> tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ >> tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ >> tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ >> tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ >> tools/perf/tests/shell/record.sh | 1 + >> tools/perf/tests/shell/stat+csv_output.sh | 1 + >> tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- >> tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- >> tools/perf/tests/shell/stat+std_output.sh | 1 + >> tools/perf/tests/shell/test_intel_pt.sh | 1 + >> tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + >> 15 files changed, 35 insertions(+), 4 deletions(-) >> >> -- >> 2.31.1
> On 08-Sep-2023, at 7:47 PM, Athira Rajeev <atrajeev@linux.vnet.ibm.com> wrote: > > > >> On 08-Sep-2023, at 5:20 AM, Ian Rogers <irogers@google.com> wrote: >> >> On Thu, Sep 7, 2023 at 10:17 AM Athira Rajeev >> <atrajeev@linux.vnet.ibm.com> wrote: >>> >>> From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> >>> >>> shellcheck was run on perf tool shell scripts s a pre-requisite >>> to include a build option for shellcheck discussed here: >>> https://www.spinics.net/lists/linux-perf-users/msg25553.html >>> >>> And fixes were added for the coding/formatting issues in >>> two patchsets: >>> https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ >>> https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ >>> >>> Three additional issues are observed with shellcheck "0.6" and >>> this patchset covers those. With this patchset, >>> >>> # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done >>> # echo $? >>> 0 >>> >>> Athira Rajeev (3): >>> tests/shell: Fix shellcheck SC1090 to handle the location of sourced >>> files >>> tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh >>> tetscase >>> tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts >> >> Series: >> Tested-by: Ian Rogers <irogers@google.com> >> >> Thanks, >> Ian > > Thanks Ian for checking the patch series > > Athira Hi Arnaldo Can you please check and pull this if it looks good to go . Thanks Athira >> >>> tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ >>> tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ >>> tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ >>> tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ >>> tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ >>> tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ >>> tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ >>> tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ >>> tools/perf/tests/shell/record.sh | 1 + >>> tools/perf/tests/shell/stat+csv_output.sh | 1 + >>> tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- >>> tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- >>> tools/perf/tests/shell/stat+std_output.sh | 1 + >>> tools/perf/tests/shell/test_intel_pt.sh | 1 + >>> tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + >>> 15 files changed, 35 insertions(+), 4 deletions(-) >>> >>> -- >>> 2.31.1
On 9/7/23 22:45, Athira Rajeev wrote: > From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> > > shellcheck was run on perf tool shell scripts s a pre-requisite > to include a build option for shellcheck discussed here: > https://www.spinics.net/lists/linux-perf-users/msg25553.html > > And fixes were added for the coding/formatting issues in > two patchsets: > https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ > https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ > > Three additional issues are observed with shellcheck "0.6" and > this patchset covers those. With this patchset, > > # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done > # echo $? > 0 > Patchset looks good to me. Reviewed-by: Kajol Jain <kjain@linux.ibm.com> Thanks, Kajol Jain > Athira Rajeev (3): > tests/shell: Fix shellcheck SC1090 to handle the location of sourced > files > tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh > tetscase > tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts > > tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ > tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ > tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ > tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ > tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ > tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ > tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ > tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ > tools/perf/tests/shell/record.sh | 1 + > tools/perf/tests/shell/stat+csv_output.sh | 1 + > tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- > tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- > tools/perf/tests/shell/stat+std_output.sh | 1 + > tools/perf/tests/shell/test_intel_pt.sh | 1 + > tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + > 15 files changed, 35 insertions(+), 4 deletions(-) >
> On 25-Sep-2023, at 1:34 PM, kajoljain <kjain@linux.ibm.com> wrote: > > > > On 9/7/23 22:45, Athira Rajeev wrote: >> From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> >> >> shellcheck was run on perf tool shell scripts s a pre-requisite >> to include a build option for shellcheck discussed here: >> https://www.spinics.net/lists/linux-perf-users/msg25553.html >> >> And fixes were added for the coding/formatting issues in >> two patchsets: >> https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ >> https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ >> >> Three additional issues are observed with shellcheck "0.6" and >> this patchset covers those. With this patchset, >> >> # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done >> # echo $? >> 0 >> > > Patchset looks good to me. > > Reviewed-by: Kajol Jain <kjain@linux.ibm.com> > > Thanks, > Kajol Jain > Hi Namhyunbg, Can you please check for this patchset also Thanks Athira >> Athira Rajeev (3): >> tests/shell: Fix shellcheck SC1090 to handle the location of sourced >> files >> tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh >> tetscase >> tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts >> >> tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ >> tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ >> tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ >> tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ >> tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ >> tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ >> tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ >> tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ >> tools/perf/tests/shell/record.sh | 1 + >> tools/perf/tests/shell/stat+csv_output.sh | 1 + >> tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- >> tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- >> tools/perf/tests/shell/stat+std_output.sh | 1 + >> tools/perf/tests/shell/test_intel_pt.sh | 1 + >> tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + >> 15 files changed, 35 insertions(+), 4 deletions(-)
On Tue, Sep 26, 2023 at 9:29 PM Athira Rajeev <atrajeev@linux.vnet.ibm.com> wrote: > > > > > On 25-Sep-2023, at 1:34 PM, kajoljain <kjain@linux.ibm.com> wrote: > > > > > > > > On 9/7/23 22:45, Athira Rajeev wrote: > >> From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> > >> > >> shellcheck was run on perf tool shell scripts s a pre-requisite > >> to include a build option for shellcheck discussed here: > >> https://www.spinics.net/lists/linux-perf-users/msg25553.html > >> > >> And fixes were added for the coding/formatting issues in > >> two patchsets: > >> https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ > >> https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ > >> > >> Three additional issues are observed with shellcheck "0.6" and > >> this patchset covers those. With this patchset, > >> > >> # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done > >> # echo $? > >> 0 > >> > > > > Patchset looks good to me. > > > > Reviewed-by: Kajol Jain <kjain@linux.ibm.com> > > > > Thanks, > > Kajol Jain > > > > Hi Namhyunbg, > > Can you please check for this patchset also Sure, it's applied to perf-tools-next, thanks!
> On 28-Sep-2023, at 9:24 AM, Namhyung Kim <namhyung@kernel.org> wrote: > > On Tue, Sep 26, 2023 at 9:29 PM Athira Rajeev > <atrajeev@linux.vnet.ibm.com> wrote: >> >> >> >>> On 25-Sep-2023, at 1:34 PM, kajoljain <kjain@linux.ibm.com> wrote: >>> >>> >>> >>> On 9/7/23 22:45, Athira Rajeev wrote: >>>> From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> >>>> >>>> shellcheck was run on perf tool shell scripts s a pre-requisite >>>> to include a build option for shellcheck discussed here: >>>> https://www.spinics.net/lists/linux-perf-users/msg25553.html >>>> >>>> And fixes were added for the coding/formatting issues in >>>> two patchsets: >>>> https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ >>>> https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ >>>> >>>> Three additional issues are observed with shellcheck "0.6" and >>>> this patchset covers those. With this patchset, >>>> >>>> # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done >>>> # echo $? >>>> 0 >>>> >>> >>> Patchset looks good to me. >>> >>> Reviewed-by: Kajol Jain <kjain@linux.ibm.com> >>> >>> Thanks, >>> Kajol Jain >>> >> >> Hi Namhyunbg, >> >> Can you please check for this patchset also > > Sure, it's applied to perf-tools-next, thanks! Thanks Namhyung Athira
From: root <root@ltcden13-lp4.aus.stglabs.ibm.com> shellcheck was run on perf tool shell scripts s a pre-requisite to include a build option for shellcheck discussed here: https://www.spinics.net/lists/linux-perf-users/msg25553.html And fixes were added for the coding/formatting issues in two patchsets: https://lore.kernel.org/linux-perf-users/20230613164145.50488-1-atrajeev@linux.vnet.ibm.com/ https://lore.kernel.org/linux-perf-users/20230709182800.53002-1-atrajeev@linux.vnet.ibm.com/ Three additional issues are observed with shellcheck "0.6" and this patchset covers those. With this patchset, # for F in $(find tests/shell/ -perm -o=x -name '*.sh'); do shellcheck -S warning $F; done # echo $? 0 Athira Rajeev (3): tests/shell: Fix shellcheck SC1090 to handle the location of sourced files tests/shell: Fix shellcheck issues in tests/shell/stat+shadow_stat.sh tetscase tests/shell: Fix shellcheck warnings for SC2153 in multiple scripts tools/perf/tests/shell/coresight/asm_pure_loop.sh | 4 ++++ tools/perf/tests/shell/coresight/memcpy_thread_16k_10.sh | 4 ++++ tools/perf/tests/shell/coresight/thread_loop_check_tid_10.sh | 4 ++++ tools/perf/tests/shell/coresight/thread_loop_check_tid_2.sh | 4 ++++ tools/perf/tests/shell/coresight/unroll_loop_thread_10.sh | 4 ++++ tools/perf/tests/shell/probe_vfs_getname.sh | 2 ++ tools/perf/tests/shell/record+probe_libc_inet_pton.sh | 2 ++ tools/perf/tests/shell/record+script_probe_vfs_getname.sh | 2 ++ tools/perf/tests/shell/record.sh | 1 + tools/perf/tests/shell/stat+csv_output.sh | 1 + tools/perf/tests/shell/stat+csv_summary.sh | 4 ++-- tools/perf/tests/shell/stat+shadow_stat.sh | 4 ++-- tools/perf/tests/shell/stat+std_output.sh | 1 + tools/perf/tests/shell/test_intel_pt.sh | 1 + tools/perf/tests/shell/trace+probe_vfs_getname.sh | 1 + 15 files changed, 35 insertions(+), 4 deletions(-)