public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* Re: Status of pwritev2/preadv2 in glibc?
       [not found] <ECD1C373-3464-4AEF-8E73-3130C5BFAF87@raithlin.com>
@ 2017-04-21 10:56 ` Christoph Hellwig
  2017-04-21 11:19   ` Florian Weimer
  0 siblings, 1 reply; 9+ messages in thread
From: Christoph Hellwig @ 2017-04-21 10:56 UTC (permalink / raw)
  To: Stephen Bates
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha

It would probably help to add the glibc list as the glibc developers
might know better..

On Sat, Apr 15, 2017 at 04:24:29PM +0000, Stephen  Bates wrote:
> Hi
> 
> Does anyone know the status of support for the pwritev2/preadv2 system calls in glibc? I am doing some more IO polling testing and realized that the support is not in 2.25 and does not seem to be staged for later release?
> 
> Right now I am using the FIO_HAVE_PWRITEV2 support that fio provides but it’d be nice to be able to write/adapt other user-space programs to test this capability…
> 
> Stephen 
> 
> 
---end quoted text---

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-21 10:56 ` Status of pwritev2/preadv2 in glibc? Christoph Hellwig
@ 2017-04-21 11:19   ` Florian Weimer
  2017-04-21 14:18     ` Adhemerval Zanella
  2017-04-24 17:56     ` Stephen  Bates
  0 siblings, 2 replies; 9+ messages in thread
From: Florian Weimer @ 2017-04-21 11:19 UTC (permalink / raw)
  To: Christoph Hellwig, Stephen Bates
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha

On 04/21/2017 12:56 PM, Christoph Hellwig wrote:
> It would probably help to add the glibc list as the glibc developers
> might know better..

So far, no one has submitted a patch.

I hope the off_t parameter is passed exactly the same way as for pwritev 
and its 64-bit variant, for all architectures.

Thanks,
Florian

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-21 11:19   ` Florian Weimer
@ 2017-04-21 14:18     ` Adhemerval Zanella
  2017-04-21 14:38       ` Florian Weimer
  2017-04-24 17:56     ` Stephen  Bates
  1 sibling, 1 reply; 9+ messages in thread
From: Adhemerval Zanella @ 2017-04-21 14:18 UTC (permalink / raw)
  To: libc-alpha



On 21/04/2017 08:19, Florian Weimer wrote:
> On 04/21/2017 12:56 PM, Christoph Hellwig wrote:
>> It would probably help to add the glibc list as the glibc developers
>> might know better..
> 
> So far, no one has submitted a patch.
> 
> I hope the off_t parameter is passed exactly the same way as for pwritev and its 64-bit variant, for all architectures.
> 

It seems to have the same kernel abi indeed, I will work on this.  I think
it falls on the criteria for kernel syscalls wrappers [1].

[1] https://sourceware.org/glibc/wiki/Consensus?highlight=%28syscall%29#WIP:_Kernel_syscalls_wrappers

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-21 14:18     ` Adhemerval Zanella
@ 2017-04-21 14:38       ` Florian Weimer
  0 siblings, 0 replies; 9+ messages in thread
From: Florian Weimer @ 2017-04-21 14:38 UTC (permalink / raw)
  To: Adhemerval Zanella, libc-alpha

On 04/21/2017 04:18 PM, Adhemerval Zanella wrote:
> It seems to have the same kernel abi indeed, I will work on this.  I think
> it falls on the criteria for kernel syscalls wrappers [1].

We added the non-standard pwritev interface as a GNU API.  It may make 
sense to do the same to pwritev2, for consistency.

Thanks for working on this,
Florian

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-21 11:19   ` Florian Weimer
  2017-04-21 14:18     ` Adhemerval Zanella
@ 2017-04-24 17:56     ` Stephen  Bates
  2017-04-24 18:44       ` Adhemerval Zanella
  2017-04-25 11:46       ` Adhemerval Zanella
  1 sibling, 2 replies; 9+ messages in thread
From: Stephen  Bates @ 2017-04-24 17:56 UTC (permalink / raw)
  To: Florian Weimer, Christoph Hellwig
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha


> So far, no one has submitted a patch.

OK, unless I hear that someone else is working on one I will take a look at this.

> I hope the off_t parameter is passed exactly the same way as for pwritev 
> and its 64-bit variant, for all architectures.

Duly noted.

For the kernel peeps I think this makes a case for revisiting the “big hammer” control for IO polling since it is going to be a while before applications can utilize the preadv2/pwritev2 approach… I will take a look at that too.

Stephen



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-24 17:56     ` Stephen  Bates
@ 2017-04-24 18:44       ` Adhemerval Zanella
  2017-04-25 11:46       ` Adhemerval Zanella
  1 sibling, 0 replies; 9+ messages in thread
