Skip to content

Commit 1b51122

Browse files
committed
Do not log in the hot path
1 parent 17c55ee commit 1b51122

File tree

1 file changed

+2
-7
lines changed
  • vm/devices/net/net_mana/src

1 file changed

+2
-7
lines changed

vm/devices/net/net_mana/src/lib.rs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1331,8 +1331,8 @@ impl<T: DeviceBacking> ManaQueue<T> {
13311331
// the header and should not exceed 256 bytes. Otherwise, it treats
13321332
// the WQE as "corrupt", disables the queue and return GDMA error.
13331333
// To meet the hardware requirements, do not coalesce SGE0 when LSO is enabled.
1334-
let coalesce_sge_possible =
1335-
!meta.flags.offload_tcp_segmentation() || tail_idx > header_segment_count;
1334+
let coalesce_sge_possible = !meta.flags.offload_tcp_segmentation()
1335+
|| tail_idx > header_segment_count;
13361336
if !last_segment_bounced
13371337
&& coalesce_sge_possible
13381338
&& coalesce_possible
@@ -1403,11 +1403,6 @@ impl<T: DeviceBacking> ManaQueue<T> {
14031403
let check_lso_segment_count = !self.test_configuration.allow_lso_pkt_with_one_sge;
14041404

14051405
if check_lso_segment_count && meta.flags.offload_tcp_segmentation() && sgl.len() == 1 {
1406-
tracelimit::error_ratelimited!(
1407-
sgl_len = sgl.len(),
1408-
"LSO packets should have more than one SGL entry"
1409-
);
1410-
14111406
return Ok(None);
14121407
}
14131408

0 commit comments

Comments
 (0)