Fwd: [regression] dvb_usb_cxusb (was Re: ir-keytable: infinite loops, segfaults)

[Vincent McIntyre]

Hi list

I missed you in the cc: field...

-- Forwarded message --
From: Vincent McIntyre <vincent.mcint...@gmail.com>
Date: Thu, 16 Feb 2017 23:51:05 +1100
Subject: Re: [regression] dvb_usb_cxusb (was Re: ir-keytable: infinite
loops, segfaults)
To: Sean Young <s...@mess.org>

On 2/16/17, Sean Young <s...@mess.org> wrote:
>
> The problem is that DVB_USB_CXUSB Kconfig has this line:
> select DVB_SI2168 if MEDIA_SUBDRV_AUTOSELECT
> The make_kconfig.pl script transforms this into a dependency, but
> DVB_SI2168 is only available when compiling against kernel 4.7 or later.
> I think only one select line needs to match, so I created this patch.
>
> Please apply this patch against media_build, you might need to do make
> clean before building again.

Awsome - build is working again, thank you. See the other thread for
my progress report.

> Thanks,
>
> Sean
>
>
> From: Sean Young <s...@mess.org>
> Date: Wed, 15 Feb 2017 14:58:00 +
> Subject: [PATCH] only one select Kconfig needs to match

Tested-by: vincent.mcint...@gmail.com

> ---
>  v4l/scripts/make_kconfig.pl | 20 +++-
>  1 file changed, 19 insertions(+), 1 deletion(-)
>
> diff --git a/v4l/scripts/make_kconfig.pl b/v4l/scripts/make_kconfig.pl
> index ba8c134..a11f820 100755
> --- a/v4l/scripts/make_kconfig.pl
> +++ b/v4l/scripts/make_kconfig.pl
> @@ -169,6 +169,7 @@ sub depends($$)
>   push @{$depends{$key}}, $deps;
>  }
>
> +my %selectdepends = ();
>  sub selects($$$)
>  {
>   my $key = shift;
> @@ -181,7 +182,7 @@ sub selects($$$)
>   # Transform "select X if Y" into "depends on !Y || X"
>   $select = "!($if) || ($select)";
>   }
> - push @{$depends{$key}}, $select;
> + push @{$selectdepends{$key}}, $select;
>  }
>
>  # Needs:
> @@ -228,6 +229,23 @@ sub checkdeps()
>   return 0;
>   }
>   }
> + my $selectdeps = $selectdepends{$key};
> + if ($selectdeps) {
> + my $found = 0;
> + foreach (@$selectdeps) {
> + next if($_ eq '');
> + if (eval(toperl($_))) {
> + $found = 1;
> + last;
> + }
> + }
> + if ($found == 0) {
> + print "Disabling $key, select dependency '$_' 
> not met\n" if $debug;
> + $allconfig{$key} = 0;
> + return 0;
> + }
> + }
> +
>   return 1;
>   }
>
> --
> 2.7.4

Vince

Re: [regression] dvb_usb_cxusb (was Re: ir-keytable: infinite loops, segfaults)

[Sean Young]

On Wed, Feb 08, 2017 at 10:30:30PM +1100, Vincent McIntyre wrote:
> Hi
> 
> I have been working with Sean on figuring out the protocol used by a
> dvico remote.
> I thought the patch he sent was at fault but I backed it out and tried again.
> 
> I've attached a full dmesg but the core of it is when dvb_usb_cxusb
> tries to load:
> 
> [7.858907] WARNING: You are using an experimental version of the
> media stack.
> As the driver is backported to an older kernel, it doesn't 
> offer
> enough quality for its usage in production.
> Use it with care.
>Latest git patches (needed if you report a bug to
> linux-media@vger.kernel.org):
> 47b037a0512d9f8675ec2693bed46c8ea6a884ab [media]
> v4l2-async: failing functions shouldn't have side effects
> 79a2eda80c6dab79790c308d9f50ecd2e5021ba3 [media]
> mantis_dvb: fix some error codes in mantis_dvb_init()
> c2987aaf0c9c2bcb0d4c5902d61473d9aa018a3d [media]
> exynos-gsc: Avoid spamming the log on VIDIOC_TRY_FMT
> [7.861968] dvb_usb_af9035 1-4:1.0: prechip_version=83
> chip_version=02 chip_type=9135
> [7.887476] dvb_usb_cxusb: disagrees about version of symbol

