Message ID | 20210121172201.22582-3-david.hunt@intel.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Thomas Monjalon |
Headers | show |
Series | power: fix make build for power apps | expand |
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
On 21-Jan-21 5:21 PM, David Hunt wrote: > From: Bruce Richardson <bruce.richardson@intel.com> > > Move the 2 public functions into rte_power_guest_channel.h > > Fixes: 210c383e247b ("power: packet format for vm power management") > Fixes: cd0d5547e873 ("power: vm communication channels in guest") > Cc: stable@dpdk.org > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > Signed-off-by: David Hunt <david.hunt@intel.com> > > --- > changes in v3 > * Mark both added functions as experimental > changes in v4 > * add @internal tag for Doxygen > * add @warning EXPERIMENTAL tag for Doxygen > * improve description of lcore param > --- > lib/librte_power/guest_channel.h | 40 +---------------- > lib/librte_power/rte_power_guest_channel.h | 50 ++++++++++++++++++++++ > 2 files changed, 51 insertions(+), 39 deletions(-) > > diff --git a/lib/librte_power/guest_channel.h b/lib/librte_power/guest_channel.h > index d3d87f0ae..69020b030 100644 > --- a/lib/librte_power/guest_channel.h > +++ b/lib/librte_power/guest_channel.h > @@ -65,21 +65,7 @@ void guest_channel_host_disconnect(unsigned int lcore_id); > */ > int guest_channel_send_msg(struct channel_packet *pkt, unsigned int lcore_id); > > -/** > - * Send a message contained in pkt over the Virtio-Serial to the host endpoint. > - * > - * @param pkt > - * Pointer to a populated struct channel_packet > - * > - * @param lcore_id > - * lcore_id. > - * > - * @return > - * - 0 on success. > - * - Negative on error. > - */ > -int rte_power_guest_channel_send_msg(struct channel_packet *pkt, > - unsigned int lcore_id); > + > Nitpick but you left an extra newline there :) can probably be fixed on apply, so Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
On 1/21/2021 5:21 PM, David Hunt wrote: > From: Bruce Richardson <bruce.richardson@intel.com> > > Move the 2 public functions into rte_power_guest_channel.h > > Fixes: 210c383e247b ("power: packet format for vm power management") > Fixes: cd0d5547e873 ("power: vm communication channels in guest") > Cc: stable@dpdk.org > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > Signed-off-by: David Hunt <david.hunt@intel.com> <...> > index ef3b064a8..c500c0cda 100644 > --- a/lib/librte_power/rte_power_guest_channel.h > +++ b/lib/librte_power/rte_power_guest_channel.h > @@ -116,6 +116,56 @@ struct channel_packet_caps_list { > uint8_t num_vcpu; > }; > > +/** > + * @internal > + * > + * @warning > + * @b EXPERIMENTAL: this API may change without prior notice. > + * > + * Send a message contained in pkt over the Virtio-Serial to the host endpoint. > + * > + * @param pkt > + * Pointer to a populated struct channel_packet. > + * > + * @param lcore_id > + * Use channel specific to this lcore_id. > + * > + * @return > + * - 0 on success. > + * - Negative on error. > + */ > +__rte_experimental > +int rte_power_guest_channel_send_msg(struct channel_packet *pkt, > + unsigned int lcore_id); > + > +/** > + * @internal > + * > + * @warning > + * @b EXPERIMENTAL: this API may change without prior notice. > + * > + * Receive a message contained in pkt over the Virtio-Serial > + * from the host endpoint. > + * > + * @param pkt > + * Pointer to channel_packet or > + * channel_packet_freq_list struct. > + * > + * @param pkt_len > + * Size of expected data packet. > + * > + * @param lcore_id > + * Use channel specific to this lcore_id. > + * > + * @return > + * - 0 on success. > + * - Negative on error. > + */ > +__rte_experimental > +int rte_power_guest_channel_receive_msg(void *pkt, > + size_t pkt_len, > + unsigned int lcore_id); > + As far as I can see these function are public, so why they have '@internal' tag?
diff --git a/lib/librte_power/guest_channel.h b/lib/librte_power/guest_channel.h index d3d87f0ae..69020b030 100644 --- a/lib/librte_power/guest_channel.h +++ b/lib/librte_power/guest_channel.h @@ -65,21 +65,7 @@ void guest_channel_host_disconnect(unsigned int lcore_id); */ int guest_channel_send_msg(struct channel_packet *pkt, unsigned int lcore_id); -/** - * Send a message contained in pkt over the Virtio-Serial to the host endpoint. - * - * @param pkt - * Pointer to a populated struct channel_packet - * - * @param lcore_id - * lcore_id. - * - * @return - * - 0 on success. - * - Negative on error. - */ -int rte_power_guest_channel_send_msg(struct channel_packet *pkt, - unsigned int lcore_id); + /** * Read a message contained in pkt over the Virtio-Serial @@ -103,30 +89,6 @@ int power_guest_channel_read_msg(void *pkt, size_t pkt_len, unsigned int lcore_id); -/** - * Receive a message contained in pkt over the Virtio-Serial - * from the host endpoint. - * - * @param pkt - * Pointer to channel_packet or - * channel_packet_freq_list struct. - * - * @param pkt_len - * Size of expected data packet. - * - * @param lcore_id - * lcore_id. - * - * @return - * - 0 on success. - * - Negative on error. - */ -__rte_experimental -int -rte_power_guest_channel_receive_msg(void *pkt, - size_t pkt_len, - unsigned int lcore_id); - #ifdef __cplusplus } diff --git a/lib/librte_power/rte_power_guest_channel.h b/lib/librte_power/rte_power_guest_channel.h index ef3b064a8..c500c0cda 100644 --- a/lib/librte_power/rte_power_guest_channel.h +++ b/lib/librte_power/rte_power_guest_channel.h @@ -116,6 +116,56 @@ struct channel_packet_caps_list { uint8_t num_vcpu; }; +/** + * @internal + * + * @warning + * @b EXPERIMENTAL: this API may change without prior notice. + * + * Send a message contained in pkt over the Virtio-Serial to the host endpoint. + * + * @param pkt + * Pointer to a populated struct channel_packet. + * + * @param lcore_id + * Use channel specific to this lcore_id. + * + * @return + * - 0 on success. + * - Negative on error. + */ +__rte_experimental +int rte_power_guest_channel_send_msg(struct channel_packet *pkt, + unsigned int lcore_id); + +/** + * @internal + * + * @warning + * @b EXPERIMENTAL: this API may change without prior notice. + * + * Receive a message contained in pkt over the Virtio-Serial + * from the host endpoint. + * + * @param pkt + * Pointer to channel_packet or + * channel_packet_freq_list struct. + * + * @param pkt_len + * Size of expected data packet. + * + * @param lcore_id + * Use channel specific to this lcore_id. + * + * @return + * - 0 on success. + * - Negative on error. + */ +__rte_experimental +int rte_power_guest_channel_receive_msg(void *pkt, + size_t pkt_len, + unsigned int lcore_id); + #ifdef __cplusplus }