Re: [ovs-dev] [PATCH] rhel: Fixed a bug for checking the correct major version and revision.

2019-06-09 Thread Ben Pfaff 
On Fri, Jun 07, 2019 at 03:54:38PM -0700, Gregory Rose wrote:
> On 6/7/2019 3:39 PM, Ashish Varma wrote:
> > Fixed a bug where checking for major version 3.10 and major revision not
> > equal to 327 or 693 or 957 should have gone to the default else at the end.
> > In the current code, the default else condition will not get executed
> > for kernel with major version 3.10 and major revision not equal
> > to 327/693/957 resulting in failure to load the kernel module.
> > 
> > Signed-off-by: Ashish Varma 
> > ---
> 
> Should there be a fixes tag?

Looks like it.  Ashish, if you'll supply that, I'll commit this.

Thanks,

Ben.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] rhel: Fixed a bug for checking the correct major version and revision.

2019-06-07 Thread Gregory Rose 

On 6/7/2019 3:39 PM, Ashish Varma wrote:

Fixed a bug where checking for major version 3.10 and major revision not
equal to 327 or 693 or 957 should have gone to the default else at the end.
In the current code, the default else condition will not get executed
for kernel with major version 3.10 and major revision not equal
to 327/693/957 resulting in failure to load the kernel module.

Signed-off-by: Ashish Varma 
---


Should there be a fixes tag?

- Greg


  rhel/openvswitch-kmod-fedora.spec.in | 13 ++---
  1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
b/rhel/openvswitch-kmod-fedora.spec.in
index 92d763f..27f443a 100644
--- a/rhel/openvswitch-kmod-fedora.spec.in
+++ b/rhel/openvswitch-kmod-fedora.spec.in
@@ -90,13 +90,12 @@ if grep -qs "suse" /etc/os-release; then
  if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
  %{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
  fi
-elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ]; then
-if [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
-   [ "$major_rev" = "957" ]; then
-# For RHEL 7.2, 7.4 and 7.6
-if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
-%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
-fi
+elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ] &&
+ { [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
+   [ "$major_rev" = "957" ]; }; then
+# For RHEL 7.2, 7.4 and 7.6
+if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
+%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
  fi
  else
  # Ensure that modprobe will find our modules.


___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] rhel: Fixed a bug for checking the correct major version and revision.

2019-06-07 Thread Yifeng Sun 
Thanks Ashish for the fix.

Reviewed-by: Yifeng Sun 

On Fri, Jun 7, 2019 at 3:45 PM Ashish Varma  wrote:
>
> Fixed a bug where checking for major version 3.10 and major revision not
> equal to 327 or 693 or 957 should have gone to the default else at the end.
> In the current code, the default else condition will not get executed
> for kernel with major version 3.10 and major revision not equal
> to 327/693/957 resulting in failure to load the kernel module.
>
> Signed-off-by: Ashish Varma 
> ---
>  rhel/openvswitch-kmod-fedora.spec.in | 13 ++---
>  1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
> b/rhel/openvswitch-kmod-fedora.spec.in
> index 92d763f..27f443a 100644
> --- a/rhel/openvswitch-kmod-fedora.spec.in
> +++ b/rhel/openvswitch-kmod-fedora.spec.in
> @@ -90,13 +90,12 @@ if grep -qs "suse" /etc/os-release; then
>  if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
>  %{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
>  fi
> -elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ]; then
> -if [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
> -   [ "$major_rev" = "957" ]; then
> -# For RHEL 7.2, 7.4 and 7.6
> -if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
> -%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
> -fi
> +elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ] &&
> + { [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
> +   [ "$major_rev" = "957" ]; }; then
> +# For RHEL 7.2, 7.4 and 7.6
> +if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
> +%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
>  fi
>  else
>  # Ensure that modprobe will find our modules.
> --
> 2.7.4
>
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH] rhel: Fixed a bug for checking the correct major version and revision.

2019-06-07 Thread Ashish Varma 
Fixed a bug where checking for major version 3.10 and major revision not
equal to 327 or 693 or 957 should have gone to the default else at the end.
In the current code, the default else condition will not get executed
for kernel with major version 3.10 and major revision not equal
to 327/693/957 resulting in failure to load the kernel module.

Signed-off-by: Ashish Varma 
---
 rhel/openvswitch-kmod-fedora.spec.in | 13 ++---
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/rhel/openvswitch-kmod-fedora.spec.in 
b/rhel/openvswitch-kmod-fedora.spec.in
index 92d763f..27f443a 100644
--- a/rhel/openvswitch-kmod-fedora.spec.in
+++ b/rhel/openvswitch-kmod-fedora.spec.in
@@ -90,13 +90,12 @@ if grep -qs "suse" /etc/os-release; then
 if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
 %{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
 fi
-elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ]; then
-if [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
-   [ "$major_rev" = "957" ]; then
-# For RHEL 7.2, 7.4 and 7.6
-if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
-%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
-fi
+elif [ "$mainline_major" = "3" ] && [ "$mainline_minor" = "10" ] &&
+ { [ "$major_rev" = "327" ] || [ "$major_rev" = "693" ] || \
+   [ "$major_rev" = "957" ]; }; then
+# For RHEL 7.2, 7.4 and 7.6
+if [ -x "%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh" ]; then
+%{_datadir}/openvswitch/scripts/ovs-kmod-manage.sh
 fi
 else
 # Ensure that modprobe will find our modules.
-- 
2.7.4

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev