Hi Dan, On Fri, Aug 30, 2019 at 10:06:25AM +0800, Chao Yu wrote: > On 2019/8/29 23:43, Dan Carpenter wrote: > >> p.s. There are 2947 (un)likely places in fs/ directory. > > > > I was complaining about you adding new pointless ones, not existing > > ones. The likely/unlikely annotations are supposed to be functional and > > not decorative. I explained this very clearly. > > > > Probably most of the annotations in fs/ are wrong but they are also > > harmless except for the slight messiness. However there are definitely > > some which are important so removing them all isn't a good idea. > > Hi Dan, > > Could you please pick up one positive example using likely and unlikely > correctly? so we can follow the example, rather than removing them all blindly. I'm also curious about that, what is the filesystem or kernel standard about likely/unlikely use (since I didn't find some documented standard so I used in my personal way, I think it is reasonable at least to cover all error handling paths), maybe I'm an _idiot_ as some earlier unfriendly word said somewhere so I'm too stupid to understand the implicit meaning of some document. > > Thanks, > > > > >> If you like, I will delete them all. > > > > But for erofs, I don't think that any of the likely/unlikely calls have > > been thought about so I'm fine with removing all of them in one go. Add a word (just a note), I don't think such kind of "any", "few", "all" words are meaningful without some explicit evidence. (e.g. such as EROFS have few error handling path. I don't think that is true and worth to give many details since EROFS code is here for more than one year.) Yes, EROFS is not prefectly, I have to admit, and I said similar words on other threads for many times if you decide to check each likely/unlikely line by line, I cannot say all unlikely/likely cases I wrote are reasonable (just as bug-free, I think no one can make such guarantee even for new code), but I can say the majority of them are reasonable in my personal understanding of likely/unlikely. And I can fix all your reports in time (but maybe some are not urgent at all.) In addition there will be endless discussions on detailed code since there are many personal tendencies from various people in it, as the saying goes "There are a thousand Hamlets in a thousand people's eyes. " Anyway, I have sent a patch to kill them all blindly as you like, so I think we can come to an agreement on it, but I still don't fully agree with your "for EROFS, I don't think that any of the likely/unlikely calls have been thought about" conclusion. Thanks, Gao Xiang > > > > regards, > > dan carpenter > > > > . > >
