Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IQ file playback fails on restart #1234

Closed
drmpeg opened this issue Apr 30, 2023 · 11 comments · Fixed by gnuradio/gnuradio#6647
Closed

IQ file playback fails on restart #1234

drmpeg opened this issue Apr 30, 2023 · 11 comments · Fixed by gnuradio/gnuradio#6647
Labels

Comments

@drmpeg
Copy link
Contributor

drmpeg commented Apr 30, 2023

IQ file playback always fails when started, stopped and then restarted with the Start/Stop DSP button. If stop is pressed again, the whole process locks up and needs to be killed.

Parameters are:

file=/media/re/ssd/ssb.cfile,freq=100e6,rate=1.92e6,repeat=true,throttle=true

The test IQ file is here:

https://www.w6rz.net/ssb.cfile

771,446,112 bytes

@drmpeg
Copy link
Contributor Author

drmpeg commented Apr 30, 2023

Built against gnuradio v3.11.0.0git-402-g54d199a2

@vladisslav2011
Copy link
Contributor

Could you revert 5052dbc and retest?

@drmpeg
Copy link
Contributor Author

drmpeg commented Apr 30, 2023

@vladisslav2011 It's been broken for at least a year. Long before that commit.

@vladisslav2011
Copy link
Contributor

I can't confirm this on GNU Radio 3.7.
So it may be GNU Radio or gr-osmosdr regression.

@argilo argilo added the bug label May 1, 2023
@vladisslav2011
Copy link
Contributor

GNU Radio maint-3.8 branch: not confirmed
GNU Radio maint-3.9 branch: not confirmed

I have not downloaded your test file and using locally generated IQ files (~1 minute of FM band, received with RTL-SDR) to test, so the issue may be specific to your file.
Is it reproducible with any other file, recorded with Gqrx IQ Tool?

@vladisslav2011
Copy link
Contributor

I've reproduced something, that may match the description:
https://www.youtube.com/watch?v=gUtThjXUVOk

GNU Radio maint-3.10 branch: confirmed partially

The DSP does not freeze completely, but there is noticeable startup delay, that grows with each restart.
Another GNU Radio regression...

@drmpeg
Copy link
Contributor Author

drmpeg commented May 1, 2023

Yes, that's the behavior. Except my IQ file is at a much lower sample rate (1.92 Msps), so it takes much longer to recover.

Testing now with 3.9-maint to confirm.

@drmpeg
Copy link
Contributor Author

drmpeg commented May 1, 2023

Confirmed that GNU Radio maint-3.9 works fine.

I watched your video again and realized the sample rates are not that much different. I think the long recovery I'm seeing (20 seconds for the first time) is due to the size of my test file (50 seconds long).

@argilo
Copy link
Member

argilo commented May 1, 2023

The problem appears to be the removal of this line of code:

https://github.com/gnuradio/gnuradio/pull/5670/files#diff-5dbb6299f5c5a8cc7772c76182a47d525c8af7eabaef4239ab43d8a569795ebdL47

I think that should have been replaced by d_total_items = 0;

@argilo
Copy link
Member

argilo commented May 1, 2023

Fix: gnuradio/gnuradio#6647

@drmpeg
Copy link
Contributor Author

drmpeg commented May 3, 2023

Fixed with latest GNU Radio commit gnuradio/gnuradio@b7a0ac5

@drmpeg drmpeg closed this as completed May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants