Re: [PATCH 1/3] make sparse keep its promise about context tracking

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

 



> > and you'd end up with one context named "dbl" and another one named
> > "&dev_base_lock".
> 
> That might still work, depending on how consistently kernel code uses
> the same argument.  If you suggest explicitly changing calls to
> read_lock to call __acquire__ with the appropriate context, it might
> prove equally easy to make the argument of read_lock that context.

True.

> That doesn't make sense to me; after preprocessing, x no longer
> exists, so I don't see how you could pick up the identifier "x".  I
> can understand that you might pick up two different expressions in
> place of x which you can't easily compare, though.

Oops, yes, preprocessing will remove it, of course. But yeah, you can
end up with two different expressions.

> And as for how to solve it: I think alias analysis might work.

How would I do that?

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux