From: David Gibson <david@gibson.dropbear.id.au>
To: Stefano Brivio <sbrivio@redhat.com>
Cc: Anshu Kumari <anskuma@redhat.com>,
passt-dev@passt.top, lvivier@redhat.com, jmaloy@redhat.com
Subject: Re: [PATCH 0/6] Add --dhcp-boot and --dhcp-opt options
Date: Wed, 20 May 2026 11:31:14 +1000 [thread overview]
Message-ID: <ag0O4iDWpsL7Pcn4@zatzit> (raw)
In-Reply-To: <20260520023804.131ef7ad@elisabeth>
[-- Attachment #1: Type: text/plain, Size: 1904 bytes --]
On Wed, May 20, 2026 at 02:38:04AM +0200, Stefano Brivio wrote:
> On Tue, 19 May 2026 15:30:41 +1000
> David Gibson <david@gibson.dropbear.id.au> wrote:
>
> > On Mon, May 18, 2026 at 06:49:56PM +0530, Anshu Kumari wrote:
> > > This series adds support for custom DHCP options in passt, enabling
> > > network boot (PXE/UEFI HTTP Boot) and arbitrary DHCP option injection.
> > >
> > > Two new command-line flags are introduced:
> > >
> > > --dhcp-boot URL Sets the boot file URL (DHCP option 67 and the
> > > legacy boot file field)
> > >
> > > --dhcp-opt CODE,VALUE
> > > Sets any DHCP option by numeric code, with
> > > type-aware parsing per RFC 2132
> > >
> > > The DHCP reply path is extended with option overload support (RFC 2132
> > > option 52), allowing options to overflow into the file and sname fields
> > > when the standard options area is full.
> > >
> > > *** BLURB HERE ***
> >
> > Nit: remember to remove this boilerplate that git-publish inserts :)
>
> No, no, that's from git format-patch, I swear (as somebody who doesn't
> use git-publish).
Oh, my mistake. Same thought applies, though.
> Another detail, while at it: for some reason git send-email picked
> the "chain-reply" style, that is, if you look at this in a threaded
> email client, patch #1 is a reply to the cover letter (fine, expected)
> but patch #2 is a reply to patch #1 (not expected), and so on.
>
> That's the --chain-reply-to option for git send-email. You should
> disable that (for example in gitconfig in case you have it on there)
> and just use --cover-letter.
>
> --
> Stefano
>
--
David Gibson (he or they) | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you, not the other way
| around.
http://www.ozlabs.org/~dgibson
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2026-05-20 1:33 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-18 13:19 Anshu Kumari
2026-05-18 13:19 ` [PATCH 1/6] conf: Add --dhcp-opt command-line option Anshu Kumari
2026-05-18 13:19 ` [PATCH 2/6] conf: Add --dhcp-boot " Anshu Kumari
2026-05-18 13:19 ` [PATCH 3/6] dhcp: Add option type table and value parser Anshu Kumari
2026-05-18 13:20 ` [PATCH 4/6] dhcp: Refactor fill_one() to operate on a generic buffer Anshu Kumari
2026-05-18 13:20 ` [PATCH 5/6] dhcp: Add option overload Anshu Kumari
2026-05-18 13:20 ` [PATCH 6/6] doc: Add --dhcp-boot and --dhcp-opt to man page Anshu Kumari
2026-05-19 6:11 ` [PATCH 5/6] dhcp: Add option overload David Gibson
2026-05-19 6:02 ` [PATCH 4/6] dhcp: Refactor fill_one() to operate on a generic buffer David Gibson
2026-05-19 5:59 ` [PATCH 3/6] dhcp: Add option type table and value parser David Gibson
2026-05-20 0:39 ` Stefano Brivio
2026-05-19 5:35 ` [PATCH 2/6] conf: Add --dhcp-boot command-line option David Gibson
2026-05-19 5:33 ` [PATCH 1/6] conf: Add --dhcp-opt " David Gibson
2026-05-20 0:38 ` Stefano Brivio
2026-05-19 5:30 ` [PATCH 0/6] Add --dhcp-boot and --dhcp-opt options David Gibson
2026-05-20 0:38 ` Stefano Brivio
2026-05-20 1:31 ` David Gibson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ag0O4iDWpsL7Pcn4@zatzit \
--to=david@gibson.dropbear.id.au \
--cc=anskuma@redhat.com \
--cc=jmaloy@redhat.com \
--cc=lvivier@redhat.com \
--cc=passt-dev@passt.top \
--cc=sbrivio@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).