Sorry about not getting back to you sooner, life got in the way. The
problem here is that the dvb-usb-cxusb did not get selected, so it
was not recompiled.

The problem is that DVB_USB_CXUSB Kconfig has this line:
select DVB_SI2168 if MEDIA_SUBDRV_AUTOSELECT
The make_kconfig.pl script transforms this into a dependency, but 
DVB_SI2168 is only available when compiling against kernel 4.7 or later. 
I think only one select line needs to match, so I created this patch.

Please apply this patch against media_build, you might need to do make
clean before building again.

Thanks,

Sean


From: Sean Young 
Date: Wed, 15 Feb 2017 14:58:00 +
Subject: [PATCH] only one select Kconfig needs to match

---
 v4l/scripts/make_kconfig.pl | 20 +++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/v4l/scripts/make_kconfig.pl b/v4l/scripts/make_kconfig.pl
index ba8c134..a11f820 100755
--- a/v4l/scripts/make_kconfig.pl
+++ b/v4l/scripts/make_kconfig.pl
@@ -169,6 +169,7 @@ sub depends($$)
push @{$depends{$key}}, $deps;
 }
 
+my %selectdepends = ();
 sub selects($$$)
 {
my $key = shift;
@@ -181,7 +182,7 @@ sub selects($$$)
# Transform "select X if Y" into "depends on !Y || X"
$select = "!($if) || ($select)";
}
-   push @{$depends{$key}}, $select;
+   push @{$selectdepends{$key}}, $select;
 }
 
 # Needs:
@@ -228,6 +229,23 @@ sub checkdeps()
return 0;
}
}
+   my $selectdeps = $selectdepends{$key};
+   if ($selectdeps) {
+   my $found = 0;
+   foreach (@$selectdeps) {
+   next if($_ eq '');
+   if (eval(toperl($_))) {
+   $found = 1;
+   last;
+   }
+   }
+   if ($found == 0) {
+   print "Disabling $key, select dependency '$_' 
not met\n" if $debug;
+   $allconfig{$key} = 0;
+   return 0;
+   }
+   }
+
return 1;
}
 
-- 
2.7.4

Re: [regression] dvb_usb_cxusb (was Re: ir-keytable: infinite loops, segfaults)

[Vincent McIntyre]

ping?

My media_build tree is updated as far as:
$ git log -1
commit 0721d4bde661c71cd4e41de37afb24b0694c65a3
Author: Hans Verkuil 
Date:   Mon Nov 21 13:17:19 2016 +0100

Only use Makefile.mm if frame_vector.c is present.

Signed-off-by: Hans Verkuil 

> On Wed, Feb 08, 2017 at 10:30:30PM +1100, Vincent McIntyre wrote:
>> Hi
>>
>> I have been working with Sean on figuring out the protocol used by a
>> dvico remote.
>> I thought the patch he sent was at fault but I backed it out and tried
>> again.
>>
>> I've attached a full dmesg but the core of it is when dvb_usb_cxusb
>> tries to load:
>>
>> [7.858907] WARNING: You are using an experimental version of the
>> media stack.
>> As the driver is backported to an older kernel, it doesn't
>> offer
>> enough quality for its usage in production.
>> Use it with care.
>>Latest git patches (needed if you report a bug to
>> linux-media@vger.kernel.org):
>> 47b037a0512d9f8675ec2693bed46c8ea6a884ab [media]
>> v4l2-async: failing functions shouldn't have side effects
>> 79a2eda80c6dab79790c308d9f50ecd2e5021ba3 [media]
>> mantis_dvb: fix some error codes in mantis_dvb_init()
>> c2987aaf0c9c2bcb0d4c5902d61473d9aa018a3d [media]
>> exynos-gsc: Avoid spamming the log on VIDIOC_TRY_FMT
>> [7.861968] dvb_usb_af9035 1-4:1.0: prechip_version=83
>> chip_version=02 chip_type=9135
>> [7.887476] dvb_usb_cxusb: disagrees about version of symbol
>> dvb_usb_generic_rw
>> [7.887477] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_rw (err -22)
>

