This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 1/2] Add nocancel version of pread64()
- From: Leandro Pereira <Leandro dot Pereira at microsoft dot com>
- To: Carlos O'Donell <carlos at redhat dot com>
- Cc: libc-alpha <libc-alpha at sourceware dot org>
- Date: Fri, 6 Sep 2019 23:15:50 +0000
- Subject: Re: [PATCH 1/2] Add nocancel version of pread64()
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FqxNKPliU7xXqdfOyRv/UHUA1gKQdMgNXgqp9cPsHuU=; b=Yadud47QPARf01UUv8RirqBk9nYfoQgQTP2ZlQYdL7FDNJ45cB9z4bnN0FGWxsC8VNP+LN5mumSkhGqynWosWT2splTYG7iLEhsluuTO7X+xfpaRlmbL/b40Dtp+ifQznFWBO8O8EhNpdACTYeGxUyPmN/qyXrFYJtsb5258cyuyC9GBluy78rZnAAehsgGFS7AEw1KV1JpONTuyu3p9ds91Q9tYkq71fq6lOdRl52IH0EFvB5jV7fsZ0qFkJ0cGmLq80fP0uzkjGOcf87+f0Painnw/fnZOhzkdbD1sSc+C3ZArLXsUh8quzkWwPzdrNzsvOaUV4Z5wtJvhAGtIzw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RcG+wS/O5hoeGyoJC4IABBW+D8U+lrR1a0hZ+f3Iuz6KnqL+pYhd1O3jw8tGyh77liZn7KSGwFCeO/EHZj/M9lmOi6PPvdvtgMtiuUpBJokGAVVEzDyCst/fVy26KMbj/d9+zC4aADDJ9TTLGC7Fql6OOlw8MDYnAA1aHfd6XH1QewOUvvghpv5BrrvrvufL0FUHBpWoFQFsu3QwIuzZywLiJ6A/o9P2FzU8SN2NlEtF0bjSZnkg/LIih5YXPttZgMAGEpNLnFdpUOMyxFn5Q0Dxn9SwRQ0Pp00Vif1WJki1ISxDbfLFDgXZNoTuO3ILrGr41AhrIVIwlORLmNAuAw==
- References: <MWHPR2101MB0732AF4AE4ECE64CB603997BE8DA0@MWHPR2101MB0732.namprd21.prod.outlook.com>,<1e40e0d0-3d60-8d8f-d4b6-376ced17aa49@redhat.com>
On August 6th 2019, Carlos O'Donnel wrote:
>> 2019-08-05 Leandro Pereira <leandro.pereira@microsoft.com>
>>
>> * sysdeps/generic/not-cancel.h: New macro.
>> * sysdeps/unix/sysv/linux/not-cancel.h: New macro.
>> * sysdeps/unix/sysv/linux/pread64_nocancel.c: Implement nocancel
>> version of pread64.
>> * sysdeps/unix/sysv/linux/Versions (GLIBC_PRIVATE): Add
>> __pread64_nocancel.
>> * sysdeps/unix/sysv/linux/Makefile: Reference pread64_nocancel.c.
>
> (...)
> I don't see a copyright disclaimer for you on file, and I'm going to
> follow up with you privately to see what we can do there to expedite
> the process so we can work on your patches as quickly as possible.
With the exception of the boilerplate copied from existing glibc code
(essentially the patch adding the pread() syscall wrapper), the legally
significant changes are only the changes to use pread() instead of
seek()+read(); and those are less than around 15 lines of code. This should
be fine according to FSFs guidelines.
Since the changes are small, it falls under an "de minimis" exception rule
for open source contributions at Microsoft, making it a lot easier to
contribute this patch. If we need to contribute larger changes, then we
can work out the copyright paperwork. (Or if I contribute changes as
a private individual, of course.)
Leandro