diff options
author | Jeff Long <willcode4@gmail.com> | 2018-01-16 18:19:05 -0500 |
---|---|---|
committer | Martin Braun <martin.braun@ettus.com> | 2018-02-08 16:01:08 -0800 |
commit | 6ae02d1d6d04a5f529ab3fea496c589f8d14229d (patch) | |
tree | 3491ab847228dc2bb7c315c7b421eeb1423533c5 /gr-blocks/lib/file_source_impl.cc | |
parent | 2270a41d12ee028905302233d8178568d6da6daa (diff) |
file_source: add optional offset (in items) and len (in items)
- Adds unit tests, and splits existing unit tests into different files:
qa_file_source.py - tests file_source_f
qa_file_sink.py - tests file_sink_f
qa_file_descriptor_source_sink.py - tests file_descriptor_*_f
Diffstat (limited to 'gr-blocks/lib/file_source_impl.cc')
-rw-r--r-- | gr-blocks/lib/file_source_impl.cc | 200 |
1 files changed, 128 insertions, 72 deletions
diff --git a/gr-blocks/lib/file_source_impl.cc b/gr-blocks/lib/file_source_impl.cc index c077c74fd9..6f4f8fde26 100644 --- a/gr-blocks/lib/file_source_impl.cc +++ b/gr-blocks/lib/file_source_impl.cc @@ -1,6 +1,6 @@ /* -*- c++ -*- */ /* - * Copyright 2012 Free Software Foundation, Inc. + * Copyright 2012, 2018 Free Software Foundation, Inc. * * This file is part of GNU Radio * @@ -39,35 +39,38 @@ #include <io.h> #endif #ifdef O_BINARY -#define OUR_O_BINARY O_BINARY +#define OUR_O_BINARY O_BINARY #else -#define OUR_O_BINARY 0 +#define OUR_O_BINARY 0 #endif // should be handled via configure #ifdef O_LARGEFILE -#define OUR_O_LARGEFILE O_LARGEFILE +#define OUR_O_LARGEFILE O_LARGEFILE #else -#define OUR_O_LARGEFILE 0 +#define OUR_O_LARGEFILE 0 #endif namespace gr { namespace blocks { - file_source::sptr file_source::make(size_t itemsize, const char *filename, bool repeat) + file_source::sptr file_source::make(size_t itemsize, const char *filename, bool repeat, + size_t start_offset_items, size_t length_items) { return gnuradio::get_initial_sptr - (new file_source_impl(itemsize, filename, repeat)); + (new file_source_impl(itemsize, filename, repeat, start_offset_items, length_items)); } - file_source_impl::file_source_impl(size_t itemsize, const char *filename, bool repeat) + file_source_impl::file_source_impl(size_t itemsize, const char *filename, bool repeat, + size_t start_offset_items, size_t length_items) : sync_block("file_source", - io_signature::make(0, 0, 0), - io_signature::make(1, 1, itemsize)), - d_itemsize(itemsize), d_fp(0), d_new_fp(0), d_repeat(repeat), - d_updated(false), d_file_begin(true), d_repeat_cnt(0), - d_add_begin_tag(pmt::PMT_NIL) + io_signature::make(0, 0, 0), + io_signature::make(1, 1, itemsize)), + d_itemsize(itemsize), + d_start_offset_items(start_offset_items), d_length_items(length_items), + d_fp(0), d_new_fp(0), d_repeat(repeat), d_updated(false), + d_file_begin(true), d_repeat_cnt(0), d_add_begin_tag(pmt::PMT_NIL) { - open(filename, repeat); + open(filename, repeat, start_offset_items, length_items); do_update(); std::stringstream str; @@ -86,12 +89,35 @@ namespace gr { bool file_source_impl::seek(long seek_point, int whence) { - return fseek((FILE*)d_fp, seek_point *d_itemsize, whence) == 0; + seek_point += d_start_offset_items; + + switch(whence) { + case SEEK_SET: + break; + case SEEK_CUR: + seek_point += (d_length_items - d_items_remaining); + break; + case SEEK_END: + seek_point = d_length_items - seek_point; + break; + default: + GR_LOG_WARN(d_logger, "bad seek mode"); + return 0; + } + + if ((seek_point < (long)d_start_offset_items) + || (seek_point > (long)(d_start_offset_items+d_length_items-1))) { + GR_LOG_WARN(d_logger, "bad seek point"); + return 0; + } + + return fseek((FILE*)d_fp, seek_point * d_itemsize, SEEK_SET) == 0; } void - file_source_impl::open(const char *filename, bool repeat) + file_source_impl::open(const char *filename, bool repeat, + size_t start_offset_items, size_t length_items) { // obtain exclusive access for duration of this function gr::thread::scoped_lock lock(fp_mutex); @@ -100,33 +126,61 @@ namespace gr { // we use "open" to use to the O_LARGEFILE flag if((fd = ::open(filename, O_RDONLY | OUR_O_LARGEFILE | OUR_O_BINARY)) < 0) { - perror(filename); - throw std::runtime_error("can't open file"); + GR_LOG_ERROR(d_logger, boost::format("%s: %s") % filename % strerror(errno)); + throw std::runtime_error("can't open file"); } if(d_new_fp) { - fclose(d_new_fp); - d_new_fp = 0; + fclose(d_new_fp); + d_new_fp = 0; } if((d_new_fp = fdopen (fd, "rb")) == NULL) { - perror(filename); - ::close(fd); // don't leak file descriptor if fdopen fails - throw std::runtime_error("can't open file"); + GR_LOG_ERROR(d_logger, boost::format("%s: %s") % filename % strerror(errno)); + ::close(fd); // don't leak file descriptor if fdopen fails + throw std::runtime_error("can't open file"); } //Check to ensure the file will be consumed according to item size fseek(d_new_fp, 0, SEEK_END); int file_size = ftell(d_new_fp); - rewind (d_new_fp); - //Warn the user if part of the file will not be consumed. - if(file_size % d_itemsize){ - GR_LOG_WARN(d_logger, "WARNING: File will not be fully consumed with the current output type"); + // Make sure there will be at least one item available + if ((file_size / d_itemsize) < (start_offset_items+1)) { + if (start_offset_items) { + GR_LOG_WARN(d_logger, "file is too small for start offset"); + } + else { + GR_LOG_WARN(d_logger, "file is too small"); + } + fclose(d_new_fp); + throw std::runtime_error("file is too small"); + } + + size_t items_available = (file_size / d_itemsize - start_offset_items); + + // If length is not specified, use the remainder of the file. Check alignment at end. + if (length_items == 0) { + length_items = items_available; + if (file_size % d_itemsize){ + GR_LOG_WARN(d_logger, "file size is not a multiple of item size"); + } + } + + // Check specified length. Warn and use available items instead of throwing an exception. + if (length_items > items_available) { + length_items = items_available; + GR_LOG_WARN(d_logger, "file too short, will read fewer than requested items"); } + // Rewind to start offset + fseek(d_new_fp, start_offset_items * d_itemsize, SEEK_SET); + d_updated = true; d_repeat = repeat; + d_start_offset_items = start_offset_items; + d_length_items = length_items; + d_items_remaining = length_items; } void @@ -136,8 +190,8 @@ namespace gr { gr::thread::scoped_lock lock(fp_mutex); if(d_new_fp != NULL) { - fclose(d_new_fp); - d_new_fp = NULL; + fclose(d_new_fp); + d_new_fp = NULL; } d_updated = true; } @@ -146,15 +200,15 @@ namespace gr { file_source_impl::do_update() { if(d_updated) { - gr::thread::scoped_lock lock(fp_mutex); // hold while in scope + gr::thread::scoped_lock lock(fp_mutex); // hold while in scope - if(d_fp) - fclose(d_fp); + if(d_fp) + fclose(d_fp); - d_fp = d_new_fp; // install new file pointer - d_new_fp = 0; - d_updated = false; - d_file_begin = true; + d_fp = d_new_fp; // install new file pointer + d_new_fp = 0; + d_updated = false; + d_file_begin = true; } } @@ -166,60 +220,62 @@ namespace gr { int file_source_impl::work(int noutput_items, - gr_vector_const_void_star &input_items, - gr_vector_void_star &output_items) + gr_vector_const_void_star &input_items, + gr_vector_void_star &output_items) { char *o = (char*)output_items[0]; - int i; int size = noutput_items; do_update(); // update d_fp is reqd if(d_fp == NULL) - throw std::runtime_error("work with file not open"); + throw std::runtime_error("work with file not open"); gr::thread::scoped_lock lock(fp_mutex); // hold for the rest of this function + // No items remaining - all done + if (d_items_remaining == 0) + return WORK_DONE; + while(size) { + // Add stream tag whenever the file starts again if (d_file_begin && d_add_begin_tag != pmt::PMT_NIL) { - add_item_tag(0, nitems_written(0) + noutput_items - size, d_add_begin_tag, pmt::from_long(d_repeat_cnt), _id); + add_item_tag(0, nitems_written(0) + noutput_items - size, + d_add_begin_tag, pmt::from_long(d_repeat_cnt), _id); d_file_begin = false; } - i = fread(o, d_itemsize, size, (FILE*)d_fp); - - size -= i; - o += i * d_itemsize; - - if(size == 0) // done - break; - - if(i > 0) // short read, try again - continue; - - // We got a zero from fread. This is either EOF or error. In - // any event, if we're in repeat mode, seek back to the beginning - // of the file and try again, else break - if(!d_repeat) - break; - - if(fseek ((FILE *) d_fp, 0, SEEK_SET) == -1) { - fprintf(stderr, "[%s] fseek failed\n", __FILE__); - exit(-1); - } - if (d_add_begin_tag != pmt::PMT_NIL) { - d_file_begin = true; - d_repeat_cnt++; + size_t nitems_to_read = std::min((size_t)size, d_items_remaining); + + // Since the bounds of the file are known, unexpected nitems is an error + if (nitems_to_read != fread(o, d_itemsize, nitems_to_read, (FILE*)d_fp)) + throw std::runtime_error("fread error"); + + size -= nitems_to_read; + d_items_remaining -= nitems_to_read; + o += nitems_to_read * d_itemsize; + + // Ran out of items ("EOF") + if (d_items_remaining == 0) { + + // Repeat: rewind and request tag + if (d_repeat) { + fseek(d_fp, d_start_offset_items * d_itemsize, SEEK_SET); + d_items_remaining = d_length_items; + if (d_add_begin_tag != pmt::PMT_NIL) { + d_file_begin = true; + d_repeat_cnt++; + } + } + + // No repeat: return + else { + break; + } } } - if(size > 0) { // EOF or error - if(size == noutput_items) // we didn't read anything; say we're done - return -1; - return noutput_items - size; // else return partial result - } - - return noutput_items; + return (noutput_items - size); } } /* namespace blocks */ |