Re: Include FD_SETSIZE on RTEMS to 256

[Joel Sherrill <joel@rtems.org>]

[-- Attachment #1: Type: text/plain, Size: 1347 bytes --]

On Wed, Mar 13, 2024 at 11:03 AM Joel Sherrill <joel@rtems.org> wrote:

>
>
> On Wed, Mar 13, 2024 at 10:02 AM Corinna Vinschen <vinschen@redhat.com>
> wrote:
>
>> On Mar 13 09:35, Joel Sherrill wrote:
>> > On Wed, Mar 13, 2024 at 9:21 AM Jon Turney <jon.turney@dronecode.org.uk
>> >
>> > wrote:
>> >
>> > > On 28/02/2024 23:02, Joel Sherrill wrote:
>> > > > Hi
>> > > >
>> > > > The attached patch changes FD_SETSIZE from 64 to 256 for RTEMS.
>> > > >
>> > > > Is it OK to apply?
>> > > >
>> > >
>> > > This uses the preprocessor directive '#elifdef',
>> > >
>> > > It seems like support for that was added in gcc 12.
>> > >
>> >
>> > Good catch. Unfortunately, I just pushed the posted patch just before
>> your
>> > email arrived.
>> >
>> > I have posted a small patch to change this to #elif defined(__rtems__).
>> If
>> > someone acks it, I will push it quickly to minimize who might be
>> impacted.
>>
>> You sent the wrong patch apparently.  However, if your patch just
>> changes #elifdef X to #elif defined (X), then test it with a pre-12
>> gcc and then push it.  Soon.
>>
>
> Sorry. I am building a gcc8 toolchain now to verify the change. Hopefully,
> it will not be too much longer.
>

Correction pushed and tested with gcc8 and a test program.

--joel

>
>>
>> Corinna
>>
>>