Differential D5772

Remove useless NULL checks.
ClosedPublic
Actions

Authored by trasz on Mar 29 2016, 11:51 AM.

Details

Reviewers

• hselasky

Summary

Remove NULL checks after M_WAITOK allocations.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4752
Build 4806: arc lint + arc unit

Event Timeline

trasz updated this revision to Diff 14707.Mar 29 2016, 11:51 AM

trasz retitled this revision from to Remove useless NULL checks..

trasz updated this object.

trasz edited the test plan for this revision. (Show Details)

Herald added a subscriber: imp. · View Herald TranscriptMar 29 2016, 11:51 AM

trasz added a reviewer: • hselasky.Mar 29 2016, 11:59 AM

Looks good. Can I commit it, because I need to sync this patch into an internal repository first.

Sure!

Hi. Remember about this?

Hi,

I need some more time with this patch. It makes a conflict with some internal patches I've got.

Can you just leave it open and remind me again?

--HPS

Fix conflict.

I plan to submit this one soon. Stay tuned. Sorry it has taken so long.

Ping?

Pong - I'll try to get this patch processed.

• hselasky accepted this revision.Dec 2 2016, 9:42 AM

• hselasky edited edge metadata.

This revision is now accepted and ready to land.Dec 2 2016, 9:42 AM

Committed to FreeBSD SVN r309406.

Revision Contents
Changeset List

Path

Size

sys/

dev/

mlx5/

mlx5_en/

mlx5_en_flow_table.c

4 lines

mlx5_en_main.c

16 lines

Diff 19187

View Options

sys/dev/mlx5/mlx5_en/mlx5_en_flow_table.c