Re: [regression] dvb_usb_cxusb (was Re: ir-keytable: infinite loops, segfaults)

[Sean Young]

Hi Vincent,

On Wed, Feb 08, 2017 at 10:30:30PM +1100, Vincent McIntyre wrote:
> Hi
> 
> I have been working with Sean on figuring out the protocol used by a
> dvico remote.
> I thought the patch he sent was at fault but I backed it out and tried again.
> 
> I've attached a full dmesg but the core of it is when dvb_usb_cxusb
> tries to load:
> 
> [7.858907] WARNING: You are using an experimental version of the
> media stack.
> As the driver is backported to an older kernel, it doesn't 
> offer
> enough quality for its usage in production.
> Use it with care.
>Latest git patches (needed if you report a bug to
> linux-media@vger.kernel.org):
> 47b037a0512d9f8675ec2693bed46c8ea6a884ab [media]
> v4l2-async: failing functions shouldn't have side effects
> 79a2eda80c6dab79790c308d9f50ecd2e5021ba3 [media]
> mantis_dvb: fix some error codes in mantis_dvb_init()
> c2987aaf0c9c2bcb0d4c5902d61473d9aa018a3d [media]
> exynos-gsc: Avoid spamming the log on VIDIOC_TRY_FMT
> [7.861968] dvb_usb_af9035 1-4:1.0: prechip_version=83
> chip_version=02 chip_type=9135
> [7.887476] dvb_usb_cxusb: disagrees about version of symbol
> dvb_usb_generic_rw
> [7.887477] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_rw (err -22)

-snip-

This is a problem with media_build. I'm not familiar with media_build, I
did try it out last night (for the first time) and got the same issue on
Ubuntu 16.04. I haven't been able to figure out what the problem is yet.

I'll have a look again tonight or tomorrow night. In the mean time, if
anyone else knows then that would be great. :)


Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[regression] dvb_usb_cxusb (was Re: ir-keytable: infinite loops, segfaults)

[Vincent McIntyre]

Hi

I have been working with Sean on figuring out the protocol used by a
dvico remote.
I thought the patch he sent was at fault but I backed it out and tried again.

I've attached a full dmesg but the core of it is when dvb_usb_cxusb
tries to load:

