Re: [patch PE-COFF]: Some minor nits

[Kai Tietz <Kai dot Tietz at onevision dot com>]

Dave Korn <dave.korn.cygwin@googlemail.com> wrote on 17.03.2009 00:15:12:

> Kai Tietz wrote:
> > Hello,
> 
>   Hi Kai,
> 
> > Some minor nits I found in coffcode.h. One is that debugging sections 
have 
> > to have readonly flag set (abstracted by the section flags of '.debug' 
in 
> > pe-coff spec).
> 
>   Can you point us at a reference for this?  And have you run the GDB
> testsuite to see if it causes any changes?  (As I discovered recently,
> sometimes the other tools have come to depend on these minor 
non-compliances
> of ours!)

Well, see pe-coff-v8 specification in section "46 Special Sections". Here 
are the used section flags given.
I tested it with gdb for w64 and found no regression here. Other platforms 
I didn't tested with gdb, just by binutil's testsuite.
> > Secondly a clean-up about NOREAD sections. 
> 
>   http://www.national.com/rap/files/datasheet.pdf :)
> 
> > I introduced a helper flag for 
> > section's flags to indicated, that a section has no memory flag set,
> > I tested this patch for i686-pc-cygwin and for x86_64-pc-ming32 and 
saw no 
> > regressions on testrun.
> 
>   Does anything actually currently use NOREAD?

Heh, I don't thought about write-only ram. It is more near than that. See 
e.g ".cormeta" (for .NET), ".idlsym", and ".drective".

>     cheers,
>       DaveK
> 

Cheers,
Kai

|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.