On Wed, 8 Mar 2023 12:22:13 +0000 Sarkar, Tirthendu wrote: > > On Mon, 6 Mar 2023 13:08:16 -0800 Tony Nguyen wrote: > > > In the legacy-rx mode, driver can only configure up to 2k sized Rx buffers > > > and with the current configuration of 2k sized Rx buffers there is no way > > > to do tailroom reservation for skb_shared_info. Hence size of Rx buffers > > > is now lowered to 1664 (2k - sizeof(skb_shared_info)). Also, driver can > > > > skb_shared_info is not fixed size, the number of fragments can > > be changed in the future. What will happen to the driver and > > this assumption, then? > > > > This is for the non-default path in legacy mode. If for some reason number of > fragments increase in future, we may have to think of other options like using > page pools. Is it possible to add runtime checks so that in configurations where the shinfo geometry does not match expectations we'd cleanly error out? Better to be too careful than risk silent memory corruptions..
