Closed Bug 64471 Opened 24 years ago Closed 24 years ago

PR_SendFile should use sendfilev.

Categories

(NSPR :: NSPR, defect, P3)

Product:
NSPR
Component:
NSPR
Platform:
Sun
Solaris
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(2 files)

Proposed patch.
9.90 KB, patch
Details | Diff | Splinter Review
Addtional patch. Use the 'xferred' argument when sendfilev fails with EAGAIN.
1.58 KB, patch
Details | Diff | Splinter Review 
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: 24 years ago
Resolution: --- → FIXED

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: