Re: [PATCH qxl-wddm-dod v5 2/7] Code Analysis fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 27, 2016 at 01:01:22PM +0300, Dmitry Fleytman wrote:
> 
> > On 27 Sep 2016, at 12:35 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > 
> > Hey,
> > 
> > A huge part of this patch is adding +__declspec(code_seg("PAGE"))  (and
> > removing #pragma code_seg()), which are probably related to this macro,
> > but which are added with no rationale.
> > The shortlog should probably be something different than "code analysis
> > fix", but more somethnig like "Use PAGED_CODE() at the beginning of each
> > pageable method".
> > Is it possible/desirable to prevent these methods from ending in
> > pageable memory?
> 
> Hi Christophe,
> 
> #pragma code_seg() and __declspec(codes()) are interchangeable,
> in this specific case __declspec is more convenient.

Isn't this something we want for mostly the whole file? Anyway, I'd
split this part to a different commit.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]