Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Kamil Rytarowski <kamil@netbsd.org>
To: Simon Marchi <simark@simark.ca>, gdb-patches@sourceware.org
Cc: tom@tromey.com
Subject: Re: [PATCH v2 02/10] Register a placeholder for NetBSD shared functions in gdb/nat
Date: Mon, 7 Sep 2020 21:49:05 +0200	[thread overview]
Message-ID: <59b373af-570c-6749-5d2a-e6a32aa68491@netbsd.org> (raw)
In-Reply-To: <727c98e0-1d76-3672-fb4d-5bc4720292f3@simark.ca>


[-- Attachment #1.1: Type: text/plain, Size: 2782 bytes --]

On 07.09.2020 20:44, Simon Marchi wrote:
> On 2020-09-03 8:28 p.m., Kamil Rytarowski wrote:
>> gdb/ChangeLog:
>>
>> 	* netbsd-nat.h: New file.
>> 	* netbsd-nat.c: Likewise.
>> ---
>>  gdb/ChangeLog        |  5 +++++
>>  gdb/nat/netbsd-nat.c | 24 ++++++++++++++++++++++++
>>  gdb/nat/netbsd-nat.h | 27 +++++++++++++++++++++++++++
>>  3 files changed, 56 insertions(+)
>>  create mode 100644 gdb/nat/netbsd-nat.c
>>  create mode 100644 gdb/nat/netbsd-nat.h
>>
>> diff --git a/gdb/ChangeLog b/gdb/ChangeLog
>> index faa3a612547..430f9b58a78 100644
>> --- a/gdb/ChangeLog
>> +++ b/gdb/ChangeLog
>> @@ -1,3 +1,8 @@
>> +2020-09-04  Kamil Rytarowski  <n54@gmx.com>
>> +
>> +	* netbsd-nat.h: New file.
>> +	* netbsd-nat.c: Likewise.
>> +
>>  2020-09-03  Alok Kumar Sharma  <AlokKumar.Sharma@amd.com>
>>
>>  	* gdb/i386-tdep.c (i386_floatformat_for_type): Added conditions
>> diff --git a/gdb/nat/netbsd-nat.c b/gdb/nat/netbsd-nat.c
>> new file mode 100644
>> index 00000000000..2b5a4183e30
>> --- /dev/null
>> +++ b/gdb/nat/netbsd-nat.c
>> @@ -0,0 +1,24 @@
>> +/* Internal interfaces for the NetBSD code.
>> +
>> +   Copyright (C) 2006-2020 Free Software Foundation, Inc.
>> +
>> +   This file is part of GDB.
>> +
>> +   This program is free software; you can redistribute it and/or modify
>> +   it under the terms of the GNU General Public License as published by
>> +   the Free Software Foundation; either version 3 of the License, or
>> +   (at your option) any later version.
>> +
>> +   This program is distributed in the hope that it will be useful,
>> +   but WITHOUT ANY WARRANTY; without even the implied warranty of
>> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> +   GNU General Public License for more details.
>> +
>> +   You should have received a copy of the GNU General Public License
>> +   along with this program.  If not, see <http://www.gnu.org/licenses/>.  */
>> +
>> +#include "nat/netbsd-nat.h"
>> +
>> +namespace netbsd_nat
>> +{
>> +}
>> diff --git a/gdb/nat/netbsd-nat.h b/gdb/nat/netbsd-nat.h
>> new file mode 100644
>> index 00000000000..5fa08746610
>> --- /dev/null
>> +++ b/gdb/nat/netbsd-nat.h
>> @@ -0,0 +1,27 @@
>> +/* Internal interfaces for the NetBSD code.
>> +
>> +   Copyright (C) 2006-2020 Free Software Foundation, Inc.
> 
> I presume that the copyright should say 2020 only?  It depends on the code that
> ends up here.  If you copy or move some code from some other file, in theory the
> copyright date should follow.
> 

This copyright years are picked from nbsd-nat.c. They could be narrowed
to 2020, but the original file containing these functions uses this range.

The functions from nbsd-nat.c are copied almost as is.

> Simon
> 



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2020-09-07 19:52 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-04  0:28 [PATCH v2 00/10] Add minimal NetBSD/amd64 gdbserver support Kamil Rytarowski
2020-09-04  0:28 ` [PATCH v2 01/10] Add handle_eintr to wrap EINTR handling in syscalls Kamil Rytarowski
2020-09-07 14:06   ` Simon Marchi
2020-09-07 14:59     ` Kamil Rytarowski
2020-10-12 17:56       ` [PATCH] gdb::handle_eintr, remove need to specify return type (Re: [PATCH v2 01/10] Add handle_eintr to wrap EINTR handling in syscalls) Pedro Alves
2020-10-13 13:43         ` Kamil Rytarowski
2020-10-13 14:17           ` [PATCH v2] gdb::handle_eintr, remove need to specify return type Pedro Alves
2020-10-13 14:54             ` Kamil Rytarowski
2020-10-16 20:51             ` Tom Tromey
2020-10-26 14:00               ` Pedro Alves
2020-10-26 14:20                 ` Tom Tromey
2020-10-26 18:59                   ` Pedro Alves
2020-09-04  0:28 ` [PATCH v2 02/10] Register a placeholder for NetBSD shared functions in gdb/nat Kamil Rytarowski
2020-09-07 18:44   ` Simon Marchi
2020-09-07 19:49     ` Kamil Rytarowski [this message]
2020-09-04  0:28 ` [PATCH v2 03/10] Build nat/netbsd-nat.o for the NetBSD native target Kamil Rytarowski
2020-09-04  0:28 ` [PATCH v2 04/10] Add netbsd_nat::pid_to_exec_file Kamil Rytarowski
2020-09-07  7:57   ` Andrew Burgess
2020-09-07 13:36     ` Kamil Rytarowski
2020-09-07 18:48       ` Simon Marchi
2020-09-07 18:47   ` Simon Marchi
2020-09-07 19:51     ` Kamil Rytarowski
2020-09-04  0:29 ` [PATCH v2 05/10] Add gdb/nat common functions for listing threads Kamil Rytarowski
2020-09-07 18:59   ` Simon Marchi
2020-09-07 19:57     ` Kamil Rytarowski
2020-09-04  0:29 ` [PATCH v2 06/10] Add netbsd_nat::enable_proc_events in gdb/nat Kamil Rytarowski
2020-09-04  0:29 ` [PATCH v2 07/10] Add a common utility function to read and write siginfo_t in inferior Kamil Rytarowski
2020-09-04  0:29 ` [PATCH v2 08/10] Avoid double free in startup_inferior Kamil Rytarowski
2020-09-07 19:19   ` Simon Marchi
2020-09-08  0:54     ` Kamil Rytarowski
2020-09-08  2:21       ` Simon Marchi
2020-09-04  0:29 ` [PATCH v2 09/10] Switch local native code to gdb/nat shared functions Kamil Rytarowski
2020-09-07 19:24   ` Simon Marchi
2020-09-08  0:04     ` Kamil Rytarowski
2020-09-04  0:29 ` [PATCH v2 10/10] Add minimal and functional NetBSD/amd64 gdbserver Kamil Rytarowski
2020-09-07 19:58   ` Simon Marchi
2020-09-08  0:03     ` Kamil Rytarowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=59b373af-570c-6749-5d2a-e6a32aa68491@netbsd.org \
    --to=kamil@netbsd.org \
    --cc=gdb-patches@sourceware.org \
    --cc=simark@simark.ca \
    --cc=tom@tromey.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox