This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[pushed] [PATCH v2] Associate target_ops with target_fileio file descriptors
- From: Gary Benson <gbenson at redhat dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Wed, 25 Mar 2015 11:32:01 +0000
- Subject: [pushed] [PATCH v2] Associate target_ops with target_fileio file descriptors
- Authentication-results: sourceware.org; auth=none
- References: <20150318184538 dot GA32134 at blade dot nx> <1426867612-32278-1-git-send-email-gbenson at redhat dot com> <5511651F dot 4050400 at redhat dot com> <20150324161752 dot GA10601 at blade dot nx> <551192A0 dot 3070107 at redhat dot com> <551196FA dot 7020702 at redhat dot com>
Pedro Alves wrote:
> Because there's no point in starting the next lookup at the
> handle we just opened if we're reusing a slot, either. And
> then that allows getting rid of the VEC_address, by simply
> doing that increment at the end:
>
> /* Index into fileio_fhandles of the lowest handle that might be
> closed. This permits handle reuse without searching the whole list
> each time a new file is opened. */
> static int lowest_closed_fd = 0;
>
> static int
> acquire_fileio_fd (struct target_ops *t, int fd)
> {
> fileio_fh_t *fh, buf;
>
> gdb_assert (!is_closed_fileio_fh (fd));
>
> /* Search for closed handles to reuse. */
> for (;
> VEC_iterate (fileio_fh_t, fileio_fhandles,
> lowest_closed_fd, fh);
> lowest_closed_fd++)
> if (is_closed_fileio_fh (fh->fd))
> break;
>
> /* Push a new handle if no closed handles were found. */
> if (lowest_closed_fd == VEC_length (fileio_fh_t, fileio_fhandles))
> fh = VEC_safe_push (fileio_fh_t, fileio_fhandles, NULL);
>
> /* Fill in the handle. */
> fh->t = t;
> fh->fd = fd;
>
> /* Return its index, and start the next lookup at
> the next index. */
> return lowest_closed_fd++;
> }
That's nice, I pushed that one, thanks!
Gary
--
http://gbenson.net/