[7.858907] WARNING: You are using an experimental version of the
media stack.
As the driver is backported to an older kernel, it doesn't offer
enough quality for its usage in production.
Use it with care.
   Latest git patches (needed if you report a bug to
linux-media@vger.kernel.org):
47b037a0512d9f8675ec2693bed46c8ea6a884ab [media]
v4l2-async: failing functions shouldn't have side effects
79a2eda80c6dab79790c308d9f50ecd2e5021ba3 [media]
mantis_dvb: fix some error codes in mantis_dvb_init()
c2987aaf0c9c2bcb0d4c5902d61473d9aa018a3d [media]
exynos-gsc: Avoid spamming the log on VIDIOC_TRY_FMT
[7.861968] dvb_usb_af9035 1-4:1.0: prechip_version=83
chip_version=02 chip_type=9135
[7.887476] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_generic_rw
[7.887477] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_rw (err -22)
[7.887499] dvb_usb_cxusb: disagrees about version of symbol rc_keydown
[7.887500] dvb_usb_cxusb: Unknown symbol rc_keydown (err -22)
[7.887507] dvb_usb_cxusb: disagrees about version of symbol
dib0070_wbd_offset
[7.887508] dvb_usb_cxusb: Unknown symbol dib0070_wbd_offset (err -22)
[7.887513] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_device_init
[7.887514] dvb_usb_cxusb: Unknown symbol dvb_usb_device_init (err -22)
[7.887518] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_generic_write
[7.887519] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_write (err -22)
[7.900099] usb 1-4: dvb_usb_v2: found a 'Leadtek WinFast DTV
Dongle Dual' in cold state
[7.900768] usb 1-4: dvb_usb_v2: downloading firmware from file
'dvb-usb-it9135-02.fw'
[8.124533] input: HDA NVidia HDMI/DP,pcm=3 as
/devices/pci:00/:00:01.0/:01:00.1/sound/card1/input14
[8.124616] input: HDA NVidia HDMI/DP,pcm=7 as
/devices/pci:00/:00:01.0/:01:00.1/sound/card1/input15
[8.124690] input: HDA NVidia HDMI/DP,pcm=8 as
/devices/pci:00/:00:01.0/:01:00.1/sound/card1/input16
[8.124763] input: HDA NVidia HDMI/DP,pcm=9 as
/devices/pci:00/:00:01.0/:01:00.1/sound/card1/input17
[8.144601] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_generic_rw
[8.144603] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_rw (err -22)
[8.144638] dvb_usb_cxusb: disagrees about version of symbol rc_keydown
[8.144639] dvb_usb_cxusb: Unknown symbol rc_keydown (err -22)
[8.144648] dvb_usb_cxusb: disagrees about version of symbol
dib0070_wbd_offset
[8.144649] dvb_usb_cxusb: Unknown symbol dib0070_wbd_offset (err -22)
[8.144654] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_device_init
[8.144655] dvb_usb_cxusb: Unknown symbol dvb_usb_device_init (err -22)
[8.144659] dvb_usb_cxusb: disagrees about version of symbol
dvb_usb_generic_write
[8.144660] dvb_usb_cxusb: Unknown symbol dvb_usb_generic_write (err -22)


Vince
[0.00] Initializing cgroup subsys cpuset
[0.00] Initializing cgroup subsys cpu
[0.00] Initializing cgroup subsys cpuacct
[0.00] Linux version 4.4.0-59-generic (buildd@lgw01-11) (gcc version 
5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4) ) #80-Ubuntu SMP Fri Jan 6 
17:47:47 UTC 2017 (Ubuntu 4.4.0-59.80-generic 4.4.35)
[0.00] Command line: BOOT_IMAGE=/boot/vmlinuz-4.4.0-59-generic 
root=UUID=420e8415-6799-4f8e-bb39-7d9077271ea6 ro
[0.00] KERNEL supported cpus:
[0.00]   Intel GenuineIntel
[0.00]   AMD AuthenticAMD
[0.00]   Centaur CentaurHauls
[0.00] x86/fpu: Legacy x87 FPU detected.
[0.00] x86/fpu: Using 'lazy' FPU context switches.
[0.00] e820: BIOS-provided physical RAM map:
[0.00] BIOS-e820: [mem 0x-0x0009ebff] usable
[0.00] BIOS-e820: [mem 0x0009ec00-0x0009] reserved
[0.00] BIOS-e820: [mem 0x000e-0x000f] reserved
[0.00] BIOS-e820: [mem 0x0010-0x7ed12fff] usable
[0.00] BIOS-e820: [mem 0x7ed13000-0x7ed14fff] reserved
[0.00] BIOS-e820: [mem 0x7ed15000-0x7ee2dfff] usable
[0.00] BIOS-e820: [mem 0x7ee2e000-0x7eee4fff] ACPI NVS
[0.00] BIOS-e820: [mem 0x7eee5000-0x7eee8fff] usable
[0.00] BIOS-e820: [mem 0x7eee9000-0x7eef2fff] ACPI data
[0.00] BIOS-e820: [mem 0x7eef3000-0x7eef3fff] usable
[0.00] BIOS-e820: [mem 0x7eef4000-0x7eefefff] ACPI data
[0.00] BIOS-e820: [mem 0x7eeff000-0x7eef] usable
[0.00]