From: Adhemerval Zanella @ 2017-04-24 18:44 UTC (permalink / raw)
  To: libc-alpha



On 24/04/2017 14:55, Stephen  Bates wrote:
> 
>> So far, no one has submitted a patch.
> 
> OK, unless I hear that someone else is working on one I will take a look at this.

I am working on it btw [1]

> 
>> I hope the off_t parameter is passed exactly the same way as for pwritev 
>> and its 64-bit variant, for all architectures.
> 
> Duly noted.
> 
> For the kernel peeps I think this makes a case for revisiting the “big hammer” control for IO polling since it is going to be a while before applications can utilize the preadv2/pwritev2 approach… I will take a look at that too.
> 
> Stephen

[1] https://sourceware.org/ml/libc-alpha/2017-04/msg00461.html

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-24 17:56     ` Stephen  Bates
  2017-04-24 18:44       ` Adhemerval Zanella
@ 2017-04-25 11:46       ` Adhemerval Zanella
  2017-04-25 21:23         ` Stephen  Bates
  1 sibling, 1 reply; 9+ messages in thread
From: Adhemerval Zanella @ 2017-04-25 11:46 UTC (permalink / raw)
  To: Stephen Bates, Florian Weimer, Christoph Hellwig
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha



On 24/04/2017 14:55, Stephen  Bates wrote:
> 
>> So far, no one has submitted a patch.
> 
> OK, unless I hear that someone else is working on one I will take a look at this.
> 
>> I hope the off_t parameter is passed exactly the same way as for pwritev 
>> and its 64-bit variant, for all architectures.
> 
> Duly noted.
> 
> For the kernel peeps I think this makes a case for revisiting the “big hammer” control for IO polling since it is going to be a while before applications can utilize the preadv2/pwritev2 approach… I will take a look at that too.
> 
> Stephen
> 
> 

I am working on it btw [1]

[1] https://sourceware.org/ml/libc-alpha/2017-04/msg00461.html

PS: resending with cc to all.

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-25 11:46       ` Adhemerval Zanella
@ 2017-04-25 21:23         ` Stephen  Bates
  2017-04-25 22:42           ` Adhemerval Zanella
  0 siblings, 1 reply; 9+ messages in thread
From: Stephen  Bates @ 2017-04-25 21:23 UTC (permalink / raw)
  To: Adhemerval Zanella, Florian Weimer, Christoph Hellwig
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha


> I am working on it btw [1]

Thanks for working on this Adhemerval!

> PS: resending with cc to all.

Can you cc linux-block when you submit the patchset?

Stephen


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Status of pwritev2/preadv2 in glibc?
  2017-04-25 21:23         ` Stephen  Bates
@ 2017-04-25 22:42           ` Adhemerval Zanella
  0 siblings, 0 replies; 9+ messages in thread
From: Adhemerval Zanella @ 2017-04-25 22:42 UTC (permalink / raw)
  To: Stephen Bates, Florian Weimer, Christoph Hellwig
  Cc: linux-block, Jens Axboe, Christoph Hellwig, jonathan.derrick, libc-alpha

On 25/04/2017 18:23, Stephen  Bates wrote:
> 
>> I am working on it btw [1]
> 
> Thanks for working on this Adhemerval!
> 
>> PS: resending with cc to all.
> 
> Can you cc linux-block when you submit the patchset?

Sure, I am just finishing the sanity tests on the supported architectures.

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2017-04-25 22:42 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <ECD1C373-3464-4AEF-8E73-3130C5BFAF87@raithlin.com>
2017-04-21 10:56 ` Status of pwritev2/preadv2 in glibc? Christoph Hellwig
2017-04-21 11:19   ` Florian Weimer
2017-04-21 14:18     ` Adhemerval Zanella
2017-04-21 14:38       ` Florian Weimer
2017-04-24 17:56     ` Stephen  Bates
2017-04-24 18:44       ` Adhemerval Zanella
2017-04-25 11:46       ` Adhemerval Zanella
2017-04-25 21:23         ` Stephen  Bates
2017-04-25 22:42           ` Adhemerval Zanella

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).