Closed Bug 64471 Opened 19 years ago Closed 19 years ago

PR_SendFile should use sendfilev.

Categories

(NSPR :: NSPR, defect, P3)

Sun
Solaris
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(2 files)

PR_SendFile should detect at run time whether
the new Solaris sendfilev system call is
available and use it.
Attached patch Proposed patch.Splinter Review
I checked in the patch on the main trunk and
NSPRPUB_RELEASE_4_1_BRANCH.
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → 4.1.1
During testing, I found that sendfilev may have sent some
bytes when it fails with EAGAIN/EWOULDBLOCK.  So it is
necessary to use the value of the 'xferred' argument in
that case.  I just attached a patch (id=22333) that does
this.

I checked in this patch on the tip and the
NSPRPUB_RELEASE_4_1_BRANCH.  Tested it in both 32-bit and
64-bit builds.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.